folder_free.c: Move interface to own file.

folder_delmsgs.c: Move interface to own file.

folder_addmsg.c: Move interface to own file.

context_foil.c: Move interface to own file.

context_save.c: Move interface to own file.

context_replace.c: Move interface to own file.

context_read.c: Move interface to own file.

context_del.c: Move interface to own file.

context_find.c: Move interface to own file.

readconfig.c: Move interface to own file.

brkstring.c: Move interface to own file.

atooi.c: Move interface to own file.

ambigsw.c: Move interface to own file.

vector.c: Move interface to own file.

Don't #include it in all its users yet as it's too tangled with mh.h's
other content, so just #include it in mh.h instead.

pidstatus.c: Move interface to own file.

push.c: Move interface to own file.

path.c: Move interface to own file.

print_version.c: Move interface to own file.

print_sw.c: Move interface to own file.

print_help.c: Move interface to own file.

seq_nameok.c: Move interface to own file.

seq_list.c: Move interface to own file.

seq_bits.c: Move interface to own file.

seq_getnum.c: Move interface to own file.

seq_del.c: Move interface to own file.

seq_print.c: Move interface to own file.

seq_add.c: Move interface to own file.

arglist.c: Move interface to own file.

Strip directory from #include when it's not needed.

Remove symbols-wanted comments after #include.

They're unlikely to be up to date, and they are few.

Use #include with `""' for including nmh's source.

Compilers vary on how they handle #includes with `""' and `<>', but
consistency on using `""' for nmh's files, and `<>' for the system's,
means it's easy to tell which is which at a glance.

error.c: Move interface to own file.

Support a -timeout switch.

Use va_copy() to get a copy of va_list, instead of using original.

netsec_vprintf() can call vsnprintf() twice if the outgoing buffer is
full (but it happens rarely in practice, given the way the current
code uses it).  But if this DOES happen, vsnprintf() will use the
same va_list argument twice, and the second time around either it will
grab a random bit of memory off of the stack OR it will segfault.
So we always use va_copy() to get our own copy of the passed-in va_list
and work on that.

Add support for batch mode.

Add a new switch, -batch, which will run a sequence of commands out
of a batch file.

Various IMAP protocol improvements

Make the CAPABAILITY parser work correctly (and not include CAPABILITY
as a capability).

Support reading an untagged CAPABILITY response during an authentication
exchange.

Removing some debugging statements.

Make LITERAL+ and LITERAL- work correctly.

Make APPEND work; still needs some changes, though.

Start of making APPEND functional; not quite there yet.

Support sending multiple commands at once.

Report on the TLS negotation time

prototypes.h: Delete duplicate prototypes.

Those that already appear in other header files.

prototypes.h: Delete default_done() prototype.

Function doesn't exist;  removed in 38615191.

forwsbr.c: Move interface declaration to own forwsbr.h.

md5.c, md5.h: Reformat and delete needless portability.

The algorithm is not going to change, bugs are unlikely to appear in the
upstream RSA version, so reformat the source to be more readable, and
delete portability code now we have function prototypes with parameters,
void pointer, etc.

Split function definition prototypes so identifier starts a line.

Nearly all were already split like this.  The consistency helps ad hoc
scripting.

Don't have `static' of a function definition on its own line.

Join it to the next line that species the return type.

Move the opening brace of a C function to its own line.

If the brace doesn't start a line of its own then vi(1)'s
function-related commands don't spot the function.

repl.c: Add `static' to function definitions to match declarations.

repl.c, replsbr.c: Move shared data into header file.

repl.c declared global variables, and replsbr.c had extern statements
that hopefully matched them.  But they're really inputs into replsbr.c's
code, so move the declarations into a new header file that both then
include, and define them in replsbr.c.

uip: Replace some ints that are only ever 0 or 1 with bool.

Knowing the domain of the variable is Boolean aids comprehension of the
variable's name and purpose.

send.c, whatnowsbr.c: Delete option variables set but never read.

Some options are passed on without the code parsing the options having
to act upon their value.  These don't need to store the option's Boolean
value in a local variable that's never read.

Merge branch 'master' of git.sv.gnu.org:/srv/git/nmh

Clean up things a bit

uip: Replace some ints that are only ever 0 or 1 with bool.

Knowing the domain of the variable is Boolean aids comprehension of the
variable's name and purpose.

mhbuild.c, mhbuildsbr.c: Replace three global ints with bools.

Knowing the domain of the variable is Boolean aids comprehension of the
variable's name and purpose.

Replace some ints that are only ever 0 or 1 with bool.

Knowing the domain of the variable is Boolean aids comprehension of the
variable's name and purpose.

dtimep.l: Delete unused macros SKIPSP and SKIPTOSP.

Silences gcc's warnings.

Display the command when reporting timing.

uip: Change global int userrs to bool.

uip: Replace some ints that are only ever 0 or 1 with bool.

Knowing the domain of the variable is Boolean aids comprehension of the
variable's name and purpose.

sbr: Replace some ints that are only ever 0 or 1 with bool.

Knowing the domain of the variable is Boolean aids comprehension of the
variable's name and purpose.

mts/smtp/smtp.c: Replace int with bool.

nmh_init(): Replace integer parameter with two bools.

Removes the 0, 1, or 2 passed by callers.  If the options get more
complex then an enum of possible actions might be a better fit.

Replace adios(NULL, ...) with die(...).

It's the common case, and saves having to skip over the NULL when
reading, and avoids the risk of the format string being passed as the
first parameter by mistake.

Replace printf(3) with fputs(3) where possible.

The format string has no format specifiers, and doesn't end with `\n',
so fputs() to stdout is sufficient.

mhbuild.man: Avoid using format-free printf(3) in example.

It pops up in searches for printf()s that could be fputs(),
and as printf() it doesn't output the trailing `\n' to end the line.
Replace with puts().  Lowercase Kernighan's message, as he wrote it.

msgchk.c: Rewrite checkmail(), removing need for macros.

NONEOK, MMDFOLD, etc., were only used to track simple Boolean
conditions.  Rewriting the function removes the need for them, and gives
shorter, more clear, code.  The macros pre-date git's history.

SPECS: Use `command-line interface' in one-line summary.

Requested by David on nmh-workers.  `shell' is still mentioned in the
description for searches.

whatnow: Remove undocumented printf-formatting of -prompt string.

whatnow(1)'s -prompt option would use the value as a printf(3) format
string, supplying `invo_name' as the sole value to be formatted.  This
was undocumented, and upset gcc's checking of format strings.  As no
requests to keep the ability appeared on nmh-workers in reply to
2017-08-09's email, treat -prompt's value as a simple string.

SPECS: Alter description to not require a GUI interface.

xmh or exmh are optional, not required to use nmh.  Mention them both,
but remove the dependency on whether either is available as a package.
Add possible search terms, e.g. `MIME' and `shell'.

configure.ac: Test for libtinfo to resolve setupterm(3).

ncurses 6.0+20170902-2 on Arch Linux doesn't provide setupterm();  it
did in earlier versions.  This stops ./configure finding a suitable
library.  Add tinfo to the search list, as that's the lower-level
library that contains setupterm() now, and place it before curses and
ncurses so the search stops if it's sufficient instead of the larger
libraries.

msgchk.c: Remove UUCP tests; unused since `#ifdef MF'.

The values UUCPOLD et al are never set so don't bother testing for them.
They haven't be possible in git's history.  They were guarded in the
past by `#ifdef MF';  that stood for `mail filter'.

Replace getcpy(f()) calls with mh_xstrdup() when f() can't be NULL.

And comment a few functions as never returning NULL along the way.

Delete "blank" lines at start of files to bring content into view.

utils.c: Improve comment describing add().  Suggest mh_xstrdup().

imaptest.c: Don't use error message as adios()'s format.

Add a `"%s"' format string.  Stops gcc complaining.

imaptest.c: Don't test possibly uninitialised local variable.

get_imap_response() doesn't necessarily set the content of a non-NULL
`char **status'.  Do this before calling, and check if it's been set to
non-NULL before using upon return.

imaptest.c: Don't free brkstring()'s value; it's re-used.

Subsequent calls to brkstring() write to free'd memory, corrupting
malloc()'s internal data.

imaptest.c: Add `format(printf, ...)' attribute to functions.

Stops warnings from gcc for send_imap_command() and add_msg().

Fix invalid pointer arithmetic.

Reported by David Levine.

Support command issuance and timestamps.

Still more fiddling.

Add netsec_get_sasl_ssf().

Add the function netsec_get_sasl_ssf(), which retrieves the SSF
(strength security factor) negotiated as part of the SASL exchange.

A little closer to being somewhat useful.

Make snoop work like it used to for SMTP.

Fix up some const and unsigned char warnings.

Improved, a bit.  Slightly more useful.

Rework snoop support slightly for the new world order.

Make sure we do base64 snoop decoding at flush time.  Still need to
fix SMTP support.

Rototill snoop support so it works at flush time.

Before on writes, the output snoop support would only work when it was
doing netsec_vprintf().  Do it at flush time, so it should work for
any type of write.

It does just what it says on the tin.

You should not use this unless you're the sort of person who regularly
reads the commit log. Still incomplete for now.

Add context information to SASL callback.

Provide an extra context argument for the SASL negotiation callback
that application code can use to pass down information that the
callback might need.

Switch to the use of to_upper() instead of custom code.

Improve this documentation a bit.

Remove mmdf_magic and empty_line temp files on success.

Fixes commit b46d49ba.

picksbr.c: fprintf(3) function pointer using `%p' and union.

Using `%#llx' and cast to `unsigned long long' caused Cygwin woes.
Fixes 07916b2ff.

slocal.c: Use C99's `%td' to printf(3) ptrdiff_t values.

Using `%ld' caused Cygwin woes.  Fixes 6015bb1f7.

inc/test-eom-align: Cut down on the flicker by re-ordering tests.

inc/test-eom-align: Replace case on $MBOX_TYPE with interpolation.

inc/test-eom-align: rm(1) unwanted +inbox emails, not rmm(1).

Using rmm(1) is using the software under test.  And about 50 times
slower.

inc/test-eom-align: Create test mboxes in less steps.

Use a couple of temporary files created at the beginning to allow runs
of alternating cat(1) and echo(1) to be a single cat.  Knocks 15% off
the `make check' time for that single test in a simple trial here.