Fixed a couple of calls to m_getfld():  they were in while loops
and the loop bodies were missing brackets.

Deprecated msh(1).

In build_nmh's output from the test suite:  retain
colorization and list any test failures.

Enabled colorized test output (PASS/FAIL/summary) by default
via "color-tests" automake option.  To disable at runtime,
either set your AM_COLOR_TESTS environment variable to "no",
or provide that on the make check command line.

Reworked the new documentation in m_getfld.c.

Handle the case in the test suite where wcwidth() can return different
values for combining Unicode codepoints on different operating systems.

Fix the format engine so it properly keeps track of multibyte characters
and column positions when processing components and other functions.

When working on this code I realized we have nothing that checks to make
sure right justification of format function output works properly, so a
test for right justification is also included.

Added functions for inc, msh, rcvtty, and scan to access
scansbr's gstate instance so it can be hidden, except for
msh's uses of m_getfld().

Fix a bug leading to message corruption with inc when incorporation more
than one message at a time.

When I reworked the scan() code to remove the use of stdio internals,
two code paths were created.  The inc code path consumed another buffer
from the nxtbuf list for processing the message body.  At the end of
scan() there was code to push back all of the consumed buffers to the
nxtbuf list.  The extra buffer consumed in the inc code path wasn't
being pushed back, and as a result the nxtbuf list would end up with
duplicate pointers in it.  This would result in two components sharing
the same text buffer, which caused the message corruption.

When I looked at this closer, I realized that there was no reason to
push those buffers back to the list; nxtbuf gets reset to compbuffers
at the start of every call to scan(), and nxtbuf isn't used after the
buffers are pushed back.  Some researched revealed that this was the
same back in the original MH code.  I decided it was simplest to simply
delete the buffer recycling code.
(cherry picked from commit 22d2acba60374851c291260ff6aa716e9bf44a6d)

Brought over changes to test-pop from master to m_getfld branch.

Added scan and inc from mbox with multiple messages.
(cherry picked from commit c4c53777458bfcfd288d45b7708e52eb47c56405)

Added test-msg-buffer-boundaries.
(cherry picked from commit dbd0fa0a56a222bd118fea627eba53e5d9811d01)

Brought over changes to test-pop from m_getfld branch.

Changed m_getfld() to automatically allocate a new state
instance as needed.  Callers no longer use
m_getfld_state_init().

Removed unnecessary m_getfld_state_destroy() from scan.c.

Reworked enter_getfld() and Peek().

Merge branch 'master' into format-test

Fix a bug leading to message corruption with inc when incorporation more
than one message at a time.

When I reworked the scan() code to remove the use of stdio internals,
two code paths were created.  The inc code path consumed another buffer
from the nxtbuf list for processing the message body.  At the end of
scan() there was code to push back all of the consumed buffers to the
nxtbuf list.  The extra buffer consumed in the inc code path wasn't
being pushed back, and as a result the nxtbuf list would end up with
duplicate pointers in it.  This would result in two components sharing
the same text buffer, which caused the message corruption.

When I looked at this closer, I realized that there was no reason to
push those buffers back to the list; nxtbuf gets reset to compbuffers
at the start of every call to scan(), and nxtbuf isn't used after the
buffers are pushed back.  Some researched revealed that this was the
same back in the original MH code.  I decided it was simplest to simply
delete the buffer recycling code.

Plugged small memory leak in m_unknown().

Replaced hard-coded value with MAX_DELIMITER_SIZE in m_getfld.c.

Removed the static m_getfld() state instance and replaced
with functions for callers to create and destroy their own
instances.  This also allows inc, msh, and pick to call
m_getfld() with an initial file stream position other than 0.

There is now a global state instance so that scan can call
m_unknown() on maildrop files.  It'd be nice to get rid of
that.

Fix memory leak in arglist code

Added scan and inc from mbox with multiple messages.

Updated test-inc-scanout on master to match version of
m_getfld branch.

Added a FILE * to the m_getfld_state struct so that most
functions don't need to pass it explicitly.  Note that
m_getfld() must still be passed the FILE * because some
callers, e.g., those in mhparse.c, reopen their input file
and therefore pass different FILE *'s for the same input.

Moved all file static data in m_getfld.c into the
m_getfld_state struct.  Changed MSG_INPUT_SIZE from
hard-coded number to BUFSIZ.

Added check of -pack to test-pop.

Fixed test-header to work on Cygwin, which has a BUFSIZ of
only 1024.  That caused an error message to be truncated.

Beginning of implementation of new argsplit() function to handle arguments
to profile-defined commands.

Removed all traces of m_getfld() FLDEOF and BODYEOF states.
They were never set by m_getfld(), so all caller code that
supported them was completely unexercised.

Minimized message files reads in m_getfld() when the caller
interleaves calls to fseek().  Added check of handling of
long header field name without a colon to test-header.

Added test-msg-buffer-boundaries.

More cleanup of m_getfld(), changed some variable names.

Added check of header field body with more than 511
characters to test-inc-scanout.  Also changed a couple of
test message files to be generated on the fly.

Cleaned up m_getfld() code a bit.  The most notable change
was to remove the check for mmdlm2 not being null.  It's
global data so that shouldn't happen.

Added check of a header with name that's 998 bytes long to
test-header.

Removed the for-loop that was used to get a header field
name in m_getfld ().  It was no longer necessary when not
drawing directly from the stdio buffer.

Check that a character array isn't empty before looking for its
next-to-last element.

In build_nmh, removed configuration of pager and editor.
And replaced use of which(1) with a self-contained function.

Support the VISUAL and EDITOR environment variables, and remove support
for configuring a default editor with --with-editor.

Remove the --with-pager configure option and replace it with a default
of "more".

Renamed a couple of m_getfld auxiliary functions.

Cleaned up m_getfld's read_more().

In m_getfld(), fixed the loop that handles the FLDPLUS case.
In Getc(), removed the increment of the byte count at EOF,
it didn't seem correct or necessary.

m_getfld() now supports interleaving calls with
fseek()/ftell().  Reverted the changes to uip/mhparse.c of
commit 484eb1003ae647e4b751e4d7829c7daf44fb99c9 because
they're no longer necessary.

Moved test of empty header field from
test/inc/test-inc-scanout to test/bad-input/test-header.

Cleaned up a bit.

Added support for m_getfld() callers interleaving fseek() calls
with their m_getfld() calls.

Added bytes_read to m_getfld() buffer state.  This is the
next step in supporting ftell()/fseek().

Removed unused local char_read from forwsbr.c.

Added check of m_getfld() handling of fields with trailing
whitespace to test-inc-scanout.'

Removed "sleazy hack" of negative bufsz because commit
05d8af21d26673279a1ca079ad9c6f301426fe0e removed the only
use of it.

Cleaned up FLDPLUS handling in m_getfld().  As part of this,
removed the backing up of the read pointer in the "something
went wrong" portion of the FLD handling.  It had been moved
back two positions.  The first is no longer necessary
because a Getc() was replace by a Peek(), and the second
caused the extra blank line noted in the old comments.
Updated the expected output in test/bad-input/test-header to
correspond.

Fixed addition to test-inc-scanout in
683e0fb1a14da5b9ce7bc5db6e76951d14d79348 to not depend on
current date.

Very minor cleanup to m_getfld() to remove an increment of j.

We never use the autoconf checks for errno.h, crypt.h, or netdb.h, so
remove them.

At this point we're assuming at least C89 support, so there's no reason
we should check for support for const and standard header files.

Started revising m_getfld() code to replace direct buffer
access with Getc().  The goal is to again support fseek()
after return from m_getfld().

Convert the MIME content cache switches over to the smatch() New World Order.

Added explicit third initializer to some swit struct values
to silence gcc -Wmissing-field-initialziers warning.

Convert all argument processing over to using X-Macros for cleanup.
Based on a suggestion by Ralph Corderoy.

Added comments to and started cleaning up m_getfld.c.

When read_more() loads more bytes, save some from the end of the
current chunk and move them to the front of the buffer.  This
allows backing up of the current position, which is necessary when
looking for the message delimiter in a maildrop file.

Fix bug where a "cur" sequence that was outside the folder allocation limits
could trigger a core dump on some systems.

Removed support for #ifdef LBL.  It was broken anyways,
sometime between MH 6.8.5 and the Apr 1999 entry into CVS.
Also removed the addtoseq format function that could only be
accessed with it.

Fixed test-utf8-body to actually contain the UTF-8 character
instead of the single byte, 8-bit character.  This was caught
using Heirloom shell with a UTF-8 locale:  when it wrote the
test draft from a here document, it converted the single byte
character to UTF-8.  This verifies that behavior:

LANG=en_US.utf-8 /usr/lib/heirloom/5bin/sh -c "od -x <<EOF
`printf '\xA1'`
EOF
"

Expanded warning message for invalid Content-Transfer-Encoding.

Added reference to RFC-1864 to mhbuild man page.

Remove support for -ebcdicsafe/-noebcdicsafe switches in various utilities.

Create a dependency for mh-chart.man on all of the man page sources so
it gets rebuilt correctly if the man pages change.

Block SIGPIPE before killing the child in sm_end().  This replaces
the discard() calls that were removed in commit 4548a3ecc4248a1d4.

Removed discard() call in post(8).  It was only used with -debug
and seemed to serve no useful purpose any more.  Its comment
said that it was needed to help the loader.

In cpstripped() and cptrimmed(), if a multibyte character is
found, only count it as taking up one character in the
destination buffer (if it has enought room).  This way,
scan(1)'s output won't be jagged if there are any subjects,
for example, that have multibyte characters.

Added a test for this to test-scan-multibyte, and moved the
test for an invalid multibyte sequence from test-scan to it.

Removed empty file name .null from the historical mh-6.8.5 tree.

Fixed decoding of header fields when they contain a character that
can't be converted.  Added test case to test-scan.  valgrind also
noticed that things went amiss.

Remove test .netrc file at the end of test-pop.

Switch to using native automake rules for cscope.

Change TLS support to only link the TLS libraries with the programs that
need them.

Clean up (and hopefully improve) the autoconf iconv tests.

Still MORE autoconf cleanup.

Autoconf/automake cleanup

Document removal of --enable-nmh-debug

Finally deprecate --enable-nmh-debug

Note changes to autoconf requirements here.

We are now requiring autoconf 2.68 and automake 1.13.

Note that the minimum required version of automake is 1.12.

Ignore top-level ar-lib file.

This is detritus from the AM_PROG_AR macro I added to configure.ac.

Add an AM_PROG_AR program check to configure.ac  This quells a
warning when configuring on FreeBSD 10.

Corrected fseek() emulation to fix test/bad-input/test-header.

Replace use of ftell() in mhparse.c:get_content() by keeping track
of bytes read by m_getfld().

Tweaked last commit to test/common.sh.in to eliminate a mv.

Added note about PATH when running "make check" on Cygwin.

Fixed test-slocal to work with mmdfldir set to /dev/null, and on
Cygwin, which needs the full path to tee(1).

Restored an explicit refill of the buffer in m_getfld() that I
removed in the big stdio cleanup.  Its need is shown by reducing
the message buffer size to 2048.

Removed call to fpurge() and its platform-specific emulations
from discard.c.  discard() is called on stdout from mhl(1) and
msh(1) in interrupt handlers, and post(8) but only with -debug.
tcflush() should do the purge when stdout is a terminal.  If
it's a file or pipe, well, maybe some garbage could get flushed
out.  And it's called on the streams that post uses to talk to
its smtp server, so removed those calls because they no longer
do anything.  And removed LINUX_STDIO define from configure.ac.

Replaced all the stdio buffer access in m_getfld.c with a single call
to fread() and then some almost straightforward buffer manipulations.

Fixed formatting of last change to scansbr.c.

Updated m_getfld() interface description.

Changed bufsz argument of m_getfld() to be in-out instead of in.
This allowed removal of the msg_count global, in turn allowing
removal of sbr/m_msgdef.c

Changed msg_style and msg_delim to be file static to m_getfld.c
instead of global.