getpass.c: Don't fileno(NULL) when fopen("/dev/tty") fails.

If stdin is a TTY, but opening it for writing fails, then stdin and
stderr are used as defaults and the TTY doesn't have echo disabled.  The
later test to restore echo just checks if stdin is a TTY and not that it
was opened successfully causing a NULL FILE pointer to be fileno(3)'d.
Fixes da6af9633.  This function's logic remains a bit contorted;  I went
for the minimal fix.

mhbuildsbr.c: Don't free(3) string then print it; do reverse.

Fixes c87ca5cca.

smtp.c: Use read-end of pipe, not random integer.

Instead of `pdi[0]' the array of two elements was being indexed by `i'.
That is typically zero so everything works, but can be up to five
depending how many times fork(2) failed before succeeding.  Fixes
e65127948.

Revised expected output with -prefer in test/mhlist/test-mhlist.

Update to commit 5dce1b38ace6cf61398df3ee4ca5aee0e1dedb5d.

man: Fix some of the font-changing macros' parameters.

Various recurring problems on two themes.  `.BR foo,' lacks the space in
`.BR foo ,'.  `.IR foo' doesn't need to alternate and can be just `.I
foo'.  Other similar changes made, e.g. `.B foo' becoming `foo'.

Add -noprefer switch to mhshow, mhstore, and mhlist

Allow profile-specified -prefer switches to be cancelled from the
command line.

Change the priority ordering for multiple -prefer switches

Interpreting multiple -prefer switches in ascending priority
order allows the command line to override the user's profile.

man: Use `.IR' for `foo(1)', not the few `.BR'.

man: Change the tautological `the foo(1) man page' to `foo(1)'.

oauth: Replace `if...return false; return true' with `return ...'.

In functions returning bool, these are equivalent.

    Before:                 After:
        if (!foo())             return foo();
            return false;
        return true;

oauth.c: Remove unused assignments of function return value.

`if ((status = foo()) != OK) return false;' doesn't need to set local
variable status.

Replace FALSE and TRUE with C99's false and true.

Remove the duplicate definitions of FALSE and TRUE.  Change a few
variables from int to bool at the same time.  The semantics of bool are
different to int because `bool b = 42' maps 42 to 1, but if the code was
relying on that then it needs shaking out anyway.

print_intro(): Change parameter brief from int to bool.

flist.c: Use bool instead of int for some of the flags.

Replace boolean with bool everywhere.

boolean's comment said it existed to ensure storage was a char and not
an int so it could be packed in a struct, but the only struct using it
doesn't care about the space taken.

mh.h: Always define NMH_UNUSED as a cast to void.

The not-gcc-or-earlier-than-gcc-2 version was the identity function
seems unnecessary as what C99 compiler would complain about a cast to
void?

mh.h: Move the gcc-4.3-conditional macro definitions.

Having them outside of the gcc-2 ones makes the logic easier to follow
as there's no nesting.

dropsbr.c: Remove unused ntohl() macro, and its NTOHLSWAP test.

Not used since e6c91771 removed support for ./.foo.map index for ./foo
mailbox.

fakehttp.c: Remove unused LINESIZE macro; never used.

config.c: Remove unused nmhdocdir() macro.

ap.c, dp.c: exit(3) with 0 or 1, not [0, MAX_EXIT].

Don't attempt to indicate the number of failures through the exit status
as that's too constrained.  Just stick to zero for none, and 1 for some.
Delete, the now unused, MAX_EXIT.

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.