Fixed MIME parser so it can read binary content.

Second of two-part fix.  Replaced use of strlen().

Have m_getfld() pass back indication of missing blank line.

This is part one of a two part fix to the MIME parser to
read binary content.  It should have no effect.

Also added start/finish test calls to test-header.

Fix warnings caused by recent edits;  don't mix declarations and code.
Quite right too, I prefer all the declarations, codeless, up front.  The
standard -Wall didn't pick this up.  David pointed it out, probably due
to his defaults of `-pedantic -ansi'.

Don't `else' after goto.  Simplify control flow.

Don't `else' after goto.  Simplify control flow.

Don't `else' after goto.  Simplify control flow.

Don't `else' after goto.  Simplify control flow.

Added test/mhshow/test-binary.  The MIME parser doesn't
properly handle null bytes because it uses getline().

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return or goto.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Hoist test of search string for NULL out of loop.

Rototill credentials code so that we only prompt for the username and
password when it is necessary.

Added locale profile component, value is passed to nmh_init().

Pass CFLAGS to make distcheck via DISTCHECK_CONFIGURE_FLAGS.

Replace sizeof string constant - 1 with LEN().

Replace sizeof string constant - 1 with LEN().

Replace strlen(), malloc(), and sprintf() with concat().

Use LEN() on string constants instead of strlen().

Use LEN() on string constants instead of strlen().

Use LEN() on string constants instead of strlen().

Add LEN(s) definition.
LEN gives the strlen() of string constant s, excluding the
terminating NUL.

Put, rather than print, single characters.

Put, rather than print, single characters.

Put, rather than print, single characters.

Put, rather than print, single characters.

Put, rather than print, single characters.

Put, rather than print, single characters.

Put, rather than print, single characters.

Put, rather than print, single characters.

Put, rather than print, single characters.

Put, rather than print, single characters.

Put, rather than print, single characters.

Put, rather than print, single characters.

Replace `(char *)0' et al with `NULL'.
None of them are vararg parameters.

Replace `(char *)0' et al with `NULL'.
None of them are vararg parameters.

Replace `(char *)0' et al with `NULL'.
None of them are vararg parameters.

Replace `(char *)0' et al with `NULL'.
None of them are vararg parameters.

Replace `(char *)0' et al with `NULL'.
None of them are vararg parameters.

Replace `(char *)0' et al with `NULL'.
None of them are vararg parameters.

Replace `(char *)0' et al with `NULL'.
None of them are vararg parameters.

Replace `(char *)0' et al with `NULL'.
None of them are vararg parameters.

Replace `(char *)0' et al with `NULL'.
None of them are vararg parameters.

Replace `(char *)0' et al with `NULL'.
None of them are vararg parameters.

Replace `(char *)0' et al with `NULL'.
None of them are vararg parameters.

Replace `(char *)0' et al with `NULL'.
None of them are vararg parameters.

Replace `(char *)0' et al with `NULL'.
None of them are vararg parameters.

Replace `(char *)0' et al with `NULL'.
None of them are vararg parameters.

Replace `(char *)0' et al with `NULL'.
None of them are vararg parameters.

Replace strlen(), malloc(), and sprintf() with concat().

Replace sprintf() with strcpy();  it's no worse.

Replace sprintf() with strcpy();  it's no worse.

Add comment describing concat().

Use the imperfect snprintf(3) instead of sprintf(3).

Use the imperfect snprintf(3) instead of sprintf(3).

Use the imperfect snprintf(3) instead of sprintf(3).

Use the imperfect snprintf(3) instead of sprintf(3).

Add EndsWithC(char *s, int c) and call it in some initial places.
EndsWithC returns true if non-NULL string s ends with a c before the
terminating NUL.

Add stdbool.h plus fallback to h/nmh.h.
Change h/mh.h's FALSE and TRUE to use stdbool.h's.  In time, everything
can switch over and FALSE and TRUE can go.  Delete some local
re-definitions of FALSE and TRUE.

Fix ruserpass() to not assume `\n' present before trimming.

Removed remaining relics of generated ctypoe-checked.[hc].

Alter ctype-checked to handle EOF and be hand-written.
Along the way, replace the code generation with code that punts the work
onto the stdlib's C functions, needing only a single `identity' array
for all the macros.  A mkdir needed to be added in Makefile.am because
otherwise etc didn't exist in a distcheck and attempting to redirect
stdout to a file in etc failed.  I'm not happy that's the best automake
way to achieve this, i.e. what if another rule attempts to create in etc
first?

Remove ./configure's --enable-debug support;  it did nothing.
David on nmh-workers said "that looks like a relic that can be removed".

Don't recurse in advertise() on writev(2) error, just write(2).
Some errors, e.g. EPIPE, cause the advise()'s call to another
advertise() to fail again, ad segvinitum when stack busts.  Instead,
snprintf(3) and write(2) that single string, ignoring any error.