Use ToUpper() instead of loop.

Add ToLower(s) and ToUpper(s).

putcomp(): Use local to avoid repeating tautological ternary.

Alter netrc permissions error message to match code.  Add test.

The code wants `go=', but the error message, and FAQ, said to clear read
permission from group and other.  Add test to inc/test-pop as that is
the existing netrc test location, but it might be better in its own test
file as other tests for netrc parsing could be added.

Fix netrc's token()'s bug if backslash escapes EOF.

Discard the whole token so the user hopefully investigates.

Fix netrc's token() bug of first character being backslash.

Merge .netrc's token() file-reading loops;  quoted and not.

Check for ferror(3) as well as feof(3) on entry.
Comment on lack of definition of quoted-string syntax, e.g. «"foo""bar"»
is two tokens without needing a separator.
Add comments on bugs for first non-quote character of a token being
backslash, and if a backslash escapes EOF.

Have ruserpass() return TOK_EOF and put "" in the token table.

Gives 0 a name and removes a little logic.

Replace getcpy() with mh_xstrdup() where the string isn't NULL.

Check advertise()'s iob[] isn't too short with assert(3).

Using DIM(a) added to h/mh.h

Use macros in advertise() to simplify source.

Happens to remove strlen() of literals at the same time.

Document advise(), adios(), admonish(), and advertise().

It doesn't make their design any better.  Their "ad" prefix goes against
common advice of having distinguishable identifiers to avoid mix-ups,
and seems to be a theme that got out of hand.  The out-of-order
parameters to advertise seem due to lack of strerror(3) that's no longer
a problem.

Before:
    advertise("open", "continuing...", "couldn't read profile: %s",
        path);
Could be:
    error("couldn't read profile: %s: open: %s, continuing...\n",
        path, strerror(errno));
And the "open" is typically dropped as superfluous so there's a
repetitive leading NULL passed a lot.

POSIX, 2016 Ed, removes bcopy(3).  Stop referring to it.

Even though it's in a comment, it shows up in searches.

Don't need to cast to `char *' for free(3) these days.

Don't need to cast to `char *' for free(3) these days.

Don't need to cast to `char *' for free(3) these days.

Don't need to cast to `char *' for free(3) these days.

Don't need to cast to `char *' for free(3) these days.

Don't need to cast to `char *' for free(3) these days.

Don't need to cast to `char *' for free(3) these days.

But still need a cast to discard the const.
Use `void *' as it saves needing to check the original type.

Don't need to cast to `char *' for free(3) these days.

Don't need to cast to `char *' for free(3) these days.

Don't need to cast to `char *' for free(3) these days.

Don't need to cast to `char *' for free(3) these days.

Don't need to cast to `char *' for free(3) these days.

Move smtp_init's last-ditch attempt for client inside first test.

Didn't realise there was a second lot of similar code before the last
commit.

Move smtp_init's last-ditch attempt for client inside first test.

No point checking it twice in a row if it was false the first time.

Don't `else' after return.  Simplify control flow.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Don't test boundary in output_content() for NULL;  it never is.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Replace `if (p) free(p)' with `mh_xfree(p)'.

Add FIXME raising if iscntrl(3) test should be based on isprint(3)

!iscntrl blocks characters that could be isspace or isblank whereas
isprint would let them through unharmed.

Assume POSIX ctype.h;  don't vet toupper()'s parameter.

Explain isascii() test more in netsec_set_sasl_params().

It stops non-ASCII lower in the SASL mechanism being uppered, just in
case there is some.

Assume POSIX ctype.h;  don't vet toupper()'s parameter.

Assume POSIX ctype.h;  don't vet tolower()'s parameter.

Assume POSIX ctype.h;  don't vet tolower()'s parameter.

Oops, stop showbuildenv reading from stdin when run from a TTY.

David spotted the problem I introduced with 8374c3c.  Alter offset to
process any arguments, falling back to stdin if none.

Assume POSIX ctype.h;  don't vet tolower()'s parameter.

Assume POSIX ctype.h;  don't vet tolower()'s parameter.

Assume POSIX ctype.h;  don't vet tolower()'s parameter.

Fixed MIME parser so it can read binary content.

Second of two-part fix.  Replaced use of strlen().

Have m_getfld() pass back indication of missing blank line.

This is part one of a two part fix to the MIME parser to
read binary content.  It should have no effect.

Also added start/finish test calls to test-header.

Fix warnings caused by recent edits;  don't mix declarations and code.
Quite right too, I prefer all the declarations, codeless, up front.  The
standard -Wall didn't pick this up.  David pointed it out, probably due
to his defaults of `-pedantic -ansi'.

Don't `else' after goto.  Simplify control flow.

Don't `else' after goto.  Simplify control flow.

Don't `else' after goto.  Simplify control flow.

Don't `else' after goto.  Simplify control flow.

Added test/mhshow/test-binary.  The MIME parser doesn't
properly handle null bytes because it uses getline().

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.

Don't `else' after return.  Simplify control flow.