Added const to char * arguments of uprf().

Added ct_str_type() and ct_str_subtype() functions, to translate
MIME type and subtype names to the the internal MIME parser codes.

Merged two adjacent if (verbose) blocks.

Fixed memory leak in mime_type().  get_file_info() always allocates
its return value.

Always initialize those two variables made static in commit
771706ac9fed2b902bd4d3ddccb15ee27b0f58ae in case they were
set in previous calls.

And removed another improperly copied comment.

Removed improperly copied comment from test-mhfixmsg.

Added mention of profile components that are described
in other man pages.  Suggested by Michael Richardson.

Rearranged code to avoid "might be clobbered by longjmp" warnings from
gcc 4.9.2

Removed unnecessary #includes of setjmp.h.

Set COLUMNS in test-cd if using readline so that its wrapping doesn't
disturb the test output.  The problem arose with a long workspace path.

Fixed format engine output of negative number with 0 fill character.
Improper output was reported by Bob Carragher.

Rearranged mhn.defaults.sh a bit:  consolidated web-browser specific
settings and put most of the mhstore-store- directives in a here doc.

Clarified in repl(1) man page that -noformat is only the
default when no -filter switch is used.

Only remove extraneous trailing semicolon from Content-Type and
Content-Disposition headers because those are the only ones that
parse_mime() warns about, but it does appear on others.

Terminate last arg in proxy argv n popsbr.c:parse_proxy().

This bug seems to have existed since this code was born.  I guess
others have been lucky?

Set all Content structure elements to NULL as they are free()'d; in a few
cases a content structure is reused and you can get into problems with
double-free()ing elements depending on your MIME composition content syntax.

Merge commit '8206fbf', due to my screwup of committing it on a detached
HEAD after a git-bisect.

Turns out those warnings for ali(1) actually happen when the prefix is
33 characters, not 35.

Explicitly depend on $(srcdir)/config/version.h rather than depend
on VPATH.

cast isspace() arg to unsigned char in previous commit

handle extraneous trailing ';' characters that are followed by whitespace

mhfixmsg now removes an extraneous trailing semicolon from header
parameter lists.

Use Nmh-Attach for attach header field name, and accept Attach.

Have to check for missing type parameter in message/related part
even though it's required according to RFC 2387 Sec. 3.1.

Expanded explanation of mhfixmsg return status in its man page.

Look at type of parent multipart/related when determining whether
a non-plain text part already has a text/plain sibling.

Added support for multipart/related to MIME parser.

Added some checks to mhfixmsg.c to avoid dereferecing null pointers
after a failed transformation.

Fixed double fclose() in InitMultiPart() and leaks introduced in
commit 5be8db81.

prevent spurious errors when requesting specific message parts

no one should be reporting NOTOK for failure to display parts that
weren't actually requested.  failure should be reserved for parts that
we really attempted to display.

(this fix is an extension of c9794733.)

Updated simple integration example in mhfixmsg(1) man page
and added return status values.

Updated test-mhfixmsg to support -[no]changecur.

Added -[no]changecur switches to mhfixmsg(1).

Updated test-repl to compare with new -help message.

Fixed repl help message to show "-[no]cc all|to|cc|me" instead of
different options for -cc and -nocc.

Fixed leak in content_charset():  if get_param() found the charset,
it returns a copy.

Moved mhstore declarations from uip/mhn.c, uip/mhstore.c, and
uip/mhstoresbr.c to h/mhparse.h.

Removed "If this draft originated on the local host, then" phrase
from mh-alias man page.  Alias resolution applies no matter where
the draft originated.

Changed content_charset() to not cache, so the user can always, and
should, free the return value.

Cleaned up leaks from calls to content_charset() in mhfixmsg.

Added deallocation of cts to freects_done().

Fixed repl(1) man page to show "msg" instead of "msgs" because
it only allows reply to one message.

Fixed mlistfree() so that it doesn't dereference a free'd list node
in order to find the next node.

Fixed commit 6cc9e85e73ff6f6c67ae6e645d184bbf57ecc2f0 to properly
return other than the first mbox/address if there's a match in
getmymbox/getmyaddr.

Fixed memory management in mhstore.  The main problem was a double
free when both -file and -outfile switches were used on a MIME
message.

Added initial "MH-Profile-Version: 1.0\n" line to newly created
profiles.  The main purpose of this first line is to fool file(1).
Without it, if the first line of the profile is Path:, file 5.19
reports its type as message/news.  With it, it reports the type as
text/plain.

Renamed getmymbox mh-format escape function to getmyaddr, and
added new getmymbox that returns the user's address with any
personal name.

Added getmymbox function escape.

Fixed memory leak in do_name() when multiple dates are parsed.

Added free_fs(), corresponding to new_fs().

Fixed dst format escape to return 1 instead of 16, to agree with its
documentation.

Added mhstore-store-text/html formatting string to mhn.defaults.

Fixed typo in mhstore man page.

Reworked charstring_buffer_copy() to not use strdup() so that
charstrings with null bytes can be supported.  Added
charstring_append_cstring().

Added mh_xcalloc().

The units of the zone format escape are minutes, not hours.

Merge branch 'master' of ssh://git.sv.gnu.org/srv/git/nmh

no-op to force check the buildbots.

Merge branch 'master' of ssh://git.sv.gnu.org/srv/git/nmh

mhn.defaults: make fetch less agressive on FreeBSD.

Drop the -a flag to fetch.  Its retry logic has a bug where it will
go into an endless loop when trying to re-fetch an object behind
an https:// link when it is unable to validate the X.509 certificate
chain.

Use "mhparam etcdir" instead of hard-coded etc in build_nmh.

Clean atags file as part of maintainer-clean.

Add a missing option to mhn fetch command for FreeBSD.

I *always* forget the '-o -' to redirect to stdout :-P

On FreeBSD, use fetch(1) to retrieve external content through URLs.

fetch(1) is part of the base system, so it's guaranteed to always be there.
And it is more likely to have been configured for any local site
customizations (local caches, proxies, etc).

If building from a non-master git branch, include the branch name in the version string.

New locations for config files and support binaries, and smarter default for install root directory.

The default locations for config files and back-end executables has
changed.

Config files have moved from ${sysconfdir} to ${sysconfdir}/nmh, and
the back-end programs from ${libdir} to ${libexecdir}/nmh.

The only user-visible change is the addition of a new mhparam(1)
component 'libexecdir', which replaces the previous 'libdir'.  User
scripts invoking 'mhparam libdir' will need an update.  'libdir'
has been kept around for the time being, to give people a chance
to update their scripts.  It returns the same value as 'libexecdir'.
'libdir' has been marked deprecated in the mhparam(1) manpage and
the release notes; it will be removed in a couple of releases.

In conjunction with this change there was an opportunity to do a
bit of Makefile cleanup.  The old 'auxexec' construct has been
replaced with a new scheme that leverages automake's templating.
The auxexec_* references have been replaced with nmhlibexec_* forms,
and a corresponding nmhlibexecdir variable defined as ${libexecdir}/nmh.

Likewise, etcdir has been replaced with nmhetcdir.

These name changes have percolated downstream to some extent, primarily
in the man pages and test scripts.

Finally, configure has been made a bit smarter about setting the
default ${prefix}.  If it finds an existing nmh installation, it
will set the default ${prefix} to match, falling back to /usr/local/nmh.
configure looks for an existing installation by searching $PATH for
a viable mhparam binary.

Fixed commit 2adafe760a9c45b417727a3f5d9481de26471a8b to not
try to allocate a charstring_t of length INT_MAX with -width 0.

Simplified test-mhl-flags a bit.

Add a 'guide' file for acme(1).

If we ever find more than one acme user working on the source,
the single-guide-file premise might need a review.

Ultimately, there should be a way for the configure script to
search for and import a personal guide file from the developer's
$HOME or some such.  The next person to come along can figure
that out.

git: ignore top-level tags files

atags is the output from a ctags variant that generates its
output in a format recognized by Plan 9's acme editor.
(ftp://orthanc.ca/unix/atags/ for details.)

FreeBSD packages destination directory fixup.

If the ports collection is not installed, assume ports/packages
install under /usr/local.

Also, when querying <bsd.port.mk>, use the value of the $(LOCALBASE)
macro. ($(PREFIX) is for port-specific overrides of $(LOCALBASE).)

For FreeBSD builds, also search the ports tree for includes and libs.

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.