Added context_find_prefix().

Wrapped code block of conditional statement with {}, so that the
second statement is now included in the conditional.  Found by
gcc 6 -Wmisleading-indentation.

Moved free(cp) to right after last (and only) use of cp.  The statement
was flagged by gcc 6 -Wmisleading-indentation.  In its original location,
it would not always have been called, resulting in leaked memory.

Fixed indentation to prevent warning from gcc 6 -Wmisleading-indentation.

No longer need const cast in sendfrom.c.

Added consts to adios(), advise(), and advertise() to sync master
and nmh-oauth branch.

Added const to filename args of cpydata().  Then added const to some
args in mhfixmsg.c.

Added const to arg of m_backup().

Added const to first arg of adios().

If parse_mime() fails and the -file switch was used, pass the input
message to it unchanged so that pipelines won't break.

Added free_fs() call to plug leak from fmt_compile().

spelling-fixes ( code-comments in .c, .h. .y ... )

spelling-fixes ( docs, man, readme )

Added sendfrom to docs/contrib.

Added docdir to mhparam.

Removed unnecessary memset on fmt_mnull:  it gets initialized to
all 0's.

Check to see if cross-directory links are supported; if they are not,
then don't run the refile -link test.

For slocal, changed default user, if -user switch isn't used, to be
the user part of the Local-Mailbox profile entry, if present, or the
current user.  addr uses the same default.  This allows addr
to contain a '.', which slocal did not allow before.

Rename files to match renamed functions

Follow-up to commit 840b3f0e45a43b8378e88c66f004b2f186662e35

Added start_test() and finish_test() functions to test/common.sh.in.
See usage information in the (new) cleanup() function in that file.
This allow us to isolate test failures based on naming of individual
tests within a single test file.  As part of this, added set -e to
setup_test().  This required minor updates to test/new/basic and
test/pick/test-stderr to conform.  So, migrated test/new/test-basic
to start_test()/finish_test().

(cherry picked from commit cd6133dd23422f92534328f768a322f63a5c6dfe)

Fix to commit 84932d0a96c79bfb2f0384ebab806dc51a35f600 to
insert blank line between header and content for non-text
content.

Quoted variables in trap so that distcheck doesn't trip over them if
not set.

Moved "const" to after "register" to silence compile warning fron gcc.

Rename and document functions that read from stdin.

- gans becomes read_switch
- getans becomes read_switch_multiword
- getans_via_readline becomes read_switch_multiword_via_readline
- getanswer becomes read_yes_or_no_if_tty
- install-mh.c geta becomes read_line

Changed minchars swit values for -[no]concat switches to show(1)
from negative to 0, so that they can be abbreviated and so that they
appear with -help.  Added them to the man page, and fixed text that
was copied from mhshow(1) man page.

Use WIFXEXITED to check return status of system(), because
comparison with NOTOK wasn't valid.

Another pass at cleaning up (some of) the manpages.

This continues work I started a couple of years ago, but then got
derailed on.  There are two main goals behind these updates:

1) replace some low-level troff constructs with more modern
   man(7) facilities, and

2) adopt a more consistent visual style to the layout and
   formatting of the manpages, particularly in reference to
   typeset pages.

This time around I have updated README.manpages to describe the new
guidelines.  They are mostly non-intrusive, and serve to clean up
a lot of visual clutter that has crept in over the years.

Added check of return value of write() to quash compiler warning.

Recover properly if the character conversion fails.

Fix substitution code so it uses the right pointer for the utf-8 test.

Have fix_boundary() return NOTOK if it couldn't fix a boundary,
so that mhfixmsg won't modify the message.

Have -nocrlflinebreaks apply to the message parts that -decodetypes
designates.

Added EXAMPLES and BUGS sections.

Implement the -reverse option to pick(1) to change the search order.

Updated date on mh-folders(5) man page.

Added application/ics to default list if types to decode.

mh-folders.man: note that folders should not be named numerically

Added -[no]crlflinebreaks switches to mhfixmsg(1).

Added check for successful decode of a different part with
attempted -decodetext of binary text to test-mhfixmsg.

Don't consider failure to build a text/plain part to be an error.
This allows mhfixmsg to succeed even if there isn't a suitable
mhfixmsg-format-text/subtype profile entry for every text part in
the message.

Added -decodetypes switch to mhfixmsg(1).

Add missing -infile switch name to mhfixmsg-format-text/calendar
rule in mhn.defaults.

mhical enhancement:  If a datetime is missing the time, assume
midnight.  If there is no end datetime (DTEND) and the DTSTART
doesn't contain a time, assume that the event is for the entire day.

Set MH to the resolved, absolute pathname inside context_read() so
that exec'd children can find it.

Added all of mhshow's documented switches to show's handling, on the
notion that nmh's pieces should know about each other.

Added mention of mhfixmsg -fixcte to error message for improper C-T-E in
multipart message.  Thanks to Conrad Hughes for suggesting this.

Moved addition of -ansi -pedantic to CFLAGS with gcc from
configure.ac to docs/contrib/build_nmh, so that they're not
included by default with builds by users.

Allow a RRULE in an iCalendar request to not have an INTERVAL on its
yearly FREQ, because INTERVAL defaults to 1.

Remove any enclosing quotes from a timezone identifier in an
iCalendar event request.  See comment added to format_datetime()
that says that I don't believe that RFC 5545 allows them to be
quoted.  But Oliver Kiddle found them in the wild.

If format_datetime() returns null, store a copy of node->value on c->c_text
to prevent a double free().

If str == buffer, then do NOT do a strncpy(buffer, str).  Many systems
this works, but on some systems it causes a SIGABRT.  POSIX says the
behavior on overlapping copies with str*cpy() is officially undefined.

Add support for setting the environment variable MH_TEST_NOCLEANUP to
prevent a test from cleaning up the test working directory.

Add support for the -sendmail flag to send/post, to support changing the
sendmail executable.

Unquoted addresses in mhshow and scan output.  Suggested by
Anthony Bentley.

Change the reference to the valgrind suppression file from $MH_OBJ_DIR to
$srcdir so "make distcheck" works.

Include a new valgrind warning suppression for MacOS X.

Make sure the valgrind suppression file is honored, and add some extra
suppressions needed when running under the Debian fakeroot environment.

Fix error message to correctly report the program that failed to execute
when doing HTML conversion.
Also, remove perl function prototypes as they are not considered best
practice.

Rearrange all callers of post(8) so that the filename is always last.

mhparse.c: verify all MIME-Version: headers encountered

in addition to verifying, silently ignore duplicate version headers.

Merge remote-tracking branch 'origin/convertargs'

use %l uniformly on all the text-mode browser display strings

otherwise behavior changes depending on which browser was present
when nmh is configured/installed.

Filter converted text/html to remove blank lines and leading space
that lynx inserts.  Also, changed ISO-8859-1 to US-ASCII because it
is, so it doesn't need to be converted.

Removed m_chkids(), per suggestion of Marcin Cieslak.

Removed unnecessary #include of h/signals.h.

Removed #include of signal.h because it's in h/signals.h.

Reset done in inc_done() to prevent endless loop when setegid()
fails (in which case adios() is called, which calls done()).

That didn't work, so removed offset.

Use +0000 isntead of -0000 offset because FreeBSD displays it that way.

Replaced setgid() calls with setegid() so that it works with dot
locking on FreeBSD.  setegid() should be supported on modern POSIX
systems.

Added showmimeproc to test/common.sh.in so that test-show will test
it in the workspace, not the user's installation.  Also added all of
the other procs defined in config/config.c that are currently unused,
just in case they some day are.

Removed a few more msh (MS_MSH and mshproc) and vmh (vmhproc) relics.

Replaced individual assignment of array elements, one of which was
out of range, with array initialization.

Fixed the error message in lockit() when it fails to create tmp file.

Fixed "inc" to be installed setgid mail with dot locking.  MAILGROUP=1
was not set in the config.h because we no longer use "LOCKTYPE"
configure variable.

update tests for new message separator header from mhshow

also added test names to tests, in any file i touched

update man pages for new mhshow message separator header

add new message separator header to mhshow

new switches to mhshow (-headerform and -[no]header) control whether
and what is displayed before the display of every message's content.
the default separator looks like "[ Message folder:msg ]".  the
implementation parallels the -markform feature.

since mhshow is often started by show, show now passes the -[no]header
options through to mhshow.  (also, show no longer attempts to pass
such options through to cat (in the case of -noshowproc)).

fix typo in mh-format.man

Test that show always shows message numbers.

show shows message numbers in two different formats when showing one
non-MIME, multiple non-MIME messages, or one MIME message.

show does not show any message numbers when showing multiple messages
any of which is MIME.

This is not a new bug, just one that has bothered me since I started
using mhshow with 1.6.

test/common.sh: allow giving individual tests a name

man pages -- clarify what -part and -type do when used together

Merge remote-tracking branch 'origin' into convertargs

Added support to mhfixmsg -reformat for multipart/related.  If it has
only a single part that is not text/plain and can be converted to
text/plain, a text/plain part is added and the type of the part is
changed to multipart/alternative.  If the multipart/related has more
than one part but does not have a text/plain part, try to add one.

Fixed test-convert to work on Cygwin and other platforms that
default to UTF-8 locale.

Merge remote-tracking branch 'origin' into convertargs

Removed unused local variable from move_preferred_part().

pending-release-notes: add mhshow's "-prefer", and mh-format's %(kibi/kilo)

test-subpart: add mhlist and mhshow tests for -type/-part/-prefer

update mhshow, mhlist, and mhstore man pages for -prefer switch

implement -prefer, for multipart/alternative preferences

mhshow.man: more detail on use of -part and -type switches

mhshow: ensure that -part always shows every part requested

previously, "mhshow -part 1 -part 1.3" would show 1.1 but not 1.3.

mhshow: fix display of alternatives after certain nested multiparts

adjust return codes so that we now have a clear indication of whether
there was a hard error (NOTOK), a failure to display due to type
mismatch (DONE), or complete success (OK).  use those codes in
show_multi_internal() to decide whether we're finished with this
multipart or not.

prior to this change, it was impossible to show part 1.2 of a
message constructed like the following, because the disinction
between (the new, as described above) DONE and OK states was
lost while processing multipart/related part 1.1.
 msg part  type/subtype              size description
  28       multipart/mixed           1040
     1     multipart/alternative      623
     1.1   multipart/related          275
     1.1.1 text/html                  105
     1.2   text/plain                  33
     2     text/plain                  83

this fix is related to 9cc2510 and c9794733.  hopefully it's right
this time.

mhshow: simplify code -- the DONE status was adding no value

drop the second argument to part_ok(), since it is always 1

the second argument hasn't been used since "historical" times.
also add some commentary to part_ok().

Add a note about new bindir option to mhparam.

'mhparam bindir' now prints NMH's idea of $(bindir).

Added a line break to mhbuild man page because the
%docdir%/contrib/replaliases line caused format complaints from groff.

Merge remote-tracking branch 'origin' into convertargs