Fixed abort of e.g. comp when draft folder is empty.
In this case mp->lowoff is 1 but mp->lowmsg equal mp->hghmsg equal 0.
Hence we have an access at mp->msgstats[-1], which corrupted our mem.
Gdb hadn't helped me for this, but valgrind showed me the problem.

Syntactic and comment changes.

spost: refactoring; removed -(no)format switches as they were not used.

spost: refactoring, added comments, removed unused code

Scan listings shall not contain body content. Hence, removed this feature.
Scan listings shall operator on message headers and non-message information
only. Displaying the beginning of the body complicates everything too much.
That's no surprise, because it's something completely different. If you
want to examine the body, then use show(1)/mhshow(1).
Changed the default scan formats accordingly.

Removed another deprecated mhn check.
The comment appeared to be wrong ... maybe inverted. I removed it.

Removed the, in 1997 deprecated, send option -noshowproc and env var NOMHNPROC.

Removed -pause from mhshow(1).
My goal is to have show/mhshow generate a single stream of bytes to be
piped through a pager. Content not convertable to text shall not be displayed
directly. This is my idea currently.

Bugfix: Removed -reverse switch from scan(1).
No more to say but to quote MTR:
-[no]reverse under #ifdef BERK (I really HATE this)
And JLR:
Lists messages in reverse order with the `-reverse' switch.
This should be considered a bug.

Alias: Removed the address group `*', which meant everyone on the system.
Therefore could remove `everyone' and `noshell' from mts.conf.
If one really needs to write mail to everyone on the system (not just
all regular users, which probably are in a group `user'), then he
shurely can invest a bit more effort to do so. Likely, the simple condition
to uids > 200 and not the shell `noshell' would anyways be to limited.
Better use awk or the like and do it by hand.

Rest of the MMDF-style mail drop support removal.
Only Unix-style (mbox) mail drops are supported now. Their location
is in `mailspool', which is reported in the  configure summary now.
If users like to have different locations, they can use the environment
variable MAILDROP or the profile entry MailDrop to overrule the default.
From mts.conf, mmdfldir and mmdflfil were removed.
msgchk(1) received several cleanups.
I touched sbr/m_getfld.c ... Wooohh ...

Removed mmdelim[12], which were the delimiters for MMDF-style mail drops.

Removed paragraphs which looked like forgotten to remove.

Removed support for MMDF-style mail drops.
I'm not sure yet if I've removed all of it but it should be most of it.

Synced man page mh-chart with send.
The man page mh-chart definately should be generated!
Automate this task, man!

Fixed path lookup for alias files: ali, send, whatnow.

forw/repl/dist annotations won't add address lists anymore.
spost does not support it and I wonder if we need it.
Remove the undocumented -idanno switch, actually a no-op, from spost.

Trivial typesetting fix in man page.

Removed the ATTVIBUG compile option.
If you do have the bug, then workaround it like described in the FAQ:

Subject: 06.01 What to do with "Problems with edit - draft removed".
From: John Romine <jromine at ics.uci.edu>
Date: Mon, 1 May 1995 00:00:00 -0800

  If your users are using an AT&T version of "vi", it's exiting with
  non-zero status (supposedly a count of the "errors" during the
  edit). Move "vi" to "broken_vi" and put it its place :

    #! /bin/sh
    /usr/ucb/broken_vi "$@"
    exit 0

  Alternatively, compile MH with the ATTVIBUG option.

  Then complain to your vendor that "vi" is broken, and they shouldfix it.

btw: ex-vi does not have this bug.

Removed EISREMOTE check.
It's not in POSIX and Google finds only nmh code containing it.
Maybe it was some BSD experiment. Who knows.

Removed the undocumented -unique switches. Seems they were JLR-only stuff.

Removed the -width switch from send, spost and repl.
Adressee header fields will always be folded for OUTPUTLINELEN (=72) chars.

Removed options to deactivate dashstuffing. Always dashstuff.
Maybe there are few RFC 934 messages around, but dashstuffing is IMO
more useful than bad.

Removed unneeded options from send.
Most of them were supported by post but are not by spost.
We don't need them. (More about dashstuffing soon.)

Removed the UCI code to read the signatur (= name) from ~/.signature.
If we want to read something from ~/.signature, then those lines that
form the footer of a mail message. What is called ``signature'' here
is the name of the sender; the name that is put into the From: line.
There are already several options to put it:
- environment variable: SIGNATURE (to change temporary)
- profile entry: Signature: (to set only for the mail system)
- GECOS field in passwd (fallback and global)
These should be enough.

Removed UCI-specific group leader code from conflict(8).

Removed the UCI alternative name of anno(1): `fanno'.
fanno didn't add the date. If you want this, run `anno -nodate'. Hey, you
can create a shell alias for it. What about that? ;-)

Removed the -header option from scan(1).
The heading had been very simple since the advent of format strings anyway.
If someone wants a heading, he can run folder(1) before.

Removed (allmost all) JLR-specific code.
There still is one fragment in fmt_scan(), of which I don't understand
the logic. I don't want to remove stuff I don't understand.

Removed LBL compile condition (format: addtoseq).

Removed several `#if 0' fragments.
We better don't carry dead code with us, it might start smelling.

Finally fixed the annoying `Invalid cross-device link' problem.
Due to a missing slash (/), temporary files were put into the parent
directory instead of the same one. See:
mhbuild: unable to rename output /home/meillo/Mail/send0zxfDu to
/home/meillo/MailmhbuildzQjMUm: Invalid cross-device link

Updated the man pages to the recent changes.

Fixed -mime for forw and repl.
If the value of an attachment header starts with `+', then it is treated
as the specification of a message in MH style: ``+folder msg''

Removed the mime command from whatnow and automimeproc (automhnproc).
The man pages need to be updated.
repl and forw are broken right now.

Activated Jon's attachment system by default and steamlined it.
I had done similar work already some time ago. See:
http://www.mail-archive.com/nmh-workers@nongnu.org/msg02112.html
This time it's the second go.

I changed Jon's attachment system to have a default attachment header
field name, or get it from the profile. I removed the command line options.
The new profile name is: `Attachment-Header'. Stored in attach_fmt.
I removed the the choice between several attachment formats.

A message gets mimeified if it has attachment headers or (and this is new)
contains any non-ASCII character. The body won't be scanned for lines
matching /^#/. The MIME structure is created automatically.

The content-typing of unknown attachments is still poor and needs to
be improved.

You shouldn't use the `mime' command at the whatnow prompt anymore.

Added a comment and a variable initialization.

Free memory after use in m_draft().

Explicit checks for return values of strcmp().
The return value of strcmp() is unintuitive for string equality tests.
I prefer explicit comparisions against 0. I changed this everywhere.

Removed isprefix() and replaced it with strncmp().
isprefix() had been ssequal(), which had not been what it advertised to be.
Now, all this is gone.
We could get problems if one of the arguments is NULL, as isprefix()
handled this situation, but strncmp() does not. It appears as if this case
does not show up in the code.

Removed forgotten unused variable.

Removed the non-LOCALE character code and the #ifdefs and simplified.
POSIX says: tolower() won't change its argument if it is not isupper().
And isupper() doesn't need an additional check for isalpha().

Refactoring in sbr/m_convert.c

Reserved special sequence names may be upper case too. Even without LOCALE.

Much more straight-forward handling of the beyond sequence.
Could remove the message flag SELECT_EMPTY in the same go.
This frees one more space for a user-defined sequence.

Various minor cleanups in sbr/m_convert.c.

Removed the unused message flag: DELETED.
Now we can store one user-defines seq more.

s/ALLOW_NEW/ALLOW_BEYOND/

Renamed all standard sequences (e.g. cur->c) and made them globally changeable
The full list:
cur -> c
first -> f
last -> l
next -> n
prev -> p
all -> a
new -> b (mnemonic: one beyond the last)
They are globally changeable in config/config.c
Could be that I'll add the old names again, as convenience aliases.

Inverted the second arg of seq_setunseen(), to be more natural.

Another fix for the strchr() transition: Check for NULL arguments.
Is now the point to reintroduce the brkany() helper? Hmm...

Activated default sequence negation (exclamation mark).

Activated a default unseen sequence; updated man pages to it.

Fixed bug in brkstring(), newly introduced by the switch to strchr().
strchr() treats the trailing null byte surprisingly:
``The terminating null byte is considered to be part of the string.''

Globally defined the names for the standard sequences.
I'll rework the whole code to use them and not have hard-coded names anymore.
The `new' sequence will be called `beyond' sequence (`b') from now on.
The `unseen' sequence will be set by default to `u' from now on.
The sequence negation will be set by default to `!' from now on.

sbr/brkstring.c: replaced static brkany() with call to strchr(); plus cleanups

Fixed minor typo in man page.

Removed remaining lines from the --with-pager=... removal, months ago.

s/vfork/fork/
I truly believe that vfork is wrong. Forking should not be too slow for us.

Relayouted all switch statements: case aligns with switch.
Desired style is:
switch (n) {
case foo:
...
}

Removed ``Face'' support in mhl.
It appears to be about `Face:' headers, but neither Jerry Peek, nor
Ken Hornstein knew more.

Removed references to unused default procs and commented config/config.c

Removed the configuration option NOPUBLICSEQ.
You can change it still, by setting mh_seq to NULL in config/config.c.

Renames: s/+outbox/+sent/ and s/cc:/Cc:/
IMO ``sent'' specifies more exactly what it contains. ``outbox'' would
mean to me that these messages are prepared for delivery, but are not
already sent.

Updated INSTALL and changed `nmh' strings to `mmh'

Reworked the man page system and some man page contents (mmh-intro).
Now it's possible to have two equally named pages in two different sections.
This appeared to be interesting for mmh(X), but I don't use it now. Let's
see if we keep the new system or even rework it to become simpler.

Be quiet! Only talk if you have something useful to say.

Adjusted Makefile to the rename: etc/scan.default -> etc/scan.nmh

Updated all man pages and renamed install-mh.man to mmh.man.

Unimportant stuff.

Replaced C program `install-mh' by shell script `mmh'.
It is updated to the separated directories.

Split nmh dir into mmh dir (~/.mmh) and mail storage (~/Mail).
The profile (~/.mh_profile) became (~/.mmh/profile).

The idea was to separate the configuration files and the data (= messages).

The environment variables changed accordingly:
MH -> MMHP (profile)
MHCONTEXT -> MMHC (context)
new: MMH (mmh dir)

mh-install is not adjusted, yet.

Fixed bug, reported by Phil: Profile entry Draft-Folder was case sensitive.
The components in the profile should be matched case*in*sensitive, always.

Cleanup in m_draft. There was some confusingly unnecessary code.

Minor change in output message

Removed the global alias file
Either use personal alias files or the MTA alias file.

Moved etcpath() from config/config.c to sbr/path.c

Code cosmetic

Adjusted the default scan listing: remove the body preview
The original listing is still available as etc/scan.nmh

Code cosmetics

We have `NULL' in order to not need to write `(char *)0' anymore.

Removed void casts for ignored return values of function calls.
IMO, this distracts from the important without being much worth.

We don't want to use copy() anymore; but kept it for concat() only

s/pfolder/curfolder/g

"." is no longer an alias for "cur"
Don't fear more typing, as "cur" will soon be cut down to "c",
likewise the other predefined sequences.

Removed the option to disable `~user' support (MHRC)

Completely reworked the path convertion functions
Moved everything (from sbr/getfolder.c and sbr/m_maildir.c) into
sbr/path.c, but actually replaced the code almost completely.
See h/prototypes.h for the function changes.
sbr/path.c provides explaining comments on the functions.
None of them allocates memory automatically.

Additionally:
- Like for other ``files'', `inc -audit file' places file relative
  to the cwd, not relative to the mh-dir. This is for consistency.
- Replaced add(foo, NULL) with getcpy(foo), which ist clearer.

Cosmetic refactoring

Cosmetic change

Cosmetic change

Added comment

Simplification: Removed unnecessary variable

The existence of the profile entry Path is already enforced by context_read().
Hence, removed the obscure ``free(path("./", TFOLDER));'' calls.
Still I don't see the sense in these lines, but they won't get executed,
as the condition is checked before (and the program aborts then).
Probably that's obsolete stuff that remained through several reworks.

Code reformating.

Improved pluspath() and used it instead of path() where appropriate.

Refactored sbr/getfolder.c

Introduced FDEF and FCUR for speaking arguments to getfolder().
Maybe it would be better to two separate functions, but I cannot
imagine good names for them ... :-(

Rearranged code in the file.

Refactoring in sbr/path.c.
Especially: Replaced compath() with a more general packpath().

Renamed r1bindex() to mhbasename(), to make its function becomes clear.
All calls to r1bindex() had '/' as second argument. For everything else
one can use strrchr(3) instead of r1bindex().