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.

Add NEW(p) that sets p to mh_xmalloc'd memory sized by *p.
Use it for the simple cases.  Again, saves having to check the same
identifier is given, possibly far apart after a superfluous cast.

Add macro NEW0(p) that callocs, and use it in simple calls.
p is a pointer, the size of the contents of the pointer is allocated
with calloc via mh_xcalloc.  It saves having to check every calloc to
ensure the space requested matches the pointer to which it is assigned.

Put parameter names in h/utils.h memory function prototypes.
It can be a problem if there is a system creeps in a clashing macro,
e.g. `size', but that's unlikely and I think the documentation benefits.

Add mh_xfree(), guarding free(3) from NULLs.
Perhaps all nmh's platforms cope with free(3) these days, but it gives
symmetry with the other mh_x* memory functions and if there is code
testing the pointer before calling free then it can use this instead.

Tweak mh_xcalloc();  print size on error, follow POSIX.
Using `%zu' for the size_t value.  That might be a problem on older
platforms, but we'll see.  If asked for zero bytes then allocate one to
get a unique pointer.

Tweak mh_xrealloc();  print size on error, follow POSIX.
Using `%zu' for the size_t value.  That might be a problem on older
platforms, but we'll see.  If asked for zero bytes then free an existing
pointer, only passing it to free(3) if it's non-NULL, and then allocate
a byte so a unique pointer is returned.

Tweak mh_xmalloc();  print size in error, allocate zero bytes.
Using `%zu' for the size_t value.  That might be a problem on older
platforms, but we'll see.  If asked for zero bytes then allocate one
rather than exit;  it's sometimes useful to allocate zero, but some
older platforms might not like it.

Don't check mh_xcalloc() and friends for a NULL return.
They never return NULL.  That's their prime purpose.

Expand the test of scan's -forma;  fails on first attempt.
For an email that's just `a:\nb:\nc:\n', a scan with `%{a}\n%{b}\n%{c}'
is one blank line short.  If the `%{c}' has anything added then its line
prints.  Break out of the loop so all the other tests are skipped.

Don't test -reverse's boolean on every message number.
Test just once before the loop and define start, end, and increment.
Keep going whilst start is not end.  Happy for underflow or overflow in
calculating end because msgnum will make the same transition.  It can't
affect whether the loop runs at all since 0 is not a valid message
number.