In the tiny version, do_prompt() will now have an extra NULL
parameter, which will cost maybe twenty extra bytes of code.
That is acceptable when it saves thirty lines in the source.

Adjust to the changed meaning of 'whereis': it is no longer do_search()
but do_search_forward().
Hinted-at-by: Brand Huntsman <alpha@qzx.com>

Simply redraw the browser screen upon every keystroke when --showcursor
is in effect, to be certain that the cursor will be placed again at the
correct spot.

This fixes https://savannah.gnu.org/bugs/?52078
and fixes https://savannah.gnu.org/bugs/?52079

.
Reported-by: David Lawrence Ramsey <pooka109@gmail.com>

The basic idea is that the cursor is always off, except when it needs
to be on: when waiting for text input, and in a few other cases: when
something was searched and found in the help viewer, and in the file
browser when option -g is in effect.

This fixes https://savannah.gnu.org/bugs/?51923

.
Reported-by: Mike Frysinger <vapier@gentoo.org>

This should have been added and amended in the previous commit --
my mistake.

Also bind 'N' to a backward re-search, while retaining 'n' for a
forward one,  This matches the functionality in the help viewer.

Further, slightly reword an error message so it is appropriate also
when an out-of-bounds file is specified on the command line.

There is no need to retain the (relative) path that the user
specified, so we can simply reuse that variable.

The new option 'set selectedcolor' applies to marked text, to the
currently selected file in the file browser, and to the highlighted
match during interactive search-and-replace.

Without them, nano still compiles for me, with everything enabled,
even when using --enable-debug, --enable-utf8, and --with-slang.

Also, allow Ctrl+Left/Right with --enable-tiny --enable-browser.

Also, trim some comments and avoid an unused-variable warning.

And in the process transform the token DISABLE_HELP to ENABLE_HELP.

And allow 'N' for searching the next occurrence.  Add the
same keystrokes to the file browser too, for consistency.

The interval 2013-2017 for the Free Software Foundation is valid
because in those years there were releases with changes by either
Chris or David, and the GNU maintainers guide advises to mention
a new year in all files of a package, not just in the ones that
actually changed, and be done with it for the rest of the year.

Making ^Up and ^Down go to top and bottom row in the file browser
complements and completes the behavior of ^Left and ^Right.

"Q" is a pretty standard key to exit from something, and "X" is fairly
mnemonic -- better than "E" at least.

Most of these variables are freed moments later -- reallocating
them is thus a waste of time.

The conditionalizing saved negligible amounts of space, of memory, and
of speed.

In that case setting the latter to TRUE is misleading.  So don't do that.

This fixes https://savannah.gnu.org/bugs/?49884.

And certainly don't return zero when the key wouldn't have been found,
because that would have meant jumping a word to the right.

Also, swap the logic around, to use less braces.

Because changing anything to a null effectively means to truncate
the name.

This fixes https://savannah.gnu.org/bugs/?49868
and fixes https://savannah.gnu.org/bugs/?49874.

Most full paths are needed only temporarily and will be freed within
milliseconds.  Only 'full_operating_dir' and 'backup_dir' continue to
exist for the whole current session.  Any partition, too, will soon be
unpartitioned, so the extra reallocation is just a waste of time.

It doesn't align anything -- any allocations are already aligned to
whatever multiple is required -- it just shrinks the allocated space.

And which normally is just some ten or twenty characters long, and
never gets wildly overallocated.

This fixes https://savannah.gnu.org/bugs/?48622.