Editing of man/rcvstore.man.

- delete some \- things
- some formatting
- change protection to mode, in line with inc.man
- Uncapitalize .SS Locking and \-unseen
- sort SEE ALSO

Editing of man/prompter.man.

- remove some \- items; quite a few, actually
- put .B prompter on a line by itself, a number of times
- some formatting

Editing of man/prev.man.

- Delete trailing whitespace

Editing of man/post.man.

- Delete trailing whitespace
- Change inline mh-profile(5) refs to .IR mh-profile (5)
- Some formatting
- Change take to read
- Say 'all message drafts' instead of 'all message draft'

Editing of man/pick.man.

- Some formatting
- Delete trailing whitespace
- Change complimentary to complementary, because English
- delete some \- things

Editing of man/packf.man.

Editing of man/next.man.

- Delete trailing whitespace

Editing of man/rcvtty.man.

- Add missing 'command'
- Some formatting
- Remove a \-, the mark that dare not speak its name
- Use 'switches' instead of 'options'
- Sort SEE ALSO programs

Editing of man/mhfixmsg.man.

- 8bit -> 8-bit, nbit -> n-bit, except in flags and error msgs
- Add nmh to .SH NAME
- Formatting
- use .B for -outfile
- use .IR for mh-profile (5)
- sort SEE ALSO programs

Editing of man/mhstore.man.

- store -> stored
- some formatting

Editing of man/mhshow.man.

- Add some stray s's
- Some formatting
- Remove an \- object (I now pronounce \- as uh-unh; it's easier this way)
- Use alternative instead of alternate

Editing of man/mhpath.man.

- Add some commas
- Some formatting
- Delete empty .PP

Editing of man/mhparam.man.

- Add some commas
- Some formatting
- Delete 'just'

Editing of man/mhmail.man.

- Add reference to nmh in .SH NAME
- Delete empty .PP
- Remove a few 'various'es
- Some formatting
- SEE ALSO in alphabetical order

Editing of man/mhlist.man.

- Make clear that -headers is the default
- Some formatting

Editing of man/mhl.man.

- Refer to environment variables (in text) without preceding $
- Don't put environment variables in bold (as per most other pages)
- Some better formatting

Editing of man/mhical.man.

- Don't use capitalised .SH, when uncapitalised .SS will do
- Remove some early .SH/.SS to improve narrative
- Delete superfluous .fi

It could be argued that each of the (now) .SSs, which have been
demoted from .SHs, are unnecessary or, at least, not in line with
the other man pages . Demotion may suffice, for now.

Editing of man/mhbuild.man.

- EDTIOR -> EDITOR
- remove some \- things
- formatting
- use 8-bit, like 7-bit

Editing of man/mh-tailor.man.

- Straighten up some formatting
- Don't escape hyphens unecessarily
- Pluralise message (so that outgoing messages...)

Editing of man/mh-sequence.man.

- Don't escape hyphens unecessarily
- Remove duplicate 'when' (when when)
- Straighten up some formatting
- Bring RFC reference into line (don't hyphenate)

Editing of man/mh-profile.man.

- Clean up some formatting
- Replace an .sp with a .PP
- Remove a couple of empty .PPs and .REs
- Make the good English, yes please

Editing of man/mh-mime.man.

- Clean up .SH NAME to one line without .IR nmh (7) on it
- Don't use escaped hyphens when not necessary
- Clean up some formatting
- Use 'Attach:' consistently (instead of 'Attach')

Editing of man/mh-mail.man.

I think we can rise above ourselves, and delete 'antiquated'.

Editing of man/mh-folders.man.

Editing of man/mh-draft.man.

Editing of man/new.man.

Mention `nmh' in man-page NAME section for apropos(1).

Replace `maildrop' with `mail drop' in man pages.

Review inc.man.

First cut at a cleanup of inc.man, including: maildrop -> mail drop;
Ralph's mode/octal suggestion, with a change (by me) to refer to
chmod(1).

Attempt to fix test-ap on fbsd-10 build host.

Check for sufficient room for multi-column character.

Fix to commit 92128dacf8d5db02379e8f872dc50d31c6aaa55f.  The sympton,
reported by Valdis, was overrun of scan -width.

Escaped repl, in case the user has it as a shell alias.

Added etc/rmmproc.messageid.

Fix free(3) of unmalloc()'d string in whatnow's refile.

`ref +foo' at comp(1)'s whatnow prompt was trying to free "+foo".
Take a copy of the string so the later free works.
Tom Rodman reported the problem to nmh-workers.
Bug introduced in 3b6be5607a251a3a793e97382e251ce66ea2bca0.

Work around flex 2.6.3 bug by conditionally #undef yywrap.

There's no sign on https://github.com/westes/flex/issues/162 that they
will release a new flex with the fix soon.

Cleanup of .SH NAME sections.

Fixed forw.man to pass test-manpages.

Added test case for commit 4318012376e06229307c0ed4f62a00faae9edc63.

Don't let parse_mime() try to parse a directory.

Cleanup to forw(1) man page.

Added /* FALLTHRU */ comments where gcc 7 noticed their need.

And replaced similar comments to be consistent.  Used that comment
because it satisfies gcc -Wimplicit-fallthrough=4.

Separate out backquote expression to please Heirloom shell.

Tweaked to work with Heirloom shell.

Shell parameter expansion ending in space needs it to be quoted.

Echo CFLAGS in configure line, and only include if set.

Change "servers" mts.conf entry to only support a single SMTP server.
As part of this change, fix SMTP code so the chosen SMTP server is
configured correctly for the netsec code.

Reworked handling of multiple, space separated, smtp servers.

Suppress getcanon exit status check.

To allow tests to run when the local hostname doesn't resolve.

Support mhparam -debug along with -all.

Fix to commit 234c9cc4829d86589a079ace49cfa5bc251ef62c.

Limit mhparam's exit status to 120 missing components.

Otherwise, 126 and 127 would clash with bash and zsh's use.
Higher than 127 would look like signals.  And 256 would wrap
to a falsely succesful zero.

mhfixmsg: Use folder name to open second message of a sequence.

`mhfixmsg foo' where foo was sequence of more than one message would
sucessfully process the lowest numbered message of the sequence, but
fail to open(2) the second one as the folder was missing from the path.
For example, /home/ralph/mail/inbox/1 was used to open the first, but
/home/ralph/mail/2 was attempted for the second.  I guess this was
because a static array was being trampled during the first's processing,
though didn't try too hard to find where.  Making a copy of it fixed the
problem.

Remove unused test file.

Added delay for output file to be closed.

Fixed references to --with-cyrus-sasl, and --without-, in comments.

Make sure we always generate a Content-ID header for message/external-body
entities, as required by RFC 2045.

Fixed permissions on test script.

Fix to commit 77fff244ad1bb1e746c78bedc87659400e2c6535.

Rearranged check for whether to skip param value checks so
that no files are left over.

Update of commit 2cf4ff9dd2fba77c7f98bbd706b00d2e79a11a3b.

Added valgrind suppressions for system libs on MacOS El Capitan.

Fixed commit 60052b95778a065760a437859ee9b8a1cc4e39a9, wrong sense.

Removed if ! bash-ism.

Fixed a couple of tests to not fail when there is no text browser.

Added missing AC_SUBST of AM_LDFLAGS.

crawl_context: Remove unused struct member `total'.

It was maintained inconsistently, and checking what problems that caused
showed it wasn't used.

Changes to folder.man

Kill a couple of empty .PPs, resurrect a lost .SS, a little bit
of slash and burn, stray whitespace, the usual.

As far as I know, which is not far, the \- construct is only
required for \-args, not, e.g., sub\-folder.

I've changed instances of 'subfolder' to sub-folder.

I've changed the 'the' in

These folders are all preceded by the read-only...

to 'any'.

Changes to install-mh.man

Some more simple wordsmithing...

Change instances of 'Install-mh' to 'install-mh', in line with
other pages, switch to single `' around 'Mail', in line with
other pages, option -> switch, clean up a little.

A cleanup, and possibly some controversy: I've replaced

Rather than standard message skeleton

with

User message skeleton

My thinking here is that a) we can do away with the kinda weird
'Rather than', b) I find the distinction between 'default' and
'standard' to be vague and c) it only appears in a few pages
and is easily remedied. It turns up in comp(1) as "An alternative
to the standard skeleton.", where 'standard' is used in the same
way as 'default' is used in rcvdist(1), so I think it's worth
settling on something 'better' and applying it consistently.

My proposal is 'Default foo', for the foo in %nmhetcdir% and
'User foo' for the foo in the user's <mh-dir>.

Fix skeletons in comp(1), forw(1) and dist(1)

While I was here, I removed two 'empty' paragraphs (.PP) in forw(1).
They may have been intended as line breaks, but I think they are
unnecessary; feel free to overrule me!

Aside: I would not be unhappy if, some day, 'skeleton' was replaced by
something utilitarian like 'template'.

Remove .w3m from test directory.

Reverted commit 316068ee3b7105ea802c63b732ee4cfb584a2600.

It was a repeat of commit 30a9c9b62fa92df289079080eb5ee57d2fda3951.
Also, wrapped a few $MH_TEST_DIR with quotes.

Clean up, and change .SS case to Title Case, from UPPER,
in line with other pages.

Changes to rcvpack.man

Make the -fcc switch to repl actually work properly, and make sure that
any Fcc header in a replied-to message is not carried over to the draft.

Changes to fmtdump.man

Add a reference to fmttest(1), delete a superfluous 'the',
delete the '...is simply...' line.

Added check of -fcc with no fcc in components.

Protect repl -file against dereference of null mp.

Removed #ifdef FT_PAUSE protection.

FT_PAUSE is unconditionally defined in h/fmt_compile.h.

Removed unused references to FT_ADDTOSEQ.

They were #ifdef LBL relics.

Changes to mh-format.man

This is a beast, and we may not tame it at the first attempt.

Mostly language simplification, some grammar and formatting, trailing
whitespace, that sort of thing. There are one or two instances where
I've taken (small) liberties with language, in favour of 'comprehension
at first glance'.

There are some commented lines hanging around in this. I've left
them in, but I think they should go at the first opportunity.

Removed obsolete comment about use of getcpy().

Changes to dist.man

Switch 'in place' for inplace, remove 'in truth of fact'.

Changes to flist.man

Replace 'option' with 'switch' in a couple of places, correct
(hopefully) a few sentences, restore '.SS "Multiple Folders" to its
rightful place.

Changes to comp.man

Remove the slightly legalistic 'in truth of fact', and bring the
mention of man whatnow into the preceding sentence.

The line 'If the draft already exists, comp will ask you as to the
disposition of the draft.' makes no sense to me, and I'm struggling
to make it make sense. If we put to one side the fact that I think
it's a broken sentence, I still haven't managed to get comp to
accept - or prompt me for - a 'replace' option when 'the draft
already exists', so I'm afraid I have to admit defeat here and seek
the wisdom of the elders.

Changes to burst.man

I'm not sure how relevant burst(1) is these days, but here goes...

Added clarifications to abbreviations.

1) N must be a positive number.
2) The + can be omitted.
3) As many of the N messages that exist.
4) Can also use - (or +) with first, prev, next, and last.

Changes to nmh.man

This adds a reference to the 'COMMANDS' section, corrects - and
hopefully simplifies - a couple of things, and tries to untangle
the line beginning 'show displays...' in the seventh paragraph.

The line 'The first, previous, next or last messages, if they exist.'
doesn't seem to accurately cover the accompanying 'foo:N' listing
in the following, or am I missing something?

first:N
prev:N
next:N
last:N    The first, previous, next or last messages, if they exist.

Changes to ali.man

The following is the beginning of an attempt to clean up the manual
pages somewhat.

grep tells me that 'switch' is used 193 times, whereas 'option' is
only used 51 times, so I've tried to standardise on 'switch'.

Fixed description of anno -list switch.

As noted in commit 98c2e7d95bda262ef23e8f0838e5b86d08ed4e4f.
If -text is used with -list, its argument is required but
ignored.

Changes to anno.man.

(Also noted that description of -list needs to be fixed, will
do in next commit.)

showbuildenv: Add comment describing exit-status convention.

It helps explain why some commands are `... || true' despite no `set -e'
being in effect.

Revert "Test if the buildbots care about tools/showbuildenv's exit status."

This reverts commit c92a4119fca5d677055dc616255a7d1efec9d750.
They do care, showing "environment failed".

Rename recently added ToUpper() to to_upper().

Rename recently added ToLower() to to_lower().

Rename recently added TrimSuffixC() to trim_suffix_c().

Rename recently added HasSuffixC() to has_suffix_c().

Rename recently added HasSuffix() to has_suffix().

Rename recently added HasPrefix() to has_prefix().

Sticking with local convention, as Ken requested.

Test if the buildbots care about tools/showbuildenv's exit status.

Document that adios() does not return.  abort() ensures it won't.

Discussion with David confirmed that adios()'s de facto contract with
the caller is that it won't return.  Document that, and add an abort(3)
to ensure it won't occur.  Better that than return and blunder on until
a SEGV or corruption later.

mhlsbr.c: Don't hide mhladios and mhldone behind macros.

Locally defined mhladios() and mhldone() were called as adios() and
done() thanks to #defines that hid the widely-used functions with those
names.  Don't bother as it confuses the reader.  Just call the local
variations directly to make clear it's not the standard implementation.

Generalized condition for support of parameter value tests.

One last fix to test-charset.