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.

Add check-programs target to Makefile.am to add to `make all'.

Ignore etags ugly TAGS output file.
Makefile prefers etags over ctags.

Replace strchr() condition with TrimSuffixC(), fixing assumptions.
A couple of the existing strchr()-based trims assumed fgets() returned a
string containing `\n' if it didn't return NULL.

Replace strchr() condition with TrimSuffixC().

Replace strchr() condition with TrimSuffixC().

Replace strchr() condition with TrimSuffixC().

Replace strchr() condition with TrimSuffixC().

Replace strchr() condition with TrimSuffixC().

Replace strchr() condition with TrimSuffixC().

Add TrimSuffixC(char *s, int c).
TrimSuffixC deletes c from the end of non-NULL string s if it's present,
shortening s by 1.  Only one instance of c is removed.

With EAI (mhbuild -headerencoding utf-8), force message header to
show 8-bit UTF-8, even if the message body is ASCII.

Output an error message from post(8) when user specifies
mhbuild -headerencoding utf-8 but the SMTP server doesn't
support SMTPUTF8.

Rewrite r1bindex() using strrchr(3).

Use concat() in m_putenv.  Add FIXME about memory leaks.
POSIX defines setenv(3) and unsetenv(3).  They seem like a good route,
just need to preserve the existing functions' behaviour.

Shrink the static char[] for "%d" from 8KiB to just right.

Replace getcpy() with mh_xstrdup() where the string isn't NULL.

Replace getcpy() with mh_xstrdup() where the string isn't NULL.

Replace getcpy() with mh_xstrdup() where the string isn't NULL.

Replace getcpy() with mh_xstrdup() where the string isn't NULL.

Replace getcpy() with mh_xstrdup() where the string isn't NULL.

Replace getcpy() with mh_xstrdup() where the string isn't NULL.

Replace getcpy() with mh_xstrdup() where the string isn't NULL.

Replace getcpy() with mh_xstrdup() where the string isn't NULL.

Replace getcpy() with mh_xstrdup() where the string isn't NULL.

Replace getcpy() with mh_xstrdup() where the string isn't NULL.

Replace getcpy() with mh_xstrdup() where the string isn't NULL.