Remove unused test file.

Added delay for output file to be closed.

Fixed references to --with-cyrus-sasl, and --without-, in comments.

Make sure we always generate a Content-ID header for message/external-body
entities, as required by RFC 2045.

Fixed permissions on test script.

Fix to commit 77fff244ad1bb1e746c78bedc87659400e2c6535.

Rearranged check for whether to skip param value checks so
that no files are left over.

Update of commit 2cf4ff9dd2fba77c7f98bbd706b00d2e79a11a3b.

Added valgrind suppressions for system libs on MacOS El Capitan.

Fixed commit 60052b95778a065760a437859ee9b8a1cc4e39a9, wrong sense.

Removed if ! bash-ism.

Fixed a couple of tests to not fail when there is no text browser.

Added missing AC_SUBST of AM_LDFLAGS.

crawl_context: Remove unused struct member `total'.

It was maintained inconsistently, and checking what problems that caused
showed it wasn't used.

Changes to folder.man

Kill a couple of empty .PPs, resurrect a lost .SS, a little bit
of slash and burn, stray whitespace, the usual.

As far as I know, which is not far, the \- construct is only
required for \-args, not, e.g., sub\-folder.

I've changed instances of 'subfolder' to sub-folder.

I've changed the 'the' in

These folders are all preceded by the read-only...

to 'any'.

Changes to install-mh.man

Some more simple wordsmithing...

Change instances of 'Install-mh' to 'install-mh', in line with
other pages, switch to single `' around 'Mail', in line with
other pages, option -> switch, clean up a little.

A cleanup, and possibly some controversy: I've replaced

Rather than standard message skeleton

with

User message skeleton

My thinking here is that a) we can do away with the kinda weird
'Rather than', b) I find the distinction between 'default' and
'standard' to be vague and c) it only appears in a few pages
and is easily remedied. It turns up in comp(1) as "An alternative
to the standard skeleton.", where 'standard' is used in the same
way as 'default' is used in rcvdist(1), so I think it's worth
settling on something 'better' and applying it consistently.

My proposal is 'Default foo', for the foo in %nmhetcdir% and
'User foo' for the foo in the user's <mh-dir>.

Fix skeletons in comp(1), forw(1) and dist(1)

While I was here, I removed two 'empty' paragraphs (.PP) in forw(1).
They may have been intended as line breaks, but I think they are
unnecessary; feel free to overrule me!

Aside: I would not be unhappy if, some day, 'skeleton' was replaced by
something utilitarian like 'template'.

Remove .w3m from test directory.

Reverted commit 316068ee3b7105ea802c63b732ee4cfb584a2600.

It was a repeat of commit 30a9c9b62fa92df289079080eb5ee57d2fda3951.
Also, wrapped a few $MH_TEST_DIR with quotes.

Clean up, and change .SS case to Title Case, from UPPER,
in line with other pages.

Changes to rcvpack.man

Make the -fcc switch to repl actually work properly, and make sure that
any Fcc header in a replied-to message is not carried over to the draft.

Changes to fmtdump.man

Add a reference to fmttest(1), delete a superfluous 'the',
delete the '...is simply...' line.

Added check of -fcc with no fcc in components.

Protect repl -file against dereference of null mp.

Removed #ifdef FT_PAUSE protection.

FT_PAUSE is unconditionally defined in h/fmt_compile.h.

Removed unused references to FT_ADDTOSEQ.

They were #ifdef LBL relics.

Changes to mh-format.man

This is a beast, and we may not tame it at the first attempt.

Mostly language simplification, some grammar and formatting, trailing
whitespace, that sort of thing. There are one or two instances where
I've taken (small) liberties with language, in favour of 'comprehension
at first glance'.

There are some commented lines hanging around in this. I've left
them in, but I think they should go at the first opportunity.

Removed obsolete comment about use of getcpy().

Changes to dist.man

Switch 'in place' for inplace, remove 'in truth of fact'.

Changes to flist.man

Replace 'option' with 'switch' in a couple of places, correct
(hopefully) a few sentences, restore '.SS "Multiple Folders" to its
rightful place.

Changes to comp.man

Remove the slightly legalistic 'in truth of fact', and bring the
mention of man whatnow into the preceding sentence.

The line 'If the draft already exists, comp will ask you as to the
disposition of the draft.' makes no sense to me, and I'm struggling
to make it make sense. If we put to one side the fact that I think
it's a broken sentence, I still haven't managed to get comp to
accept - or prompt me for - a 'replace' option when 'the draft
already exists', so I'm afraid I have to admit defeat here and seek
the wisdom of the elders.

Changes to burst.man

I'm not sure how relevant burst(1) is these days, but here goes...

Added clarifications to abbreviations.

1) N must be a positive number.
2) The + can be omitted.
3) As many of the N messages that exist.
4) Can also use - (or +) with first, prev, next, and last.

Changes to nmh.man

This adds a reference to the 'COMMANDS' section, corrects - and
hopefully simplifies - a couple of things, and tries to untangle
the line beginning 'show displays...' in the seventh paragraph.

The line 'The first, previous, next or last messages, if they exist.'
doesn't seem to accurately cover the accompanying 'foo:N' listing
in the following, or am I missing something?

first:N
prev:N
next:N
last:N    The first, previous, next or last messages, if they exist.

Changes to ali.man

The following is the beginning of an attempt to clean up the manual
pages somewhat.

grep tells me that 'switch' is used 193 times, whereas 'option' is
only used 51 times, so I've tried to standardise on 'switch'.

Fixed description of anno -list switch.

As noted in commit 98c2e7d95bda262ef23e8f0838e5b86d08ed4e4f.
If -text is used with -list, its argument is required but
ignored.

Changes to anno.man.

(Also noted that description of -list needs to be fixed, will
do in next commit.)

showbuildenv: Add comment describing exit-status convention.

It helps explain why some commands are `... || true' despite no `set -e'
being in effect.

Revert "Test if the buildbots care about tools/showbuildenv's exit status."

This reverts commit c92a4119fca5d677055dc616255a7d1efec9d750.
They do care, showing "environment failed".

Rename recently added ToUpper() to to_upper().

Rename recently added ToLower() to to_lower().

Rename recently added TrimSuffixC() to trim_suffix_c().

Rename recently added HasSuffixC() to has_suffix_c().

Rename recently added HasSuffix() to has_suffix().

Rename recently added HasPrefix() to has_prefix().

Sticking with local convention, as Ken requested.

Test if the buildbots care about tools/showbuildenv's exit status.

Document that adios() does not return.  abort() ensures it won't.

Discussion with David confirmed that adios()'s de facto contract with
the caller is that it won't return.  Document that, and add an abort(3)
to ensure it won't occur.  Better that than return and blunder on until
a SEGV or corruption later.

mhlsbr.c: Don't hide mhladios and mhldone behind macros.

Locally defined mhladios() and mhldone() were called as adios() and
done() thanks to #defines that hid the widely-used functions with those
names.  Don't bother as it confuses the reader.  Just call the local
variations directly to make clear it's not the standard implementation.

Generalized condition for support of parameter value tests.

One last fix to test-charset.

Fixed condition to enable skipping parameter value tests.

And skip the other parameter value test on the FreeBSD10 buildbot.

That didnt work, so skip Encoded parameter value test on that buildbot.

Use locale that the FreeBSD10 buildbot has.

It has EBCDIC-UK but not EBCDIC-US.

Added temporary debug code.

Try to fix Encoded parameter value test on FreeBSD10 buildbot.

Replaced !iscntrl() with isprint().

To address Ralph's FIXME comment.  One example where a byte would
have been printed with !iscntrl() is 0x24 is ISO 646 BASIC (Inv),
which is undefined.  Also, added test cases to cover most of
get_param_value().  Also, replaced a couple of other FIXME's with
code comments.

Fixed and restored commit 9a4b4a3d3b27fe4a7ff6d0b8724ce1c06b5917eb.

Using AC_CHECK_MEMBERS instead of AC_CHECK_MEMBER.  I don't know why
the former works but the latter doesn't.

Reverted commit 9a4b4a3d3b27fe4a7ff6d0b8724ce1c06b5917eb.

It broke the build on the FreeBSD 10 buildbot, not sure why.

Enabled detection of tm_gmtoff on _GNU_SOURCE platforms, e.g., Linux.

By temporarily adding $AM_CPPFLAGS to CPPFLAGS.

Added filename completion to bash_completion_nmh.

Close temp file.

Added flex 2.6.0 (with Fedora 25) output fixup.

Open infile before fixing each message.

Though the input file won't need to be opened if everything goes
well, do it early just in case there's a failure, and that failure
is running out of file descriptors.

Fix nmh man page so it shows correct name of profile.

mhstoresbr.c: Factor MIME-parameter-can-be-used-for-filename test.

It was several lines repeated each time, and it saves having to check
they're all identical.  The original test pressed on if the MIME
parameter's value was an empty string;  preserve that as it's spotted
later on.

Split a few more if-then statements into two lines to help gcov.

It's interesting to see how often some of them have the then-branch
taken.

Open outfile before parsing message(s).

So that we don't have to risk opening it after running out of fds.

Added application/ics entries, and "file was generated" comment.

application/ics is odd but does appear in the wild, and sometimes
without a corresponding text/calendar part.

Added mhshow-suffix-text entry.

I don't know why it wasn't there already, it's in the mhshow
man page.

Tweaked to work with Heirloom shell.

Removed bash-ism "function" declaration.

Replaced docs/COMPLETION-BASH with etc/bash_completions_nmh.

It's generated from man/mh-chart.man, which in turn is generated.
Automake doesn't like generated files in docs, hence the move to
etc.  And the new filename better fits usage, I think.

Did not provide support for completing message numbers.  It can
be fooled by programs that take multiple switches with +
arguments.

COMPLETION-BASH was broken with current bash, anyway.

Also, removed sbr/sigmsg.h from CLEANFILES in Makefile.am.  It
was a leftover from commit 5776f9b783afe8e6bdbe430adb5d5b9435d25b19.

Added copyright noticed.

Removed unused declaration of output_message().

Replace many add(s, NULL) with mh_xstrdup(s).

Where s is known to be non-NULL.

Replace some add(FIELD, NULL) with mh_xstrdup(FIELD).

The various FIELD are macros for string constants so can not be NULL.

m_getfld: Shuffle `delim' assignments slightly.

Document end state of the four pointers to parts of the delimiter.
fdelimlen can be, wrongly, one less without causing problems because a
second check uses edelimlen.  The only side effect is an extra iteration
through m_getfld().

Revert "m_getfld: Shorten fdelimlen by one;  it was too long."

This reverts commit 36a12f6a1a5a57799e48b23fd5ae47fe2351be68.

I think it was right after all, but currently trying to get a test to
fail with the faulty change.

Replace copy(s, d) with POSIX-1.2008's stpcpy(d, s).

m_getfld: Replace matchc() with memmem(3).

libc is more likely to find one lump of memory inside another quicker
than a home-grown quadratic emulation of the VAX's MATCHC instruction.
memmem() isn't POSIX, but exists on Linux and FreeBSD.  This will see if
the buildbot is happy.

m_getfld: Remove never-true test in matchc().

Now the fdelimlen doesn't include the NUL it is never true.  I didn't
understand how it could ever be true but gcov on the test suite was
showing 22e6 tests had three positives;  that's what put me on to
fdelimlen's overshoot.

m_getfld: Shorten fdelimlen by one;  it was too long.

The fdelimlen bytes starting at fdelim included a NUL at the end, e.g. 7
for "\nFrom \0".  Other code seems not to expect this and has
conditional bits to work around it, perhaps without understanding why.
All the tests still pass, including with valgrind.

configure.ac: Add FIXME regarding _BSD_SOURCE for tm.tm_gmtoff test.

dlocaltime(): Only call tzset(3) once.

This avoids the stat(2) of /etc/localtime per message scan if TZ isn't
set in the environment;  that's the default case on this Linux system.
I can't see a way the TZ environment variable is likely to change
between messages, or that it's intended an invocation of scan(1), say,
copes with the timezone shifting underneath it by /etc/localtime being
changed.

uip/scansbr.c: Use fputs(3) instead of mh_fputs().

I can't see any reason a local one exists.  It loops, calling putc(3).
The stdlib has the opportunity to be more efficient.

Add test-forw-coverage to improve uip/forw.c's coverage.

test-mhl-flags: Improve coverage of uip/mhlsbr.c.

test-fmtdump: Increase uip/fmtdump.c coverage.

test-flist: Increase coverage on argv[] processing.

Add test/format/test-ap to get 100% uip/ap.c coverage.

uip/dp.c: Increase coverage to 100%.

Replaced add(s, NULL) with mh_xstrdup(s).

Removed unused global from MIME parser.

Initialize mhparse global for each message.

Globals are evil.  It's tricky to test, so no test was added.
Just cleaned up test-mhfixmsg.

Forgot to commit test-mhbuild.  And thank Tom Lane for reporting bug.

Update to commit 6c359ae82651d117af71c55ce1e56f3c5db18bfe.

Don't parse parse # lines with mhbuild -nodirectives.

Plugged remaining mhfixmsg(1) memory leaks.

Made static char strings const.

Plug memory leak with parameter continuations.