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.

Update pending-release-notes with a couple of my recent fixes.

Fix bug where single-character headers were sometimes missed.
The hash function used to check if a header was of interest always used
its first three characters.  For a single-character header that third
byte wasn't particularly set and so it was pot luck if the two hashes
matched, swayed by what was previously in the format string and in the
email's headers.  Alter it to cope, including with an empty string.
Comment that it never gives 127 so leaves the last element of the
hash-table array unused.  Alter test-header-parsing to stop declaring it
expects to fail.

Change all Getc() callers to use EOF, not comparisons with 0.
Makes the logic that bit easier when you don't have to convert to EOF
mentally.  No functional change intended.

Don't increment bytes_read if returning EOF.
Don't know if this was actually causing a problem, or if it was wrong,
but bytes_read was always being incremented even if the later test
showed readpos was to be left unincremented and EOF returned instead.
All tests still pass so any breakage must be subtle!

Fix spelling in comment.  Alter case to match identifier.
vim will search matching case on `*', for example, so if searching for
`\<fdelim\>' the comment shouldn't use `Fdelim'.

Add failing test case for single-character header parsing.
`scan -format '%{x}'' fails to find single-character header `x' in some
cases, depending on the position of the header in the file.  Add a
simple test case, scan/test-header-parsing, for this that fails.  Alter
the test harness to allow a test to indicate it expects to fail.  When
it does, that's ignored.  If it passes, that's then an error instead.

Did this because I don't know if or when it will be fixed and didn't
want it to be forgotten in the meantime.  When it is fixed, the test can
be enhanced to cover many more possibilities.  First noticed it with ad
hoc testing of my fewer-lseek change, but it is present in the released
1.6.

Fix spelling in warning message;  s/conext/context/.

Simply enter_getfld()'s structure, no functional change.
Return ASAP to avoid having to mentally track those states as the code
indents further.

Explicitly state a postproc's last argument will be the draft.
Update the sample postproc script to make clear it's the -whom detection
that sucks.

Document fmttest(1)'s existing -help and -version options.

Move -help and -version to the front of the man page options.
The man pages had -verbose and -help listed at the end of the synopsis.
This put them after mandatory arguments in some cases, e.g. post(8).
Put then at the front, swapping the order so -help is first, as novices
will want to know -help exists early on, and others can simply skip over
them rather than encounter them at the end when they're trying to track
the combinations of what they've read so far.

Fix end of sentence spacing in man pages, and related problems.
troff(1) input that ends a sentence at the end of the line gets the
proper inter-sentence spacing when formatted.  If the sentence ends
midline then two spaces must follow to get the same result, otherwise
the spacing looks odd.  Similarly, if a full stop is not the end of a
sentence then it should have the zero-width escape appended so if
re-formatting the source happens to place it at the end of the line then
no large gap follows it: e.g. `e.g.\&'.
Replace `et. al.' with `et al'.

Reference the buildbot in docs/README.developers.

Stop one lseek(2) per header fetched by m_getfld.c in common case.
Only call ftello(3) in enter_getfld() when track_filepos has been
requested;  not often.

Fix error message when attempting to xmh_realloc() 0 bytes.
Though really, I'd expect it to perform a free(3) then, as the real
realloc(3) does.

Plugged memory leak in convert_content_charset().

Fixed mhshow warn message when it can't convert character set,
"from" instead of "to", as reported by Ralph.

Changed "existant" to "existent".  Found by misspell_fixer.

Added to convertargs documentation.

Try running script(1) for SHELL test in subshell.

Check that script(1) supports SHELL in test-version-check.

Added check to test-version-check to ensure that script(1) makes
the program look like it's connected to a terminal.

Didn't work on buildbot hosts.  Added debug statements.

Instead of feeding stdin to script(1) in test-version-check,
create a simple shell script with the command to be run and feed
it to script via the SHELL environment variable or -S option.
Thanks to Ralph for suggesting this approach.  We'll see if it
works on all of the buildbot hosts.

Tweaked formatting of welcome message.

Disable test-version-check, again, when not connected to terminal.
I can't otherwise explain why it fails on the ubuntu buildbot.

Added debug printouts to test-version-check.

Use "script file" in test-version-check for maximum portability.
Feed its stdin, which might be problematic.

Added check that replied-to text is not quoted-printable encoded.
It's up to the external converter to generate RFC 5322-legal text.

Split ${charset:+-I "$charset"} and another similar case into two
separate options, because Solaris didn't like the space between them.

For the autoconf/automake stuff to replace getline, the following
patch seems to work.

The final change on configure.ac is to cope with cc -V output having
slightly changed in the most recent version of the compiler.

I've written a getline(3) from scratch based on
https://manned.org/getdelim.3p and am happy for it to have whatever
licence nmh needs.

Removed test-local-mailbox that was mistakenly committed.

Only set CFLAGS with gcc.

Moved gcc warning options from AM_CFLAGS back to CFLAGS, to
make it easier for packagers to override.  This is a partial
rollback of commit f7eb978bd395752d3d9037ea4436b8e2db41f135.

Added LANG to procmail example in mhfixmsg man page.

Corrected date in forw man page.

Tweaked OAuth documentation in mhlogin and send man pages.

Removed reference to nmh User's Manual because there is no such
thing.  There was an MH User's Manual, but that is not distributed
with nmh.

Changed test suite's require_locale() to set the locale that it
finds, and added upper case versions of locale names to each test.
This should allow the tests to succeed on old Solaris.

Added -sasl to -saslmech xoauth examples in man pages.

Replace `id -u` with $$ so that tests have a chance of working
on old Solaris hosts.

Added comments about sidestepping check for checks to determine
whether replied-to text needs to be encoded in mhbuildsbr.c.

Added example of decoding RFC-2047 filename to mhstore man page.

A better fix than e87f37c27828723317a71291e31b34f39ec09098, because
c_reqencoding should be set for text content in replies if we don't
want it to be encoded.  Scan the text content to see if it is 7- or
8-bit, and set c_reqencoding accordingly.

Moved scan_input() from uip/post.c to sbr/utils.c.

Updated test-version-check test_skip message.

Fixed test negations in test-version-check to work with make distcheck.

OK, don't run test-version-check if not connected to terminal.

More debugging test-version-check.

Replaced strtod() with strtof().

More debugging.

More test-version-check debugging, on Linux buildbot hosts.

1) Use script(1) -t 0 instead of -f on FreeBSD.
2) Added another debug printout.

Accommodate FreeBSD script(1), and add debugging statement to diagnose
failure on ubuntu buildbot host.

Merge branch 'welcome'

Enable SMTP 8BITMIME for messages with 8-bit content:

1) In post, look for a Content-Transfer-Encoding header.  It has to
   be the header for the message, not any MIME parts.  If found,
   post trusts that it's correct.  If there isn't one, post scans
   the entire message body for any 8-bit bytes.

2) If the message body is 8-bit:
   If the server supports 8BITMIME, enable it.

   If the server doesn't support 8BITMIME, fail with a message to
   user that they need to encode the message for 7-bit transport.

Call c_ceclosefnx in mhfixmsg set_ce().  It doesn't seem to help
reduce file descriptor leaks, but can't hurt.

Don't set c_reqencoding in repl, let mhbuild set it based on
the content.

Improve these comments a bit.

Enabled check for 8-bit content in all text parts, not just those
with no specified character set, so that a Content-Transfer-Encoding
header will be added if needed.

Have mhfixmsg add a C-T-E at the message level, based on the least
restrictive C-T-E of its parts.

Base Content-Transfer-Encoding on decoded and new text parts on
content rather than the user's charset.

Fixed typo.

Fix C-T-E of message, as well as multipart, types to be 7bit, 8bit,
or binary.

Added test of SMTP server that doesn't support SMTPUTF8.

Fix some typos in the comments.

Move the read buffer shuffling to BEFORE where we call the network
read routines; otherwise we can end up calling the lower layers with
a length of zero, which messes everything up.

Also make sure we reshuffle our pointers around if that happens.

mhfixmsg now replaces RFC 2047 encoding with RFC 2231 encoding of
name and filename parameters in Content-Type and Content-Disposition
headers, respectively.

Cleaned up a bit, mostly in comments for each function.

Sigh, the return value from SSL_get_error() isn't suitable to give
to ERR_get_error(), so use a different function to get the error.

Print even more errors.

Add -initialtls/-notls flags to msgchk.

Make sure we retry if possible upon a TLS read, and give a more meaningful
error if we can't.

Added --sysconfdir to nmh.cygport.

Replaced some hard-coded strings with variables in nmh.cygport.

Fix typos and drop declaractions of 2 undefined functions.

Updated comments in h/prototypes.h.

Welcome message enhancements:
1) Disable if Welcome: disable component is in profile.
2) If MHCONTEXT is set, only print the message if the context file
   already has a Version: reference where the version is old.

Instead of relying on invo_name to decide whether to call
nmh_version_changed(), use a read_context setting of 1.

Make sure stderr is directed to terminal, as well as stdin and
stdout, when deciding whether to check for nmh version update.

Wait for user to ack new version message.

Added welcome message when nmh detects that its version changed.

back out use of RFC 2047 decoding on the filename

The RFCs clearly state that an encoded word should not be used in
the MIME parameters and the scheme from RFC2231 should be used
instead. nmh supports that. Unfortunately, other mail clients,
in particular gmail and IBM/Lotus Notes, produce messages with
the invalid encoding.

Added start_test/finish_test.

Updated start_test() and finish_test() to not update the test name
if there was a test failure.  run_test disables exit on non-zero
status, but does increment failed.