Removed m_chkids(), per suggestion of Marcin Cieslak.

Removed unnecessary #include of h/signals.h.

Removed #include of signal.h because it's in h/signals.h.

Reset done in inc_done() to prevent endless loop when setegid()
fails (in which case adios() is called, which calls done()).

That didn't work, so removed offset.

Use +0000 isntead of -0000 offset because FreeBSD displays it that way.

Replaced setgid() calls with setegid() so that it works with dot
locking on FreeBSD.  setegid() should be supported on modern POSIX
systems.

Added showmimeproc to test/common.sh.in so that test-show will test
it in the workspace, not the user's installation.  Also added all of
the other procs defined in config/config.c that are currently unused,
just in case they some day are.

Removed a few more msh (MS_MSH and mshproc) and vmh (vmhproc) relics.

Replaced individual assignment of array elements, one of which was
out of range, with array initialization.

Fixed the error message in lockit() when it fails to create tmp file.

Fixed "inc" to be installed setgid mail with dot locking.  MAILGROUP=1
was not set in the config.h because we no longer use "LOCKTYPE"
configure variable.

update tests for new message separator header from mhshow

also added test names to tests, in any file i touched

update man pages for new mhshow message separator header

add new message separator header to mhshow

new switches to mhshow (-headerform and -[no]header) control whether
and what is displayed before the display of every message's content.
the default separator looks like "[ Message folder:msg ]".  the
implementation parallels the -markform feature.

since mhshow is often started by show, show now passes the -[no]header
options through to mhshow.  (also, show no longer attempts to pass
such options through to cat (in the case of -noshowproc)).

fix typo in mh-format.man

Test that show always shows message numbers.

show shows message numbers in two different formats when showing one
non-MIME, multiple non-MIME messages, or one MIME message.

show does not show any message numbers when showing multiple messages
any of which is MIME.

This is not a new bug, just one that has bothered me since I started
using mhshow with 1.6.

test/common.sh: allow giving individual tests a name

man pages -- clarify what -part and -type do when used together

Added support to mhfixmsg -reformat for multipart/related.  If it has
only a single part that is not text/plain and can be converted to
text/plain, a text/plain part is added and the type of the part is
changed to multipart/alternative.  If the multipart/related has more
than one part but does not have a text/plain part, try to add one.

Removed unused local variable from move_preferred_part().

pending-release-notes: add mhshow's "-prefer", and mh-format's %(kibi/kilo)

test-subpart: add mhlist and mhshow tests for -type/-part/-prefer

update mhshow, mhlist, and mhstore man pages for -prefer switch

implement -prefer, for multipart/alternative preferences

mhshow.man: more detail on use of -part and -type switches

mhshow: ensure that -part always shows every part requested

previously, "mhshow -part 1 -part 1.3" would show 1.1 but not 1.3.

mhshow: fix display of alternatives after certain nested multiparts

adjust return codes so that we now have a clear indication of whether
there was a hard error (NOTOK), a failure to display due to type
mismatch (DONE), or complete success (OK).  use those codes in
show_multi_internal() to decide whether we're finished with this
multipart or not.

prior to this change, it was impossible to show part 1.2 of a
message constructed like the following, because the disinction
between (the new, as described above) DONE and OK states was
lost while processing multipart/related part 1.1.
 msg part  type/subtype              size description
  28       multipart/mixed           1040
     1     multipart/alternative      623
     1.1   multipart/related          275
     1.1.1 text/html                  105
     1.2   text/plain                  33
     2     text/plain                  83

this fix is related to 9cc2510 and c9794733.  hopefully it's right
this time.

mhshow: simplify code -- the DONE status was adding no value

drop the second argument to part_ok(), since it is always 1

the second argument hasn't been used since "historical" times.
also add some commentary to part_ok().

Add a note about new bindir option to mhparam.

'mhparam bindir' now prints NMH's idea of $(bindir).

Have show_content_aux2(), and therefore show_content_aux(), return
the status of the process used to show, instead of (usually) OK.

Fixed typo in mh-format.man page, which I wouldn't have noticed
if I didn't go back in to update its date.

Added multiply format function, requested by Norm.

More cleanup of the cp in tests/common.sh.in.

Replaced explicit list of files to be copied in test/common.sh.in
with *.  It wasn't trivial, per the comments in the file about
portability to posh, but close.

OK, this fixes the problem referenced in the last commit.  It
removes the copy of the etc files only during make distcheck.

Reverted commit a2b00e7053ca3be3d1389588580c71498222a267.  It caused
make check to pull etc files from where they were already installed,
not from the test directory.

Removed export of most of the variables in test/common.sh.in.  The
test scripts source common.sh so they don't need them to be exported.

Removed copy of all of the etc files from test/common.sh.in.
Instead, for make distcheck, pass the location of the installed etc
directory via DISTCHECK_CONFIGURE_FLAGS, configure, and
TESTS_ENVIRONMENT to a new nmhetcdirinst variable.  Outside of make
distcheck, nmhetcdir is valid by itself.

Rearranged the variable settings at the top of test/common.sh.in.

Changed direct access to etc files in a couple of tests to be the
installed (for the test) versions, not the copies made by common.sh.

Replaced nmhexecdir with nmhlibexecdir in test/common.sh.in (and
test-mhparam) for consisitency with the Makefile.

Added -fixtype switch to mhfixmsg(1).  It ensures that each part of
a message has the correct MIME type shown in its Content-Type header.

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().