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.

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().