DEFAULT_PAGER: Move from nmh.h to mh.h.  Former is "config.h".

The macro isn't related to platform independence, unlike the rest of
nmh.h, that starts by including config.h.

geteditor.c: Hard-code the default, "vi".

It was the only use of the DEFAULT_EDITOR macro, so that can be deleted.

Move base64.c's prototypes from prototypes.h to new base64.h.

mhbuild.man: Fix grammar indentation by expanding tabs.

The grammar is pre-formatted using spaces for indentation.
A few lines had switched to tabs in 9b706433f.

decodeBase64(): Limit length of error message on invalid input.

Screenfuls of email were appearing on stderr.

Restrict use of alloc_size function attribute to gcc >= 4.3.0.

It's not supported by gcc 4.2.4.  CHECK_PRINTF doesn't work
perfectly with gcc 4.2.1, so disable that with gcc < 4.3.0 as
well. Fix to commit c066b395274021182a5f0530ccfeb1bcd167d860.

Move read_line.c's prototypes from prototypes.h to new read_line.h.

Add copyright comment to start of read_line.c whilst there.

Add ENDNULL macro for gcc's `sentinel' function attribute.

Annotate a few prototypes with it.

Move terminal.c's prototypes from prototypes.h to new terminal.h.

Fix spelling in comment.

read_switch_multiword.c: siglongjmp(3) removes need for `else'.

mhlsbr.c: Don't need to `else' after longjmp() at end of then-block.

Remove comment asking if siglongjmp(3) should be used;  it is.

77a8a873b switched from longjmp(3).

Don't need to `else' after mhladios() at end of then-block.

Give mhldone() and mhladios() the NORETURN function attribute.

And remove `FALLTHRU' comments after their calls.

Don't cast zero to pid_t for setpgid(2)'s first argument.

Don't cast zero to char for storing at pointer to char.

Don't cast zero to a non-function pointer;  use NULL instead.

Better still, omit it when BSS does the job, or C's treatment of
non-zero as true is sufficient.

mhparse: Change mhfixmsg flags from int to bool.

mhparse.h: Add externs for skip_mp_cte_check, etc.

Delete the local copies from the other user.

Delete declaration already available from included header file.

mhparse.h: Add externs for preferred_types[], etc.

Delete the local copies from other users.

Delete declarations already available from included header files.

mhbuildsbr.c: Include mhcachesbr.h.  Delete copy of prototype.

mhfixmsg.c: Make mhfixmsgsbr() static.

Move NPARTS and NTYPES from mhparse.h to mhmisc.h.

It's mhmisc.c that has the global arrays that use them.

mhmisc.h: Add mhmisc.c's global variables as externs.

Delete the local copies from other users.

Create mhmisc.h with mhmisc.c's global-function prototypes.

Delete the many local copies from each caller.

Make functions with no external callers by name static.

rcvtty.c: Remove duplicate prototype.  Other in mts.h.

aliasbr.c: Remove duplicate prototypes.  Others in aliasbr.h.

mhparam: exit(3) zero if all components found, else one.

The exit status used to be a count of the number missing, clipped to
120.  That doesn't seem useful, and is overhead to document, read, and
test.  Use the normal Unix 0 or 1 instead.

Ensure that mhfixmsg exit status is either 0 or 1.

Ralph noticed where exit status values were being accumulated, and those
values could include -1.

Add gcc's pure function attribute.

Mark quite a few function prototypes with it.

inc.c: Make maildir_srt() static.

Replace add(foo, NULL) with mh_xstrdup(foo).

add()'s arguments are back to front so add(foo, bar) produces bar+foo in
the normal case.  Thus add(foo, NULL) is read as the jarring NULL+foo.
Removing the NULL with mh_xstrdup() avoids this.  FENDNULL is used when
it isn't obvious foo can't be NULL as add() treats it as "" in that
case.

Replace strlen("foo") with LEN("foo").

The existing LEN() uses sizeof.

context_find.c: Hoist strlen(3) out of search loop.

Add die(fmt, ...).  Equivalent to adios(NULL, fmt, ...).

Avoids the noise of the `NULL' first parameter that's used in over 70%
of adios() calls.  Removes the possibility of it being omitted and `fmt'
being used instead.  `die' is already in use in nmh's shell scripts.
Had to rename post.c's existing die() to avoid it clashing.

mhparam: Remove `libdir', deprecated in 1.7.

Use existing `libexecdir' instead.

pick: Remove support for MHPDEBUG environment variable.

It was deprecated in 1.7, and the -debug option gives the same
behaviour.

Add MAX_EXIT, value 120, to limit some more exit(3) values.

mhparam(1) altered to use new macro.  ap(1) and dp(1) now limit.

mh_xmalloc() et al: Use C99's `%zu' to print size_t.

Treat %(myhost) failure to match canonical hostname as information.

Not an error from test-myhost.

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

Add gcc's nonnull function attribute.

Apply it to some prototypes to get started.

Use C99's variable-argument preprocessor macros for ALLOC_SIZE.

ALLOC_SIZE2() can be deleted.

Add gcc's malloc function attribute.

Mark wrappers for malloc(3), strdup(3), etc.

fmt_rfc2047.c: Tables hexindex[] and index_64[] can be const.

Add gcc's const function attribute.

Add gcc's alloc_size function attribute.

Used for malloc-wrapper, etc.

Add more NORETURN and CHECK_PRINTF function attributes.

And fix the errors that the compiler then detects.

Changed failure exit status of mkstemp(1) from 255 to 1.

For consistency, on Ralph's suggestion.

forw.man, mh-mime.man: Add missing commas in `SEE ALSO' list.

Inspired by 6db45651.

picksbr.c: Specify parameters of nexus's n_action function pointer.

Omitting them is obsolescent by modern C standards.

Print pointers in debug with C99's `%p' rather than `0x%x'.

The `%x' needed a double cast, the `%p' needs just a single to void
pointer.  The output can differ, e.g. `0x0' v. perhaps
implementation-defined `(nil)'.

picksbr.c: fprintf function pointer with unsigned-long-long cast.

It was previously a double cast, to unsigned long, and then to unsigned
int, and formatted with `0x%x'.  Now we're C99, at least, we can use
unsigned long long and `%#llx'.  Though the fprintf() in question should
be a BUG() that abort(3)s.

Fixed extra and missing trailing commas in SEE ALSO sections of man pages.

build_nmh: Stop -d enabling assert(3)s.

c347c3bb enabled asserts by default so alter -d's description to not say
it enables them as this may make users thing that have to disable
optimisation, -d's other action, to get them.  Don't add
`--enable-assert' to configure's options.

Format with `%#x' instead of `0x%x' if difference matters not.

Zero formats as `0' rather than `0x0', but that doesn't matter in debug
output, or an error message about a byte's value to the user.

mhlist -verbose: Don't duplicate message/external-body's parameters.

The test of eb_flags remains, and since the old code used to insist
eb_access, `access-type', was set, the new code adds a note if it's
missing.

mhparse.c: Correct comment; si_val is the `anonymous' Boolean.

mhlist -debug: Print FILE pointers with `%p', not `%#x'.

Removes the double casting.

mhbuildsbr.c: Ensure temporary FILE closed regardless of success.

If the fopen() succeeded, but the fwrite() failed, then the FILE wasn't
fclose()'d.

mhbuildsbr.c: Flip logic, moving goto to then-block;  no need for else.

Also makes more clear a FILE pointer that's probably not fclose()'d.

Flip logic, moving goto from else- to then-block.

Thus removing the need for the `else'.

Don't need to `else' after done() at end of then-block.

Don't need to `else' after adios() at end of then-block.

Only true of simple if-else rather than a chain.

sh scripts: Remove `else' after exit at end of then-block.

Only for simple if-then-else-fi with no elif.

uip/rcvtty.c: Flip logic, moving `return' to then-block.

There's then no need for an `else' around the old then-block.

    Before          After
    if (foo)        if (!foo)
        bar();          xyzzy();
    else                return;
        xyzzy();    bar();
        return;

Remove the `else' after a simple if-then block ending in `break'.

    Before          After
    if (foo)        if (foo)
        break;          break;
    else            bar();
        bar();

mhlistsbr.c: Replace list_application() with body at call site.

list_application()'s comment said the function didn't need to exist.
It's correct, so delete it.  The sole caller now calls list_content(),
as list_application() used to do.  That's actually the same as other
cases in the switch so merge them.  There is a slight difference:
list_application() used to call list_content() and then return OK
regardless, now it returns list_content()'s value, but that's always OK
too.

mhmail: Turn sh's else-if into elif.

Join the lines of another couple of `else if'.

It's misleading to have them on separate lines, especially when there's
a blank line in between.

Make sure we return the descriptor from the file handle.  Noted by
Ralph Corderoy.

Reformat else-if that are split over two or more lines into one.

No functional change intended.

Convert some Boolean variables to the bool type.

Convert some Boolean variables to the bool type.

Add ZERO(p) for the typical memset(p, 0, sizeof *p) dance.

Also seen as memset(&foo->bar_xyzzy, 0, sizeof foo->bar_xyzzy).  I find
it tedious to keep checking the parameters are in agreement when reading
the code.

Removed conditional addition of -D_FORTIFY_SOURCE=2 from CPPFLAGS.

And added to CFLAGS in build_nmh.

man: Vet the NAME sections, especially mhfixmsg's.

Don't limit mhfixmsg to fixing "nmh MIME" emails.

cppflags.m4: Don't trample CFLAGS and CPPFLAGS.

They were always being restored, but not necessarily saved first.  There
might still be some odd behaviour in this area.  I'm seeing -D...
preprocessor symbols disappear on subsequent runs causing compilation
problems, e.g. strcasecmp(3) not prototyped.

Add -D_FORTIFY_SOURCE=2 to CPPFLAGS instead of AM_CPPFLAGS.

To make it easier for packagers to override.
Fix to commit 35d2b4dc00d705e6816bcb0ccab491aabda688dc.

Restored old NAME in mhical(1) man page.

mhical only operates on iCalendar event requests, not messages.
Fix to commit 0e7daaa2daf6c668c8c6feb9da3c9a6639fa4e7e.

scan.man: Note that %{body} does not decode MIME.

sbr/mf.c: Simplify logic, ditching endless for-loops and switch.

Some of the control flow follows the pattern of the larger sections: an
endless for-loop with a switch, and then cases that continue or break,
and another break out of the for loop.  For the simpler cases, it's much
easier to read a simple while-loop that achieves the same in fewer
lines.  No functional change intended.

sbr/mf.c: Remove QUOTE macro, use its '\\' definition instead.

The name suggests a quote character, not the backslash as an escape
character that quotes what comes after it.  It's easier to read '\\' and
instantly know what's meant.

contains8bit(): Don't fetch a byte when start >= end.

The test for a NUL byte was before the test for start being before end.
Whilst here, hoist the test of end out of the per-byte loop, alter the
function to return bool, and merge the two, different, comments
explaining the function's behaviour.

NEWS: Add bug fix: EPIPE when writing error no longer recurses.

Also move an existing item into the Bug Fixes section.

NEWS: Consistent punctuation.  Mention SI and IEC quantities.

test-sendfiles: Feed `lzma -cd' stdin rather than a filename.

Håkon Alstadheim reported that Debian's lzma-9.22-2 would complain at
`lzma -cd foo.tar' with `unknown suffix -- unchanged' even though it was
not being asked to alter the file, or produce a new one based on its
filename.  This appears to be https://bugs.debian.org/700681.  The lzma
from Arch Linux's xz 5.2.3-1 does no have this fault.  Work around it by
feeding the file to decompress on standard input for all the compression
programs being tested;  none of the others should mind.

Protect fileno() call against NULL argument.

Fix to commit 337b4e616e8f53ba06285b1645e1df9918ed5c16.  Thanks to
Norm for reporting and Ken for diagnosing the bug.

lock_file.c: Compiler warns of unused isnewlock if HAVE_LIBLOCKFILE.

Default CFLAGS to -std=c99 instead of -ansi.

gcc -ansi -pedantic issued a few warnings about C90 not supporting the
"ll" printf length modifier.  Ralph pointed out that
http://www.unix.org/version4/overview.html says:

    The following source code portability standards lie at the core of
    the Single UNIX Specification:

        POSIX.1-2008

    (This is technically identical to the Base Specifications, Issue 7;
    they are one and the same document.)

        The ISO/IEC 9899:1999 standard

So perhaps we're wrong to expect modern POSIX code to compile as C90.

mhical expected an ics file as input and after just pressing
^D I've found that mhical wasn't happy about an empty input.
A trivial patch that should fix this issue.

Both f_typestr(): Remove parenthesis from long list of returns.

It's not a function call, they're just noise.

Both f_typestr(): Rebuild switch from FT_* to string from scratch.

Adds missing cases, e.g. FT_LS_CFIND.  Removes inconsistent "FT_" prefix
from some strings, e.g. FT_LS_DECODE.  Re-orders cases to match
definitions.

fmtdump.c, fmttest.c: Remove tests for FT_LIT_FORCE.

It isn't defined, and git-grep(1) suggests it was a local Lawrence
Berkeley Laboratory modification that output a literal without consuming
any of the width budget.

Makefile.am: Remove `test -d' guarding `mkdir -p'.

Makefile.am: Remove multiple `mkdir -p' for etc;  use ./configure.

A `mkdir -p' was used in each rule that created a file in etc to ensure
the directory already existed.  Ken pointed out existing ./configure
code to do that for the man directory, just once.  Use that for etc too.
Remove the `test -d' because `mkdir -p' does that itself.