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.

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.