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.

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.

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.

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.

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.

Fix unlikely bug in oauth's make_query_url() of base_url overflow.

Replace a few calculations of string lengths with concat().
Saves the reader have to keep checking all the +1 are correct.
Passed the oauth tests;  they weren't skipped.

Close FILE pointer in anno for -append as well as -delete.
https://savannah.nongnu.org/bugs/?23465 has Peter Maydell pointing out
the assymetric open and close of a FILE pointer.  He's right on that
AFAICS so change the test to be for a non-NULL pointer rather than the
more wordy and prone to error flag-testing.

Oops, should have offset file in showbuildenv rather than cat.

Look for a platform-specific /etc/*-release or similar.
/etc/os-release is becoming standard due to systemd.

Protect showbuildenv's headers from glob expansion.

A-ha!  automake has support for tests expected to fail;  use that.
Backing out my recent addition of an `expectedfail` parameter to the
`check' function.

Remove use of C's register keyword.

Added reference to mh-profile(5) man page for more information on
postproc.  And note that "Nmh-" header lines are filtered out.

Add FIXME for more documentation to showbuildenv.

Have showbuildenv only indent non-blank lines.

Run lsb_release(1) if uname is Linux.

Oops, fc31cece had a syntax error that gcc ignored, clang doesn't.
Two logical-or operators in a row don't upset gcc.  Who said parsing was
a solved problem?

Replace NEW() then memset() of 0 with NEW0().

Don't call strlen() twice;  save result.
The second instance was attempting to write strlen(mmdlm2) bytes
and checking the return value against strlen(mmdlm1).  I assumed
that was an error without digging.

Don't call strlen() twice;  save result.

Deference char pointer to test for empty string instead of strlen(3).

Replace getcpy() with mh_xstrdup().

Replace mh_xmalloc() with mh_xstrdup().

Replace mh_xmalloc() with mh_xstrdup().

Delete client_getcpy();  don't know why it exists.
Replace with calls to getcpy() or mh_xstrdup().

Use mh_xstrdup() instead of getcpy() for a string constant.
It cannot be NULL.

Rewrite getcpy() using mh_x*() allocation functions.

Add const specifier to mh_xstrdup()'s src parameter.

Rename fmt_compile.c's NEW macro to NEW_FP to avoid redefine.
It grabs and fills in a pointer called `fp' so a more specific name
seems OK.

Rename local enum to avoid clashing with NEW() macro.
Even if it doesn't in practice due to the lack of parenthesis, it makes
things more clean if all the enumerates have a common abbreviation
prefix.

Add mh_xstrdup() that exits on allocation failure.
Call it from a few simple places.