Added checks of return value of fwrite() calls where missing.

Fixed formatting of rtrim/nortrim in mhl man page.

An "rtrim" flag has been added to mhl to remove any trailing
whitespace from filtered text lines.  A corresponding "nortrim" flag
has also been added.

If a component has trailing whitespace, e.g., body:component="> ",
mhl now trims that whitespace off when filtering blank text lines.

Fixed all anomalies detected by clang static analyzer (with
default checkers on Linux).  One was notable:  there was a break
missing from a switch case in fmt_scan.c, which caused the result
of the sday function escape to be garbage.

Fix typo in man page

On some platforms, -D_FORTIFY_SOURCE=2 warns about ignoring the
return value from mbtowc() even when it is called to reset the
shift state.  So wrap that in an if statement with a null body.

Added -D_FORTIFY_SOURCE=2 now that the build is clean with it.
Removed -Wno-unused-result because it shold no longer be necessary.
Adjusting all settings of CPPFLAGS to have consistent form.

Added a couple of new directories and a note about valgrind
warnings from debuginfo to README.developers.

Check return values of system calls in uip/ and test/ .c files, found
with gcc's -D_FORTIFY_SOURCE=2.

Quote "$SEARCHPATH" in mhn.defaults.sh to allow spaces, etc., in
PATH components.

Doubled size of buffers used by m_getfld(), scan, and fmttest to
8192, on any platform, so that -width 0 and -outsize max are
likely to include at least some part of the body of each message.

Fixed typo in mh-profile.man.

Don't let scan() use too large a width (BUFSIZ for now) to prevent
allocating a huge output buffer.

A value of 0 for the width switch of scan(1), inc(1), ap(1), dp(1),
fmttest(1), and mhl(1) now means as many characters as the format
engine can produce [Bug #15274].  That amount is limited by internal
buffers.

Fixed read of body in fmttest to limit size to that of rbuf, if
smaller than outwidth, to avoid overrunning it.

fmt_scan() no longer subtracts 1 from the width.  This has the effect
of no longer counting the trailing newline in the output of scan(1),
inc(1), and the other programs that rely on it.

Very very rough cut at trying to parse email addresses with Bison.

Check return values of system calls in several sbr .c files, found
with gcc's -D_FORTIFY_SOURCE=2.  Updated checks added previously to
mhfixmsg.c to be consistent.

Moved reverse_alternative_parts() from mhfixmsg.c to mhparse.c
and added declaration to mhparse.h.

Removed msh relics from scansbr.c, including a global.

Added size_t cast of another MB_CUR_MAX to silence the compiler
on FreeBSD 9.

Added cast of MB_CUR_MAX to size_t to silence compiler warning on
FreeBSD.

Dynamically allocate space for the output of fmt_scan(), using
charstring_t, so a fixed sized output buffer is no longer needed.
Also got rid of scanl global.

Added const to last argument of trace_cb signature, it'll be
needed for use with charstring.

Added charstring "class".

Refer to RFC 2046 instead of 1521 in mhlist.man.

Changed dist_contrib_DATA to dist_contrib_SCRIPTS so that all
of the scripts will be installed with execute permissions.

Updated mhfixmsg(1) man page to refer to mh-mkstemp(1) instead of
mktemp.

Checked return value of write() calls in mhfixmsg.c,
found with gcc's -D_FORTIFY_SOURCE=2.

Removed h/msh, hopefully the last msh relic.

ncurses package is required to run all of test suite on Linux and Cygwin.

Print information about the compiler toolchain on Darwin and FreeBSD.

Add new build tool: tools/showbuildenv

This prints some basic information about the build environment.
It's intended use is to print some information about the specific
build environment for each of the builds in the buildbot cluster.
In particular, the patch level of the OS, and the versions of
3rd-party packages installed on the system, compiler versions, etc.

Removed second argument from m_Eom() in m_getfld.c because
it was unused.

Here's a better fix to m_Eom() in m_getfld.c than commit
d2520ac7054ad75d60342606bf13c821305d958c.  The comparison
of the return value of Getc() with EOF must be as an
integer, not a char.

Removed eom_action from m_getfld.c because it was only used by
m_eomsbr(), which was removed by commit
68e8c25f906e7353269502d2292c8e99aa8c6605.

Removed mhlsbr().  It was only used by msh, too.

Removed m_eomsbr() from m_getfld.c, and its scan_eom_action()
wrapper.  It was only used by msh, which was removed by commit
e6917522a770cf2dba9997ca047977bc55fac061.

Added -debug switch to pick(1) and deprecated $MHPDEBUG.

Fixed refile(1) -retainsequences when the source and destination
folders are the same.

Separated out list of programs that are required to build from a
source code snapshot from those that are required to build from a
distribution.

Fixed sed usage in flex fixup to not use -s (or -e, because
even that's not needed).

Removed "true" from end of flex fixups.  I don't know why it was
there.

Apply flex fixup to dtimep.c with flex 2.5.37 as well as 2.5.36.

When mhparam(1) is going to output the value for spoollocking,
call mts_init() first in case it was set in mts.conf.

Updated test-spoollocking to restore mts.conf when it's finished
(successfully) so as to not upset subsequent tests.  Also fixed
removal of existing spoollocking option each time through.

Note in mh-folders(5) man page that spoollocking can be overridden
in mts.conf as well as with configure.

Fix bug #42718; ali(1) still refers to removed options -normalize and
-nonormalize.

If the user didn't specify any of the tls switches to post(8), try to
help them by implying -initialtls if they're using port 465 (smtps,
until IANA revoked that registration in 1998).

Here's an even simpler fix to enable TLS by default.

Fix to commit 03e76aecdf671ca13b5912af8206e9bdcb6c0919 to enable
TLS by default.  Boolean logic is our friend.

Revert "Default to enabling (Cyrus) SASL, if available."

This reverts commit fd23674a5829d5092c99a07110086f02f0b4a238.

Turn back the sasl test.  Everything broke as a result.
Not surprising.

Revert "Comments should say why a chance matters ..."

This reverts commit 7e6d0b76b1869b4fecc2412ac68f9739ee5916ea.

Comments should say why a chance matters ...

Default to enabling (Cyrus) SASL, if available.

Fix the release notes to put things in the correct section,

Note that post uses the submission port now.

Enable TLS by default.

Check return value of setlocale(3) and admonish on failure.

Add missing variable initializations on fmt_scan().

Ensure getline() is in scope on FreeBSD.

On FreeBSD, to make the getline() prototype visible, you must
'#define _WITH_GETLINE' before including <stdio.h>.

Fix to commit 0d3875d020fb17d34918f5e81ee7f87fbc5d1565:  it seems
that only Linux uses the locale name of en_US.utf8, so added check
for the more common en_US.utf-8 to the tests that use it.

Reordered #includes of sbr/terminal.c to allow compilation on
Solaris 10.  Thanks to Michael Urban for report the problem and
verifying the fix.

Skip (parts of) tests that require en_US.utf8 locale if it's
not available.  Thanks to Alexander Zangerl for reporting the
issue.

Test to make sure inc works properly with a POP server that has
multiple messages on it.

Support multiple messages with fakepop.

Always default the username to the local username, even when not
doing SASL.  Fix suggested by Michael Urban.

Use variable-size (but in one case, fixed but larger) buffers for the SASL
exchange to deal with larger SASL messages (specifically, when doing GSSAPI
with certain ticket extensions).

mh-profile.man: rewrite the Profile Lookup section for clarity

thanks to Ralph Corderoy for the text.

If "curl" is available, create a nmh-access-url entry that uses it.

Fix curl example for nmh-access-url entry.

Return an appropriate error if the input characters are 8-bit, but the
locale character set is US-ASCII.

Whoops, this should be LC_CTYPE.  Fixing that exposes the lack of an
appropriate environment variable in test-utf8-body

mhlist: don't truncate anything when -verbose

without this change to the -verbose output, there's no easy way to use
mhlist to get an unmodified content-type or content-description.  (the
non-verbose output isn't changed.)

Clarified description in comments of context_find_by_type().

Added static to declaration of docc().

Undid commit 475fef2edee9db3c63fc089df76f8e93e3d598cd, it didn't
seem popular.  We'd be better off with a proper method, or using
formail(1), to extract the addresses from a message.

Split assignment and export of shell variable.

Added mention of "Reply to <address>?" prompt to repl(1) man
page and code, as comment.  Suggested by Norm so that scripts
can depend on it.

Added context_find_by_type(), helper function to search first, if
subtype is non-NULL, for invoname-string-type/subtype and then
invoname-string-type.  Also gets rid of some fixed-size temp buffers.

Fixed test-mkstemp to work with valgrind.

Fixed to commit 378589b17424bd78acbfc57d505383ffb8c256cb:
when a -file switch will be used, insert -[no]concat before
the filename (and therefore, the -file switch itself).

Don't pass -[no]concat from show(1) to mhl(1).

Fixed typos in comments.

On platforms with no MIMETYPEPROC, declare the content to be
binary if it contains any NUL characters.

Explain how to get a literal % in format files.

Documented steps to build RPM from tarball, without using autogen.sh.

Added autogen.sh to EXTRA_DIST.

Handle unknown message types in the generic content handler rather
than in the RFC822 handler; this will cause them to be displayed using
an output marker rather than throw an error.

Output a newline if the last character in a text/plain part is
not a newline (but only for mhshow).

Move the prototype for show_content_aux() to a common header file; it
got an extra argument recently but none of the other users of it
noticed.

When fixing this I discovered that the usage of show_content_aux()
by mhstoresbr.c was wrong; the prototype had an extra argument in
the middle of it.  Apparently this has been broken for 15 years!

Switch from %p to %l (%p is no longer supported, although we silently
accept it).

fix comment syntax

clarify scan_content() logic for choosing encoding

Added support for mhbuild-disposition-<type>[/<subtype>] profile
entries when mhbuild expands Attach: headers.  Default to
'attachment', but override with 'inline' entries for text/calendar
and message/rfc822 in mhn.defaults.

change mhlist to use decimal math when abbreviating sizes

the K/M/G/T units imply factors of 1000, so make the code
match.  (the alternative would be to change the units to
Ki/Mi/Gi/Ti.)  mhlist output might someday be configurable
using mh-format, in which case the user will be able to choose.

add $(kibi) function, as complement to %(kilo)

%(kibi) will convert a number to IEC prefix units, i.e. Ki, Mi, Gi,
Ti, representing factors of 1024.  (by comparison, %(kilo) represents
factors of 1000.)

rename %(units) to %(kilo)

per discussion here:
  http://lists.nongnu.org/archive/html/nmh-workers/2014-05/msg00092.html

Added --dereference option, if supported, to file(1) command
in NMH_MIMETYPEPROC and NMH_MIMEENCODINGPROC macros.