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.

cpnumber(): Cast desired width to size_t to silence gcc's warning.

Add a comment on `wid = -wid' that it's OK because wid's value was
originally a short so won't remain the same value.

Fix bug in cpnumber().

The cpnumber() function (which handles the NUMF instruction, among others)
would hang if a 0 width was given to it.  Make sure that (and negative
widths) are handled correctly.  Note that normally NUMF did not handle
left padding which is indicated by a negative width; that may change
in the future.

Improve installation documentation

Fix up the various installation information so it matches reality.

Make sure the %(kilo) and %(kibi) instructions are in fmttest(1).

Man, we should really merge fmttest and fmtdump at some point.

INSTALL: Update c89(1) mention to c99(1), that exists today.

getcwidth.c: Use WCHAR_MAX, not __WCHAR_MAX__.

Former is clearly part of POSIX.

inc.c: Use closure for pop_retr()'s action callback.

Allows file-static variables to become local to a function.  In fact,
two of them no longer need to exist:  one of them becomes the closure's
struct's member, and the other is always 0 and probably just present due
to copy and paste.

inc.c: Alter pop_action() to check I/O, perhaps returning NOTOK.

popsbr.c: Alter traverse() to check action callback's result.

The action callback returned zero, with a 1999 comment asking if it was
checked.  traverse() didn't check it.  Change it so it does, wanting
`OK' if all's well, else returning the non-OK value to traverse's
caller;  though that only checks for NOTOK.  No functional change
intended.

pop_retr(): Add a `void *closure' to be passed to action callback.

Only caller passes in NULL for the moment, and doesn't use it in the
callback, but it will allow fewer scopes by name of the variables the
action callback uses.

inc.c: Narrow scope of inc_type.  Chain mutually-exclusive ifs.

Joining the separate if-statements with `else' tells the reader it's not
expected that more than one branch can be taken.  Useful when each is
many lines long.  No functional change intended.

scan(): Don't ioctl(2) for TTY's width every call.

Saves a system call per message scanned.  Does mean it won't adjust
should the terminal width change mid scan, but that seems fine;  after
all, the retrieved width could be out of date by the time a message's
output is written anyway.

m_getfld() et al: Replace with m_getfld2(), etc., in many places.

The difference is the FILE pointer isn't passed in on each call.
Instead, it's stored in the m_getfld_state_t once, on
m_getfld_state_init().  No functional change intended.

post.man: Detail -port's default value: submission, 587.

mhmisc.c: Indent with "%*s" rather than "%*.*s".

No functional change intended, but it does silence one of gcc's warnings
given severe enough options.

mhparse.c: Cast ptrdiff_t from subtraction to long.

Cygwin's 32-bit doesn't like `%ld' for the `p - q' difference between
two pointers as the value is an int, not a long.  printf(3)'s `t' length
modifier, for ptrdiff_t, can't be used as that's C99, and we're C90.
Cast the difference to a long explicitly instead.

m_getfld() et al: Add wrapper without FILE *iob parameter.

m_getfld(), m_getfld_track_filepos(), and m_unknown() all take a FILE
*iob that's used to update *m_getfld_state_t, even though it's probably,
hopefully, the same as the previous calls.  Create wrapper functions for
these with an arbitrary `2' suffix that don't have this parameter.
After checking the m_getfld_state_t is initialised, they pass in its
existing FILE *iob.  This allows callers to transition to the wrappers
in the cases where it's obviously the same FILE *iob being passed in.
It isn't always obvious, in part because of the convention for calling
this variable `in', even when it's a file-level static, a word that also
occurs in comments.

m_getfld.c: Make m_getfld_state_init() public.

This allows callers to use it if they wish to pass in its FILE *iob.

m_getfld.c: Alter Peek() to call Getc() and Ungetc().

Functionally equivalent, not called too often, and it gets rid of
another copy of the logic to decide if we've enough, should read some
more, etc.

m_getfld.c: Alter Peek() to match Getc()'s structure.

Their logic was equivalent, but written differently, which didn't help
the reader easily see they were the same apart from whether the returned
character was consumed.  Now they're visually similar too.

m_getfld.c: Use intmax_t and PRIdMAX to printf off_t values.

Compiling for ISO C90 prohibits casting to long long and printing with
"%lld", and printing as "%ld" is wrong on Mac OS where David reports
off_t is long long.