Switch from using script(1) to a new custom utility, runpty.  It turns out
on some systems script(1) has a bug that results in hangs that is simply
too difficult to test for, and this custom utility is simpler.

uip/mhbuildsbr.c: Fix long-line truncation when BUFSIZ is 1024.

a23477eb changed one char array from BUFSIZ to NMH_BUFSIZ elements, but
the long line read into it was passed to user_content() that
strncpy(3)'d it, silently truncating, to another char array, still
BUFSIZ long.  This show up on platforms where BUFSIZ is 1024 versus
NMH_BUFSIZ's minimum of 8192.

test/common.sh.in: Clarify it's the first failure by a *named* test.

Unnamed tests may have failed earlier.

test/mhbuild/test-cte: Move run_prog() to just before check().

Prepare the input and output files, then run the program and check the
result.  Makes it easier to see every run is being checked if a,
sometimes large, expected-output file isn't being produced in between.

test/mhbuild/test-cte: Add missing check() for earlier run_prog().

be6c3984 added a new run_prog and swiped the earlier check.

test/common.sh.in: Clarify test name in failure message.

Some test names don't stand out as that when suffixed with `failed'
amidst all the other output.  Change `foo failed' to `first test
failure: foo' instead.  Also makes clear there's possibly other tests
that failed later.

Make sure we are in the correct directory when we do "git describe".

sbr/check_charset.c: Fix warning on unspecified struct initialisers.

A different C compiler disliked `{NULL}' as the initialiser for a
two-member struct so specify NULL for the other member too.

Rewrite norm_charmap(), moving code into data.

Also removes return of static char array, and is more precise, e.g.
`CP1242' doesn't become `WINDOWS-1242' as it's not a known code page.

Merge commit '3d0e5b6a4383a5228732edd6cb1532375b771262'

Merge sbr/norm_charmap.c into sbr/check_charset.c.

Alter norm_charmap() to be static now its only caller is in the same
file.  I've checked the upstream cam.ac.uk source and it's still the
same and hasn't changed in years, so I don't think there's much benefit
from having it standalone to ease checking.

Part 1 of merging sbr/norm_charmap.c into sbr/check_charset.c.

This commit won't compile, but I'm attempting to get the rename into git
so the history after the next, merge, commit will continue through both
parents.

sbr/unquote.c: Move prototype from h/prototypes.h to new header.

sbr/mime_type.c: Move prototype from h/prototypes.h to new header.

sbr/message_id.c: Move duplicate prototypes to new header.

sbr/makedir.c: Move prototype from h/prototypes.h to new header.

sbr/m_popen.c: Move prototypes from h/prototypes.h to new header.

sbr/m_maildir.c: Move prototypes from h/prototypes.h to new header.

sbr/m_rand.c: Move prototype from h/prototypes.h to new header.

h/prototypes.h: Give sbr/lock_file.c's prototypes their own header.

sbr/utils.c: Delete unused upcase() function.

h/utils.h: Move in PLURALS() macro from h/mh.h.

h/prototypes.h: Remove duplicate prototype for pwd().

It is already in h/utils.h.

h/utils.h: Move in nmh_init() prototype from h/prototypes.h.

uip/mhshowsbr.h: Move in declarations from h/mhparse.h.

Delete non-existent markerform parameter from show_all_messages()'s
comment.  Alter show_all_messages()'s definition's concatsw parameter to
match comment and prototype.

uip/mhshowsbr.c: Create header file with exports' declarations.

Create a single set of externs for mhshowsbr.c's global variables.  Have
the other users of those use the new include file instead of their own
declarations.

uip/mhcachesbr.c: Move exported symbols to existing h/mhcachesbr.h.

Remove the user's copies of the declarations.
A static struct swit caches[] was declared everywhere that included
h/mhcachesbr.h.  The larger number of includers causes `variable not
used' for `caches'.  Move that declaration to uip/mhcachesbr.c and have
an exported `cache_policy' symbol that points to it for the existing
users.

uip/mhoutsbr.c: Move exported prototype to new header file.

Alter the callers to include the new header file instead of having their
own copy of the prototype.

sbr/m_mktemp.h: Move remaining prototypes from h/prototypes.h.

Gives the normal one header file showing interface exported by one C
file.

sbr/m_mktemp.h: Create with missing prototypes.

Some of sbr/m_mktemp.c's functions didn't have prototypes in a header
file so callers were declaring prototypes themselves.  Have them include
the new header file instead.

sbr/utils.c: Use h/signals.h for setup_signal_handlers prototype.

Deleted its own prototype for setup_signal_handlers() and added it to
the existing h/signals.h as it's defined in h/signals.c.

uip/picksbr.c: Use function prototypes for `nexus' functions.

Prototypes for ORaction(), etc., now state the parameters they expect.
Removed the args() macro, instead explicitly listing the arguments being
passed to functions.

Specify function parameters in prototypes, mainly void.

These prototypes are all in *.[cl] source compared with the *.h of the
earlier commit.  All but one had void added, that one's parameter was a
char pointer.

h/prototypes.h: Remove scan_reset_m_getfld_state() prototype.

Function removed back in 2014 by 58eaf2c2.

Use `void' in prototypes for functions with no parameters.

The old-style `int foo()' still lived on in a few places.

h/mhparse.h: Move mhfree.c's free_content prototype to new header.

It was the only thing of mhfree.c's in h/mhparse.h.

Match `NORETURN' function prototypes by adding it to definition.

Create and use header file for uip/mhfree.c's exports.

Remove all the duplicate inconsistent prototypes from mhfree.c's users.

test/mhical/test-mhical: Force UTC timezone for mhical.

Commit c3238c0e added a test but mhical's output depends on its
timezone;  force it to UTC for consistent results.

uip/mhfree.c: Make free_header() static.

uip/flist.c: Make locally defined and used functions static.

Support iCalendar event request files that don't end with a newline.

Such files do not conform to RFC 5545 § 3.1, which requires that each
contentline end with a CRLF.  But be liberal in what we accept.

Added explicit dependency to build sbr/icalparse.h before running LEX.

"Fix" to commit 665dfc96.  The build had been relying upon the listed
order of files.

Have post(1) report name of fileproc if it fails.

Replace calls to unputenv() with unsetenv(3).

Replace calls to m_putenv() with setenv(3).

sbr/dtimep.l: Remove redundant wrapping parenthesis in definitions.

It's been decades since a lex(1) didn't follow POSIX and treat `{foo}'
as parenthesis surrounding foo's definition.  Other nmh lex files work
without the extra wrapping.

uip/mhstoresbr.c: Fix single-character nmh-storage bug.

If the profile component nmh-storage was a single character, e.g. the
relative directory `d', then it would be dropped from the output path so
instead of `d/foo' being written, `/foo' would probably fail.  I think
this was due to a faulty check for nmh-storage being `/'.  Add a test
for a non-/ single-character nmh-storage.

sbr/fmt_rfc2047.c: Simplify test for /^=\?./.

Use new PLURALS(n) macro instead of variety of tests.

Whether to output a plural noun in a message was decided by ternary
operators that tested n==1, n!=1, n>1, etc.  Make them all consistent by
using PLURALS(n).

uip/folder.c: Use `%s' for plural, not `%c'.

Instead of a ternary operator with 's' and ' ' as the outcomes, switch
the format specifier from `%c' to `%s' and use strings "s" and " ".

uip/folder.c: Rewrite plural test to common form.

The common form is a choice between "" and "s".  This one occurrence was
using " " and "s" so either output was a fixed width.  Alter the
printf(3) format string from `%s' to `%1s' to provide that instead.
Allows the new code to be altered by an upcoming `plural' change.

Makefile.am: Alter long lists to be sorted, one entry per line.

It was hard to see what files were included in some variables when
searching for the best place for new content.  Some variables had
definitions that were almost sorted, by weren't.  Switch to a
one-entry-per-line list, in `LC_ALL=C sort' order, documented at the
start of the file.  Makes it easy to run through the lists and see the
patterns.

Use FENDNULL() instead of duplicate, locally defined, empty().

Three files had an empty() that was defined using FENDNULL();  use the
latter directly.

Add FENDNULL(s): fends off NULL by giving an empty string instead.

Use it in place of all the `foo->bar_xyzzy ? foo->bar_xyzzy : ""'.
Avoids the reader having to check first two of the ternary operands are
the same.

sbr/mf.c: Rename path to routepath to avoid h/mh.h clash.

Adding #include of h/mh.h brings two path global symbols together;
path() and char *path.  Rename the latter.  h/mh.h is needed by a future
commit.

Replace `a == b ? 1 : 0' and similar with `a == b'.

Replace `e ? 0 : 1' with `!e'.

Needs less thought to read.

sbr/icalparse.y: Replace max(BUFSIZ, 8192) with NMH_BUFSIZ.

Use existing macros min() and max() more.

Clarified the phrasing added by commit 44ce2010.

Added warning about when post(1) can't refile(1) a draft.

sbr/fmt_scan.c: Fix `foo%-42{bar}' right-justification bug.

The space padding was added at the start of the output buffer, not the
start of the component being formatted.  Caused by 92128dac's move to
dynamic allocation for fmt_scan()'s output.  Only shows if the component
isn't at the start of the buffer.  Expand existing
test/format/test-rightjustify to cover this.

config/version.sh: Rewrite.  Use uname(1), git-describe(1), and UTC.

Not sure why it searched through PATH manually for uname(1) and
hostname(1).  uname and its -n option are POSIX so just use those.  It
was the preference over hostname anyway.

Use git-describe(1), not just git-branch(1), as it gives more detail,
including --dirty to show the built source differs from the commit.

Specify the format for the build date, and its timezone; +0000.

sbr/mf.c: Simplify isat();  it's /^ at /i.

This one's for all those RFC 733 emails out there.
You know who you are.

uip/inc.c: Use bool, not int, for some of the options.

Makes clear there's no special meaning to repeating these options more
than once.

sbr/netsec.c: Remove extraneous semicolon.

Split some comma-operator terms into separate statements.

Some uses of the comma operator weren't the idiomatic `p++, len--'.
Split its terms into two statements instead, e.g. `close(fd); fd = -1'.

uip/picksbr.c: Increase line-buffer size for "grep" action.

Alter LBSIZE from 1024 to NMH_BUFSIZ, e.g. 8192.  This dominates the
size of the char array used to hold a single unfolded header line when
evaluating a "grep" action, e.g. `-to foo' becomes `^to[ ^I]*:.*foo'.

uip/scansbr.c: Replace uses of SBUFSIZ with NMH_BUFSIZ.

Now that the former is defined as the latter, the uses of SBUFSIZ were
all in the same expression and it simplifies from a ternary expression
to NMH_BUFSIZ.

uip/rcvdist.c: Replace SBUFSIZ with NMH_BUFSIZ.

The former was defined as the latter and had only one proper use: to
size a char array.  The other uses should have been the sizeof operator
on that array, and now are.

uip/replsbr.c: Replace single use of SBUFSIZ with NMH_BUFSIZ.

The former was defined as the latter;  not a lot of point having it.

Make many m_getfld() buffer parameters NMH_BUFSIZ big.

char arrays were often stdio.h's BUFSIZ large, and that's 8192 with here
with glibc 2.25-1 on Linux x86_64, so using NMH_BUFSIZ is no different
as that's max(BUFSIZ, 8192).  But some were 256 or 512 with a local
SBUFSIZ macro and they caused scan(1) to truncate a field, or pick(1) to
not spot text because the field was truncated.

Delete bad-input/test-header's test for m_getfld()'s detection for a
header without a colon that's under the header-length NAMESZ limit, but
longer than the passed in buffer.  This no longer happens in scan as the
buffer is larger than the longest allowed header.

configure.ac: Enable assert(3) by default.

Revert 167e542b that disabled assert(3) by default after private email
discussion with David Levine.  I suggested it being enabled by default
in git so those running nmh from git were testing them for us, with it
being disabled as part of the release process so packagers continue with
it disabled.  David pointed out that would make the release
intentionally different from what had been long tested so the default is
now enabled assertions everywhere unless disabled by a packager.

mhstoresbr.c: Flip logic to simplify.  Remove else after continue.

Former in store_partial(), latter in parse_format_string().

mhlsbr.c: Flip parse()'s logic.  Remove oneline()'s else after break.

dropsbr.c: Flip mbx_open()'s logic to simplify.

base64.c: Remove some else after break.  Flip logic to simplify.

mhfixmsg.c: Flip logic to simplify.  Remove else after break.

path.c: Simplify compath()'s flow;  return, don't break twice.

mhshowsbr.c: Remove else after break in iconv_start().

mhparse.c: Remove a couple of else after continue.

In openQuoted() and parse_header_attrs().

m_getfld.c: Remove else after break in m_unknown() and m_Eom().

picksbr.c: Remove some else after break in plist().

fmt_rfc2047.c: Remove else after decode_rfc2047()'s break.

sortm.c: Flip get_fields()'s logic to simplify.

README.developers: Add pointers to Debian's Lintian's complaints.

README.developers: Add blank lines for consistent headers.

Fix spelling in C comments.

bvector_copy: Use bvector's tiny storage if big enough.

Otherwise a source bvector that's using tiny storage has its bits
duplicated into a malloc'd area when the destination's tiny storage was
sufficient.

sbr/oauth.c: Remove const to avoid curl's debug_callback warning.

The curl_debug_callback typedef doesn't use const for any of the
function's parameters.  On one of the compilation platforms here,
oauth.c's debug_callback having const specifiers for some parameters
causes compilation warnings, and -Werror stops the compilation.  Remove
them.

sbr/folder_read.c: Don't malloc() once per message.

Instead of struct msgs having a pointer to a malloc'd array of pointers,
each to a malloc'd struct bvector, 1+N, it now has a pointer to a
malloc'd array of struct bvector;  one malloc for all of them.  This
avoids the large number of calls to malloc() and free() that's linear
with the size of the folder.

But there are some downsides.  In order to step through an array of
struct bvector, code outside of sbr/vector.c needs to know the struct's
size.  The simplest way to do this is to make the struct's definition
public, with a comment that access should be through vector.c.

New functions are needed to initialise the content of an already
allocated bvector, and to finish with its content prior to deallocation.
bvector_create() and bvector_free() now also use these new functions.

Before, it was the array of pointers to bvector that would be realloc'd.
That doesn't work for the array of bvectors as they may contain pointers
to within themselves.  The solution is to malloc a new array and
bvector_copy() the ones to keep across, as folder_realloc() now does.
The other half of its logic that coped with growth at the end of the
array, has been deleted.  Also deleted, is the code to clear the
bvectors before and after the old ones as they start in that state.

valgrind: Add suppression for dbm_open(3)'s write(2).

It writes bytes from an area it mallocs, but doesn't set all of those
bytes first.  See with gdbm 1.13-1 on Arch Linux.

sbr/vector.c: Only allocate bvector storage for set bits.

Now that a pointer to all the bit storage doesn't escape vector.c,
there's no need to allocate storage, that's initialised to 0, just to
then clear a bit in it.  Only extend the bits's storage for set bits.
Remove the unused initial size parameter from bvector_create() as it
simplifies the implementation.

sbr/vector.c: Add bvector comments, tidy identifiers.

sbr/vector.c: Rewrite BVEC_BYTES(n) macro to remove branch.

Use the idiom of integer truncation.

sbr/vector.c: Use new BVEC_BITS_BITS macro.

Factors out a common expression from a few other places.

sbr/vector.c: Zero the growth with memset(3), not loop.

When the resize functions grow the vectors, as they always do, zero the
new slots with memset(3) rather than a for loop.  Particularly of note
for the bit vector where it was bvector_clear()ing one bit at a time.
Although a NULL pointer needn't have a representation of all-zero bits,
the code was already assuming that, e.g. on the initial allocation, so
we're no worse off.

sbr/vector.c: Change bvector_bits() to return first word.

Rename it to bvector_first_bits() to represent its new behaviour.  It
has only one caller that uses it to produce debug.  Tighten the API so
other callers don't get access to the location of the bvector's bits.

sbr/vector.c: Embed initial vector storage in header struct.

Instead of a malloc(3)'d struct bvector having a pointer to a separately
malloc'd area for the bits in the normal case of the default initial
size, have non-malloc'd storage in the struct itself, and set the
pointer to that.  It's two unsigned longs, which is less than the
previous default of 256 bits for struct bvector, but still double the
pre-bvector norm of one word on 32 and 64-bit architectures.

This halves the mallocs needed to create a struct bvector in the common
case, but does mean that embedded memory is wasted should it not be
enough.  That's probably an unusual case.  It also means derefencing the
pointer to the bits probably hits the same cache line.

Have separate initial sizes for string and int vectors.