Moved all file static data in m_getfld.c into the
m_getfld_state struct.  Changed MSG_INPUT_SIZE from
hard-coded number to BUFSIZ.

Added check of -pack to test-pop.

Fixed test-header to work on Cygwin, which has a BUFSIZ of
only 1024.  That caused an error message to be truncated.

Removed all traces of m_getfld() FLDEOF and BODYEOF states.
They were never set by m_getfld(), so all caller code that
supported them was completely unexercised.

Minimized message files reads in m_getfld() when the caller
interleaves calls to fseek().  Added check of handling of
long header field name without a colon to test-header.

Added test-msg-buffer-boundaries.

More cleanup of m_getfld(), changed some variable names.

Added check of header field body with more than 511
characters to test-inc-scanout.  Also changed a couple of
test message files to be generated on the fly.

Cleaned up m_getfld() code a bit.  The most notable change
was to remove the check for mmdlm2 not being null.  It's
global data so that shouldn't happen.

Added check of a header with name that's 998 bytes long to
test-header.

Removed the for-loop that was used to get a header field
name in m_getfld ().  It was no longer necessary when not
drawing directly from the stdio buffer.

Renamed a couple of m_getfld auxiliary functions.

Cleaned up m_getfld's read_more().

In m_getfld(), fixed the loop that handles the FLDPLUS case.
In Getc(), removed the increment of the byte count at EOF,
it didn't seem correct or necessary.

m_getfld() now supports interleaving calls with
fseek()/ftell().  Reverted the changes to uip/mhparse.c of
commit 484eb1003ae647e4b751e4d7829c7daf44fb99c9 because
they're no longer necessary.

Moved test of empty header field from
test/inc/test-inc-scanout to test/bad-input/test-header.

Cleaned up a bit.

Added support for m_getfld() callers interleaving fseek() calls
with their m_getfld() calls.

Added bytes_read to m_getfld() buffer state.  This is the
next step in supporting ftell()/fseek().

Removed unused local char_read from forwsbr.c.

Added check of m_getfld() handling of fields with trailing
whitespace to test-inc-scanout.'

Removed "sleazy hack" of negative bufsz because commit
05d8af21d26673279a1ca079ad9c6f301426fe0e removed the only
use of it.

Cleaned up FLDPLUS handling in m_getfld().  As part of this,
removed the backing up of the read pointer in the "something
went wrong" portion of the FLD handling.  It had been moved
back two positions.  The first is no longer necessary
because a Getc() was replace by a Peek(), and the second
caused the extra blank line noted in the old comments.
Updated the expected output in test/bad-input/test-header to
correspond.

Fixed addition to test-inc-scanout in
683e0fb1a14da5b9ce7bc5db6e76951d14d79348 to not depend on
current date.

Very minor cleanup to m_getfld() to remove an increment of j.

Started revising m_getfld() code to replace direct buffer
access with Getc().  The goal is to again support fseek()
after return from m_getfld().

Added comments to and started cleaning up m_getfld.c.

When read_more() loads more bytes, save some from the end of the
current chunk and move them to the front of the buffer.  This
allows backing up of the current position, which is necessary when
looking for the message delimiter in a maildrop file.

Block SIGPIPE before killing the child in sm_end().  This replaces
the discard() calls that were removed in commit 4548a3ecc4248a1d4.

Removed discard() call in post(8).  It was only used with -debug
and seemed to serve no useful purpose any more.  Its comment
said that it was needed to help the loader.

Corrected fseek() emulation to fix test/bad-input/test-header.

Replace use of ftell() in mhparse.c:get_content() by keeping track
of bytes read by m_getfld().

Restored an explicit refill of the buffer in m_getfld() that I
removed in the big stdio cleanup.  Its need is shown by reducing
the message buffer size to 2048.

Removed call to fpurge() and its platform-specific emulations
from discard.c.  discard() is called on stdout from mhl(1) and
msh(1) in interrupt handlers, and post(8) but only with -debug.
tcflush() should do the purge when stdout is a terminal.  If
it's a file or pipe, well, maybe some garbage could get flushed
out.  And it's called on the streams that post uses to talk to
its smtp server, so removed those calls because they no longer
do anything.  And removed LINUX_STDIO define from configure.ac.

Replaced all the stdio buffer access in m_getfld.c with a single call
to fread() and then some almost straightforward buffer manipulations.

Fixed formatting of last change to scansbr.c.

Updated m_getfld() interface description.

Changed bufsz argument of m_getfld() to be in-out instead of in.
This allowed removal of the msg_count global, in turn allowing
removal of sbr/m_msgdef.c

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.