man/*.man: Use italic for emphasis, not bold or SHOUTING.

When reading a body of text, italic is sufficient to add emphasis to
what's already being read.  Bold draws the eye to the word on the page
and is more useful for keywords, concepts, etc.  Some `.B' remain that
should probably be `.I', but they looked to have other problems so I
left them out of this pass.

README.manpages: `user ID', not `user-id'.

man/*.man: Replace minus sign with hyphen: `\-' with `-'.

Conservatively changed many of the troff minus-signs, `\-', with a troff
hyphen, `-'.  Where it wasn't quickly obviously, I left it unchanged.
There were also cases that shouldn't be either minus sign or hyphen.
Those were changed to hyphen and still need to be fixed properly.

README.manpages: It's a `From header', not a `From: header'.

Use stat(3) instead of lstat(3), to dereference symbolic links.

Fix to commit 4318012376e06229307c0ed4f62a00faae9edc63.

Don't ignore bit 5 of a char when comparing alias names.

It's a crude method of ignoring case, but makes `^' equal to `~'.
Use tolower(3) instead.
http://lists.nongnu.org/archive/html/nmh-workers/2017-03/msg00050.html

man/mh-profile.man: Note MHTMPDIR is deprecated.

In future, only TMPDIR and /tmp will be used, e.g. not `mhpath +'.

README.manpages: Add `Common errors and subjective conventions'.

Based partly on recent mailing-list posts about Larry's patches.

README.manpages: Add question to guide update of .TH's date.

README.manpages: Clarify that man-page source is ASCII.

Fix post's die() to save errno before unlink(2)ing.

Allow advertise() to assume strerror(3) won't fail.

The code already assumes that elsewhere when it calls strerror().

Change admonish(NULL, "foo") to inform("foo, continuing...").

Removes more uses of the ad... functions with unneeded parameters.
Puts the "continuing" at the call site rather than the reader having to
remember it's being magically appended.  (Some calls were passing
"foo\n" that would result in "foo\n, continuing...\n" appearing.)

Don't store getname()'s return value during flushing.

`cp' after the loop is overwritten before being read.
Not storing the return value makes it clear it's not used.

Improve function comments for the ad... diagnostic functions.

Give the resulting layout of parameters that appears on stderr.

Fix bug in advertise("", "", "foo", ap).

A non-NULL but empty `what' would result in "fooerrno\n" without
separation.  Improve the function's comment to try and make it obvious
how its out of order parameters appear.

Replace advise(NULL, fmt, ...) with inform(fmt, ...).

Add inform() to provide an equivalent to advise(NULL, ...).

A small step in clearing up the confusing ad... names of the diagnostic
routines.  `inform' isn't great, but err(3) nabs `warn' already.

Replace add(nonnull, NULL) with mh_xstrdup(nonnull).

Only a few cases where it's obvious that the first parameter will not be
NULL.

uip/aliasbr.c: Add FIXME to prove test of add()'s return value.

add() always returns true so perhaps some other test is intended
instead.

man/*.man: Replace \(ru with an underscore, e.g. .mh_profile.

The character is an underscore, and that's what's wanted.
\(ru gives a "rule", which draws as a thick baseline in PDFs;
not a character at all.
Other devices map \(ru onto underscore, e.g. ASCII,
but an underscore should be used in the first place.

Replace some strdup() with mh_xstrdup().

(This is an old patch I had stashed.)
Some surrounding `if...adios()' are removed too, but most of the
strdup() calls weren't being checked for success, thus their
replacement.

Remove unneeded `\&' from man pages.

The zero-width character is used before a command character that is
intended to be literal but may end up at the start of a line.
Similarly, it's used after an end-of-sentence character when it may end
up at the end of the line but isn't the end of a sentence.
Remove the other cases because they're noise that confuses the issue of
when `\&' should be used.

Alter mh-chart(7)'s NAME to be lowercase.

And mention the commands' options.

man/*.man: No need for empty comment to be comments.

A line that's just a `.' suffices.

Ensure .TH is the first line of a man page.

Fix errors in man-page NAME sections.

man/fmttest.man didn't have a NAME section, but a FMTTEST one.
And it broke lexgrog(1) by splitting the section over multiple lines and
using `.IR'.  Re-write to use inline escapes.
man/rcvtty.man had two spaces before the `\-'.

Switch date to yyyy-mm-dd in generated mh-chart.man.

It's still today's date, but improving on that seems non-trivial.

Specify yyyy-mm-dd date format for a man page's .TH date.

Use `Unix', not `UNIX', in man pages.

"...in deference to dmr's wishes."
    — http://catb.org/jargon/html/U/Unix.html

Fix forward-reference and `news.*' in mh-alias(5)'s example.

Spotted by Bob Carragher.
The example's forward reference was made a backward one, and `news.*'
explanation removed, in ff3060cefb460cce2174058eefff38b6e11ee2f1.

Set man-page date to last significant change, UTC.

Wind man-page date back to last significant change, UTC.

Wind man-page date back to the earliest git revision, UTC.

No significant changes since then, and I can't find an earlier reference
date.

Convert, already correct, man-page date to UTC.

Convert, already correct, man-page date to yyyy-mm-dd format.

Remove trailing spaces from lines in man pages.

Escape literal leading full stop in man/new.man.

Error made in 5e32ede8e40251bf02f35b864e32a86e91599b3f.

Editing of man/rmm.man.

- formatting, mostly
- don't make NOT bold

Editing of man/rmf.man.

- delete trailing whitespace
- remove some \- things
- some formatting
- sort SEE ALSO

Editing of man/repl.com.

- unashamedly add nmh to .SH NAME
- replace the delightfully baroque 'may be used to produce a reply
  to an existing message' with 'may be used to reply to a message'.
- remove some \- objects
- deflower 'repl uses a reply template to guide its actions'
- remove multiple instances of 'replied\-to', with 'replied to'
- change 'tab\-stop' to 'tab stop'
- some formatting
- replace .B mhbuild with .IR mhbuild (1)
- replace whatnow with .B whatnow

Editing of man/refile.man.

- some formatting
- change .B mh\-sequence (5) to .IR mh\-sequence (5)

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.