Move smtp_init's last-ditch attempt for client inside first test.

Didn't realise there was a second lot of similar code before the last
commit.

Move smtp_init's last-ditch attempt for client inside first test.

No point checking it twice in a row if it was false the first time.

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

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Don't test boundary in output_content() for NULL;  it never is.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Add FIXME raising if iscntrl(3) test should be based on isprint(3)

!iscntrl blocks characters that could be isspace or isblank whereas
isprint would let them through unharmed.

Assume POSIX ctype.h;  don't vet toupper()'s parameter.

Explain isascii() test more in netsec_set_sasl_params().

It stops non-ASCII lower in the SASL mechanism being uppered, just in
case there is some.

Assume POSIX ctype.h;  don't vet toupper()'s parameter.

Assume POSIX ctype.h;  don't vet tolower()'s parameter.

Assume POSIX ctype.h;  don't vet tolower()'s parameter.

Oops, stop showbuildenv reading from stdin when run from a TTY.

David spotted the problem I introduced with 8374c3c.  Alter offset to
process any arguments, falling back to stdin if none.

Assume POSIX ctype.h;  don't vet tolower()'s parameter.

Assume POSIX ctype.h;  don't vet tolower()'s parameter.

Assume POSIX ctype.h;  don't vet tolower()'s parameter.

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().