Changed msg_style and msg_delim to be file static to m_getfld.c
instead of global.

Changed type of name argument to m_getfld() from unsigned char *
to unsigned char[NAMESZ].

Added m_getfld () interface description.

Removed a few #ifndef JLR/#endif pairs because they're not needed.

Removed some dead code, all inside #if 0's.  I tried to not remove
anything useful, so there are a few #if 0's remaining.

Import a copy of Markus Schnalke's master's thesis: The Modern Mail Handler.

(Added with Markus' permission.)

Remove the peeking into the stdio internals inside of scan().  Hm, this
was more complicated than I originally thought.

Added release notes for releases 1.4 and 1.3 to tail of NEWS.

mhshow:  properly treat unknown multipart subtypes as "mixed"

Added test of -nosequence to test-pick.

pick: add -nosequence command line switch

Note in dist, mh-profile, and repl man pages that the @ link
is only created with -atfile.

Fix to 25581a94c5113eb78b2baf7110408df96efc4418: always set $editalt,
even without -atfile.

Added docs/contrib/vpick.

Cleaned up sendfiles(1) man page just a bit.

Corrected the last addition to man/nmh.man.  Might be nicer to explain
the warty truth from the off.

Updated the TODO file.

Copied atexit() code from fakesmtp.c to fakepop.c so that its
pid file gets cleaned up.  Also copied the sig handler.

Fixed formatting glitch in msh.man.

Added description of @folder format to nmh.man.

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

show: remove unused variable

this should have been part of the previous removal of unreachable code.

show:  remove unreachable code

the stanza in question has been impossible to reach for some time.  i
believe its purpose was to give a non-mh display proc full pathname
arguments to work with.  since we've chdir'd to the folder, giving it
message numbers (i.e., relative pathnames) is fine.

show: remove MAXARGS limit when running any of the helper procs

use app_msgarg() to maintain both the msgs[] and vec[] string vectors.

forw: remove MAXARGS limit when running mhlproc

use app_msgarg() to maintain both the msgs[] and vec[] string vectors.

burst: eliminate use of MAXARGS

use the already available app_msgarg() utility to make this easy.
(the MAXARGS limit was highly unlikely to be exceeded, in practice.)

Update about changes to rmmproc support.

app_msgarg: de-obfuscate

add some whitespace and break things up.  and add comment to reflect
that this might be (and is) used for more than just msg parameters.

Artificial argv limits.

when i skimmed the code earlier, i was looking at all the uses of
MAXARGS -- many of which are used to size local arrays.  converting
them all to be dynamically-sized might be a fair amount of work.

but looking at rmm.c, refile.c, and folder_delmsgs(), i don't think
that issue applies.  the patch below fixes the problem, and makes
rmmproc and refile do the right thing (well, at least, the modern
thing).  NB: i didn't test to the limit where execvp should return
E2BIG.

as for the rest of mh -- it's certainly possible to hit the MAXARGS
limit with other commands (show, for instance), but i suspect the
frequency is much lower, and those, too, can be fixed as needed.

paul

>From 9636bc0e1697829966f51d2c301cd5d730c38c0a Mon Sep 17 00:00:00 2001
From: Paul Fox <pgf@foxharp.boston.ma.us>
Date: Mon, 26 Nov 2012 20:49:55 -0500
Subject: [PATCH] rmm/refile: remove the MAXARGS limit when running rmmproc

Since pick now uses decode_rfc2047, we need to be sure to link in the
iconv library (if we need it).

In pick(1), attempt to decode each header field as if it were
MIME encoded.

In pick(1), attempt to decode each header field as if it were
MIME encoded.

A new test for the POP support in inc.  Very simple for now, but it seems
to at least check the basic functionality.

Add support for a -nosasl switch.

Fixed pick(1) to properly unfold multiple-line header fields
by removing newlines instead of replacing them with spaces
[Bug #15215].

The big news for this update is nmh 1.5.

Fix the race condition with fakesmtp so it works without needing retries.

Document changes to inc.

Document the -port switch to inc.  Also, fix the -port switch to inc so
it actually works.

Added test-slocal.

slocal used to support two different formats for three of
its arguments:

  [address info sender]
  [-addr address] [-info data] [-sender sender]

But it hasn't properly supported the first form since at
least April 1999.  So, I removed that first form from the
man page and the broken support from the code.  Score
another for the test suite!

Added Message-Id to messages in test suite.

Fixed a couple of signed-unsigned comparisons.

Yozo TODA: fix for mhshow-charset- directives in .mh_profile being broken

Merge branch 'format-memory-rework'

Make sure we strip off a newline from components we want to display (we
were stripping off a newline from the wrong copy of the component).

Convert repl to the new world format API.

Support for the new world format API for comp and forw.

Whoops, with the new world order we need to make sure we allocate memory
for all strings in format instructions.

Convert scan() to the new world format API.

Change fmt_findcasecmp() to fmt_findcasecomp().  The new name is more
consistent.

Change API a bit.

Rename fmt_addcomp() to fmt_addcomptext().
Create new function fmt_addcompentry().

Remove unused variable.

Removed leading space from lines in mhbuild and rmm man pages
to correct formatting.

Finished work on rcvdist to convert to new world format API.

More (and hopefully for now) the final cleanup for mhl.

Basic support for the new world API; seems to work!

Add missing functions for manipulating component text.

Fix up documentation a bit.

Partial work for mhl; still more stuff to do.

Fix missing argument to fmt_free().

Beginnings of conversion of code to new format interface.

Update documentation to match code change.

Fix a few bugs:

Make sure to add a ref when adding to the hash table.
Forgot the increment the format array inside of fmt_free().
Marked an allocated string when doing FT_LIT.

Add some comments to explain how format loop support was supposed to work.

Add refcounting to struct comp.
Move CHASH and FINDCOMP into fmt_compile.c to make private.

Beginnings of the changes to fmt_compile for memory rework.

Update with new functions and comments describing them.

Removed support for zip from sendfiles.  I added it as part of the
sendfiles rewrite, f8baa5a34335fc0fc8ccabfabc601d0af73ab769, a few
weeks ago.  But because sendfiles fed zip from standard input, zip
would use Zip64 extensions.  That would require unzip 6.0 or later
to extract, according to the zip man page.  But current MacOS X
uses unzip 5.52, so it broke there.

1) Don't bother checking stderr output from sendfiles because tar's
   output format isn't standardized.
2) Added -c to uncompress.

Updated scan.highlighted to feed an expression to %(zputlit).

Make sure we stick to POSIX-sed only.  I _think_ this portable enough to
work everywhere.

Change %(putlit) and %(zputlit) to take TF_EXPR instead of TF_STR.  This
was originally my fault; TF_STR means a string literal as an argument, but
the documentation (and the code) really means for it to take an expression.

Updated test-scan to correspond to commit
112b90c43391644f24559f5aa58752214de0e8a5.

Added ml script to docs/contrib.

Refer to just "root" in slocal man page.

Added ml script to docs/contrib.

In scan.highlighted, set black background on the colorized lines
to enhance visibility if the user has a light background.

Removed note from BUGS section of several man pages saying
that command line arguments must be quoted for protection from
the shell.  That's not a bug, and has nothing to do with nmh.

Changed "back-quoted" to "backquoted" for consistency in a few
man pages.

Fixed typo in mhn.man.

Fixed typo in mh-format man page description of zputlit.

Added application/vnd.openxmlformats-officedocument.spreadsheetml.sheet
for .xlsx files.

Added format support for zputlit function escape.  It requires
that the str contents have zero display width, such as for
terminal escape sequences.

nmh.7 isn't the place to advise people how to change their $PATH.

Fixed comparison to be signed instead of unsigned.

Added description of "all" range abbreviation.

Added argument to fmt_scan() to specify the buffer size.

Updated date on sortm(1) man page.

Updated build-nmh-cygwin for release of Cygwin nmh 1.5-2.  To use:
1) Change VERSION to 1.5-2.
2) make dist
3) Symlink build-nmh-cygwin to nmh-1.5-2 and run that with "all".

Fixed typo reported by Ralph.

I also should have mentioned in the commit message for
sortm(1) -check/-nocheck that Norm proposed their addition.

In test-post-common.sh and test-mhmail, wait for fakesmtp to start
for up to 40 seconds.  10 seconds wasn't long enough when I leaned
on my machine.  Also, print out a message if exiting due to that
timeout.

Added -check/-nocheck switches to sortm(1).  Also removed obsolete
note in the BUGS portion of its man page about how it sorts messages
that are missing or have an invalid Date: field.  It actually sorts
them using their file modification times.

Fixed typo in Cygwin section of MACHINES.

Cyrus SASL 2.1.25 introduced the sasl_callback_ft prototype,
which has an explicit void parameter list, according to best
practice.  So we need to cast to avoid compile warnings.
And provide a sasl_callback_ft prototype for earlier versions.

Clarify that manpage references use the .IR macro everywhere.

Fix one last missed %nmhdate% substitution.

Remove empty CONTEXT section.

Remove %nmhdate% substitution from man.sed.  It's no longer used.

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

Corrected date in sendfiles.man

Updated sendfiles(1):
1) Added -from switch.
2) sendfiles will now construct a From address, using the sender's
   localmbox, if the user does not specify it with -from or the PERSON
   environment variable.
3) Added support for bzip2, lzma, and zip compression methods.  Added
   a new switch to select the method:  -compress <method>.  Because
   support for the old -compress with was broken (it did not use
   -c with compress/uncompress) and it was undocumented, it has
   been replaced.  Retained old -gzip and -none switches for backward
   compatibility even though they were undocumented.
4) Allow the recipient argument to be preceded by the optional -to
   switch iff the subject argument is preceded by the optional
   -subject switch.  Allow the delay value to be specified using the
   optional -delay switch.
5) Added -version and -help switches.
6) All switches can now be abbreviated.

/etc/sendfiles no longer needs to be configured.  It's easier to
support make distcheck by having it figure out its path at runtime.

In viamail, set postproc to post in the same directory as itself, if
invoked with a full path.  This allows it to support make distcheck.
viamail does not read the profile so its postproc is otherwise
hard-coded at compile time.

Added test-sendfiles.