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.

Reworked default SASL support to enable if header and lib are found.

Added -r to gcov invocation.

Add -O0 instead of -O2 to CFLAGS with build_nmh -d option.

Don't add -D_FORTIFY_SOURCE to AM_CPPFLAGS with -O0.

Merge branch 'master' of git.sv.gnu.org:/srv/git/nmh

Replace fputc() with putchar().

Changed build_nmh sasl and tls defaults to be configure's.

Merge branch 'master' of git.sv.gnu.org:/srv/git/nmh

Replace putc() with putchar().

Replace putc() with putchar().

Replace putc() with putchar().

Others already existed.  Not being a macro probably won't hurt.

Removed unnecessary definition of tls_support.

Replace puts() with putchar().

Replace puts() with putchar().

Reworked default TLS support to enable if header and libs are found.

Also, fixed AC_CHECK_LIB for libssl.  The crypto lib
needs to be the fifth, not fourth, argument.

Removed OAUTH support detection from because configure does it.

sbr/netsec.c: Add NMH_UNUSED(nsc) twice for if !TLS_SUPPORT.

netsec_set_tls(): Return a value in the !TLS_SUPPORT && !tls case.

Silences clang.

More enhancements to build_nmh.

1) Display failure messages that were hidden.
2) Display location of log file at end, if not in current directory,
   the build failed, or verbose was enabled.
3) Replaced use of showbuildenv with less invasive platform info.
4) Don't colorize tests summary if TERM is unset or dumb.

nmh.h: Add getline() prototype, conditional on HAVE_GETLINE.

Fix flex 2.6.1's output regarding signed/unsigned comparisons.

Add comment referencing flex bug.
Note this workaround breaks $(LFLAGS).

mhical.c: display(): Replace memcpy() with struct assignment.

fix_filename_param(): Remove redundant strlen().

fix_filename_param(): Replace strncmp() with HasSuffix().

sbr/utils.c: Add HasSuffix(s, suffix).

HasSuffix returns true if non-NULL s ends with non-NULL suffix.

Use LEN() to save having to consider the -1.

Use LEN() to save having to consider the -1.

Use LEN() to save having to consider the -1.

Remove unused NCWD and NPWD #defines.

Use HasPrefix() instead of strncmp().

Use HasPrefix() instead of strncmp().

Use HasPrefix() instead of strncmp().

Use HasPrefix() instead of strncmp().

Use HasPrefix() instead of strncmp().

Use HasPrefix() instead of strncmp().

Use HasPrefix() instead of strncmp().

Use HasPrefix() instead of strncmp().

Use HasPrefix() instead of strncmp().

Use HasPrefix() instead of strncmp().

nmh_command_generator(): Fix indentation.

Use HasPrefix() instead of strncmp().

Use HasPrefix() instead of strncmp().

Use HasPrefix() instead of strncmp().

Use HasPrefix() instead of strncmp().

Use HasPrefix() instead of strncmp().

Use HasPrefix() instead of strncmp().

Use HasPrefix() instead of strncmp().

Alter HasSuffixC()'s char * to be const.

utils.c: Add HasPrefix(s, prefix).

HasPrefix returns true if non-NULL s starts with non-NULL prefix.

Rename EndsWithC() to HasSuffixC().

mh-folders.man: Remove empty `.I' immediately followed by `.IR'.

build_nmh: If cloning with git, make a shallow clone.

showbuildenv: Drop lsb_release(1);  /etc/lsb_release tested earlier.

build_nmh: Use `gzip -d' rather than assume tar has -z.

build_nmh: Have a silent curl still show errors.

Arguably, -sS should be curl's defaults!

Fixed output of commit hash by git log command.

Check for existence of tools/showbuildenv.

Redirect verbose build_nmh messages so they're visible while running.

Enhanced build_nmh to support standalone use.

1) If not in nmh directory, download the nmh sources from savannah
   using git clone, wget, or curl.
2) Added -b branch option, for use when downloading.
3) Support -l -.
4) Capture build environment with tools/showbuildenv.

man: Use lowercase for command name at start of sentence.

`Inc' is a different command to `inc'.

uprf(): Rewrite, using types rather than masking ints.

Add comment describing uprf().

Use ToLower() instead of loop.

Use ToUpper() instead of loop.

Use ToUpper() instead of loop.

Add ToLower(s) and ToUpper(s).

putcomp(): Use local to avoid repeating tautological ternary.

Alter netrc permissions error message to match code.  Add test.

The code wants `go=', but the error message, and FAQ, said to clear read
permission from group and other.  Add test to inc/test-pop as that is
the existing netrc test location, but it might be better in its own test
file as other tests for netrc parsing could be added.

Fix netrc's token()'s bug if backslash escapes EOF.

Discard the whole token so the user hopefully investigates.

Fix netrc's token() bug of first character being backslash.

Merge .netrc's token() file-reading loops;  quoted and not.

Check for ferror(3) as well as feof(3) on entry.
Comment on lack of definition of quoted-string syntax, e.g. «"foo""bar"»
is two tokens without needing a separator.
Add comments on bugs for first non-quote character of a token being
backslash, and if a backslash escapes EOF.

Have ruserpass() return TOK_EOF and put "" in the token table.

Gives 0 a name and removes a little logic.

Replace getcpy() with mh_xstrdup() where the string isn't NULL.

Check advertise()'s iob[] isn't too short with assert(3).

Using DIM(a) added to h/mh.h

Use macros in advertise() to simplify source.

Happens to remove strlen() of literals at the same time.

Document advise(), adios(), admonish(), and advertise().

It doesn't make their design any better.  Their "ad" prefix goes against
common advice of having distinguishable identifiers to avoid mix-ups,
and seems to be a theme that got out of hand.  The out-of-order
parameters to advertise seem due to lack of strerror(3) that's no longer
a problem.

Before:
    advertise("open", "continuing...", "couldn't read profile: %s",
        path);
Could be:
    error("couldn't read profile: %s: open: %s, continuing...\n",
        path, strerror(errno));
And the "open" is typically dropped as superfluous so there's a
repetitive leading NULL passed a lot.

POSIX, 2016 Ed, removes bcopy(3).  Stop referring to it.

Even though it's in a comment, it shows up in searches.

Don't need to cast to `char *' for free(3) these days.

Don't need to cast to `char *' for free(3) these days.

Don't need to cast to `char *' for free(3) these days.