Use HasPrefix() instead of strncmp().

Alter HasSuffixC()'s char * to be const.

utils.c: Add HasPrefix(s, prefix).

HasPrefix returns true if non-NULL s starts with non-NULL prefix.

Rename EndsWithC() to HasSuffixC().

mh-folders.man: Remove empty `.I' immediately followed by `.IR'.

build_nmh: If cloning with git, make a shallow clone.

showbuildenv: Drop lsb_release(1);  /etc/lsb_release tested earlier.

build_nmh: Use `gzip -d' rather than assume tar has -z.

build_nmh: Have a silent curl still show errors.

Arguably, -sS should be curl's defaults!

Fixed output of commit hash by git log command.

Check for existence of tools/showbuildenv.

Redirect verbose build_nmh messages so they're visible while running.

Enhanced build_nmh to support standalone use.

1) If not in nmh directory, download the nmh sources from savannah
   using git clone, wget, or curl.
2) Added -b branch option, for use when downloading.
3) Support -l -.
4) Capture build environment with tools/showbuildenv.

man: Use lowercase for command name at start of sentence.

`Inc' is a different command to `inc'.

uprf(): Rewrite, using types rather than masking ints.

Add comment describing uprf().

Use ToLower() instead of loop.

Use ToUpper() instead of loop.

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.