Remove return statements at end of void functions.

tws.h: Remove ADJUST_NUMERIC_ONLY_TZ_OFFSETS_WRT_DST.

Code that's conditional on its definition has been kept;  just the tests
removed.  The comment explaining its purpose has been kept, just without
the "If defined...".

Use C's `++', `+=', etc., not the longhand.

Remove unneeded incomplete, member-less `struct tag;'.

The forward declaration isn't needed to typedef based on the struct.

mh.h: Compact the reserved sequence bit-masks.

Removes the gap created by axing DELETED.

mh.h: Remove unused MODIFIED and DELETED macros.

MODIFIED was a folder attribute used by msh(1), and DELETED a message
attribute that may have last been used in the 1980s.
Adjust corresponding FBITS and MBITS definitions.

md5.h: Remove unused UINT2 typedef for unsigned short.

ap.man, dp.man: Add fmttest(1) to SEE ALSO section.

tws.h: Remove zero-valued TW_SNIL macro, used once.

Macro TW_SNIL was the zero value for a two-bit field.  It was only used
once, and that is better ordered to check for bits set with none set
being the last, else, case, rather than in the middle.  Kept the
behaviour of undefined value 3 being treated as TW_SIMP, 2.

tws.h: Remove redundant TW_SZONE and TW_SZNIL bit-masks.

TW_SZONE is a mask of one bit that can either be 0, TW_SZNIL, or set,
TW_SZEXP.  Rather than three macros for the single  bit, have one,
TW_SZEXP.  Simplifies the test too when the mask is known to be one-bit
wide.

mh-format.man: Fix `Return' column heading alignment.

Two of the column headings were separated by spaces rather than a tab.
The other two similar tables used `Return' instead of `Result', so
switch to that at the same time.

sbr/dtime.c: Remove struct-assigning twscopy().

Historically, twscopy() assigned all the struct's members individually,
but that was `#if 0'd as far back as the repository goes.  The one
caller now uses assignment, making clear the direction of data flow.

uip/sortm.c: Remove unused variable and dlocaltimenow() call.

read_hdrs() has a auto `struct tws tb'.  It's only use is as the
destination for dlocaltimenow()'s result.  Remove the variable.  Also
remove the dlocaltimenow() call;  this I'm unsure about as it has a side
effect of calling tzset(3).  However, tests pass.  Repository history
sheds no light.

sbr/dtimep.l: Remove always false `europeandate' flag.

Switched from reverse-podium MM/DD/YY interpretation to DD/MM/YY.
Never set so conditional code unused.

Append descriptions to C source files with first-line filenames.

Add first-line comment describing C source's purpose.

I had a stab at them after a bit of a skim;  they're better than
nothing.

Ensure a space after `#!'.

The majority of the `#!' scripts have a space before the `/';
add one to the three deviants.

Correct first-line comments where filename is wrong.

Having jiggled the start-of-file comment about, it's apparent that a few
files either don't know their own name, or the format the rest of the
gang follow.  To wit, "basename(filename) -- ...".

Replace "/*\n * " with "/* " at the start of a file.

Pulls the meat of the comment onto the file's first line, now that the
blank line at the start of the file has been deleted.  Allows `sed -ns
1p ...' or similar to give a summary of lots of files.

Remove leading and trailing blank lines from code.

Many of the main source file start with a blank line, and many end with
one or more too.  They seem to serve little purpose so delete them.
This makes all the files consistent in not having them and ideally the
first line of a C source file will say something about its content in
time instead of just "/*".  Avoided editing format files, etc,. where
the line could be significant, and historical files.

mts/smtp/smtp.h: Remove one-use rp_isbad() macro.

It just cast its argument to signed char, and the sole user can do that
itself a few lines later.

mts/smtp/smtp.h: Remove unused `MMDF' macros.

The header file has a specification of a historical interface and
removing the unused parts leaves it incomplete, but it's unlikely those
parts will be needed and what remains seems more understandable;  partly
due to it being less to grok, and also because it relates more to modern
use.

mts/smtp/smtp.c: Remove unused SM_... timeout macros.

SM_TEXT, SM_CLOS, and SM_AUTH are unused.
Alter the comment for the block of macros to state they're timeouts in
seconds.

h/mf.h: Remove many unused macros;  some UUCP related.

MF... were "Codes returned by uucp2mmdf(), mmdf2uucp()".
A spot check of the others shows they've not been used in this git
repository.

sbr/mf.c: Remove unused functions: isfrom, lequal, mfgets.

isfrom(s) checked for /^>?From /.
lequal(a, b) was another case-insensitive strcmp(3).
mfgets(fp, line) fetched a header at a time from fp, indicating when the
end of headers was reached.
Investigating their historical usage was, ironically, made too tedious
to complete by the pollution of mass commits of historical MH.

uip/mhbuildsbr.c: Remove unusused MAXURLTOKEN macro.

"Maximum size of URL token in message/external-body".
Uses deleted in 95ad91ca30e1cda09efe454d12ed790a60661f56.

sbr/dtime.c: Remove unused `struct zone' definition.

The uses went in 6a68057d8ef48bef3ee752741a53a629a2840e68.

uip/post.c: Don't bother naming `struct oauth_profile'.

It's never used, and one more identifier to track when reading.
If there's no name, that's a signal it's not used elsewhere.

h/nmh.h: Remove unused NLENGTH(dirent) macro.

It just does a strlen(3) of its argument's d_name;  that's not even done
longhand anyway.  And its name is very generic for such a narrow task.

h/mts.h: Remove unused isdlm{1,2}() macros.

They just do a strcmp(3) of their argument against mmdlm{1,2} and test
it's "equal".  That's more clearly written longhand than have yet
another small-scope, specialised, macro to recall;  as practice bears
out.

h/mh.h: Remove unused set_deleted() macro.

Its comment says "for msh only".

h/tws.h: Remove TW_YES and TW_NO;  TW_SUCC good enough.

TW_SUCC is a macro for the bit-mask 0x0040.  TW_YES, a vague name, was
the same value.  TW_NO was 0, and unused by the code.  Ditch both those
as TW_SUCC is sufficient for setting and testing.  Alter its comment to
make clear it means parsing was successful.

docs/FAQ: Replace suggestion of NOMHNPROC with -nocheckmime.

Environment variable NOMHNPROC was removed in 1.6, as stated in NEWS.

sbr/discard.c: Simplify by inverting initial test.

The function used to return early, but these days that's just avoiding a
single statement, so execute that conditionally instead.

sbr/lock_file.c: Move static variables into their functions.

A couple of file-scoped statc ints were the typical "if this function is
being called for the first time" flags.  Move each into their respective
functions.  Make them bools.  Name them so the initial value is zero,
thus placing them in BSS.

send and whatnow: Remove deprecated -attach... switches.

They were changed to do nothing but warn on stderr that they were
deprecated in 521674623 and 035c5db34, both before 1.6's release, and so
can be removed ready for 1.7's release.

uip/dropsbr.c: Fix buffer overrun in mbx_copy().

The overrun occurs frequently, but typically zeroes a byte of a
character pointer on the stack that's not yet been used.
Caused by 28610ff9a604a75ae0c383be03aa19415ddb1965 appending a NUL after
the bytes stored by read(2).

h/mime.h: Delete unused isatom(c) macro.

man/*.man: Fix some multi-word .B invocations.

Some needed to be .B for the first word, but .I for the second, or just
plain text for the rest of the words that should be on their own line.
Others were just plain wrong and became .IR for a man-page reference.
Then the .TP macro uses an input trap so only a single following line of
source is used and this prevents a two-line `.B .I', so use embedded
font escapes instead.

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...)