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.

Free c_ctparams for all content types, not just text.

uip/ap, uip/dp: Remove unneeded char **arguments.

dp: Fix write past end of dates[] array.

Not spotted by valgrind because that doesn't check heap or stack arrays.

ap: Fix write past end of addrs[] array.

Not spotted by valgrind because that doesn't check heap or stack arrays.

Just show relevant portion of invalid encoded string.

Replaced use of freects_done() with free_content() for each message.

Thereby not holding on to open file descriptors until termination.

Pass through message even if from relative folder.

Use result, because (void) didn't prevent warning on FreeBSD 10.

test-rfc6532: Export LC_CTYPE after setting it.

Updates commit 2c4ac164edefd18861595ec7624e5ad344b10198.

Changed type of flags from signed to unsigned long.

Replace non-ASCII characters in headers with ?'s with non-UTF8 locale.

Fixed display of header field values with multibyte characters.

Removed NMH_UNUSED(hostname) because it no longer exists.

Add some Autoconf support for older OpenSSL libraries that don't support
hostname verification.

State netsec_set_tls()'s noverify is unused if no TLS.

Stops compiler warning.

Update docs a bit.

Add support for certificate verification when using TLS.

Print out more TLS information if -snoop is turned on.

test/folder: Add test-coverage; lots of check_exit.

Split one-line condition+body to get gcov results.

uip/dist.c: Increase coverage.

uip/send: -saslmech lacking argument indexed outside array.

Similar to problem in commit 171ad462ce7582196f91f93f91f99fc60a21189e.

man: Fix spelling mistakes.

Fix spelling mistake in comment.

Only done because the same typo appears in the man pages.

Fix spelling mistakes in UI text.

test: Remove the " # gcov" comment from check_exit calls.

Now the command is passed to check_exit, that is sufficient to indicate
it's for coverage uses only, and a candidate to be fleshed out into a
test that checks the output.

uip/comp.c: Increase coverage.

uip/burst.c: Increase coverage.

uip/anno.c: Increase coverage.

anno: Stop segmentation violation on `-number 0'.

If atoi(3) returned zero, either because `0' was given or the number
could not be parsed, then the error message dereferenced an invalid
pointer.  Found when trying to increase coverage with check_exit.

uip/ali.c: Increase coverage.

ali: Fix a memory leak spotted by valgrind.

mhbuild's fgetstr(): simplify source, logic untouched.

The tests aren't getting 100% coverage and the nested assignments make
it harder to see what's happening, and what needs to happen to gain
coverage.  Hoist invariants so it's clear they're such to the reader and
not just the compiler.

Remove test files that aren't removed by check().

Replaced BUFSIZ with NMH_BUFSIZ in a few places.

This allows the test suite to succeed with BUFSIZ of 256.  We should
consider using different constants for different purposes.

Don't set CT type if reformat failed.

Uncapitalise a few more mentions of commands in man pages.

mh-mkstemp.man: Detail exit status as 255 rather than -1.

Since that's what the outside kernel and outside world sees.

Use new check_exit in `# gcov' tests.

It showed up fmtdump wasn't being found because $? was 127, not 1.
And mkstemp exits 255 on error, unusually amongst MH commands.

test/common.sh.in: Add test_exit() for $? checking.

The stdout and stderr of the command being tested are saved but ignored,
only regurgitated if the test fails.  The test is stated as a fragment
of a test(1) condition, e.g. '-eq 1'.  This is mainly intended for the
quick one-liners to increase gcov coverage and thus valgrind's scope.
Over time, better tests that check more of the command's behaviour can
replace these, keeping the coverage level.

Replaced use of ! with || true, for ultimate Bourne shell compatiblity.

Update to commit c3ba36f92bcbf44e071961f3d0bb5e6ddab317d6.

test-nocreate: Don't use run_test and run_prog together.

valgrind tries to run run_prog and can't find the command.

fgets() reserves space for the NUL itself.

No need to knock one off the buffer size when calling it as that
suggests something special is going on.

Gain coverage with `mhparam localmbox'.

Now 100% for uip/mhparam.c

Remove mhparam's p_find()'s ability to take NULL argument.

It can't be passed NULL and couldn't get coverage.

Get coverage of quite a few of the `ambiguous switch' case.

Typically by adding a `foo -', discarding stdout and stderr, and
expecting it to exit non-zero.  A `# gcov' comment states its purpose,
and makes it easier to see if they can be deleted in time as other
coverage tests improve.

Split gcov target into multiple sub-targets.

Allows them to be run individually.

Clarify trunccpy()'s src and dest must not overlap.

Replace printf("%s", foo) with fputs(foo, stdout).

Replace strlen(foo) > 0 with *foo.

Add outputs from Makefile.am's gcov target.

*.gcda, *.gcno, and *.gcov.

Remove sbr/peekc.c containing unused peekc().

gcov showed it wasn't getting any;  we've all been there.
It didn't check for EOF, nor that the ungetc(3) succeeded.

Replace fprintf(fp, "%s", foo) with fputs(foo, fp).

Replace printf("%s", foo) with fputs(foo, stdout).

Replace printf("%s\n", foo) with puts(foo).

Add trunccpy(), and the convenience TRUNCCPY().

truncpy copies at most size - 1 chars from non-NULL src to non-NULL
dst, and ensures dst is NUL terminated.  If size is zero then it
aborts as dst cannot be NUL terminated.

It's to be used when truncation is intended and correct, e.g.
reporting a possibly very long external string back to the user.  One
of its advantages over strncpy(3) is it doesn't pad in the common
case of no truncation.

Fix test-header-parsing test script.

It was using sh's backticks that chomped the last linefeed!
Add comment referencing the commit that fixed the code this is testing.
Have fewer test cases, sticking to just one or two character headers
with empty or one-character bodies.  Avoids slowing the tests too much
and the fixed bug was to do with hashing one-character headers.

Simplify folder_exists() to just testing stat(2)'s return value.

Replace printf("...\n") with puts("...").

Replace v?fprintf(stdout, ...) with v?printf(...).

Added (y|n) domain to a couple of build_nmh config prompts.

Set first character of name[] buffer in BODY state.

Fix to 47313b9bc395470311e383b57eb184c830662c4a to prevent
uninitialized read, revealed by test/burst/test-burst-mime
with NMH_VALGRIND=1.

Removed configure options now that it determines them automatically.

Moved build_nmh to top level directory.

Reworked output handling.