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

(cherry picked from commit 9626eb33ed5ff7c59776081e1a58a20a1eed96be)

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.

(cherry picked from commit 79f012b76eba04d50a36a4a37c57458a8d521434)

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

(cherry picked from commit a814c930041c1e238f1826b1ac9c46b67e4cfb70)

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.

Merge remote-tracking branch 'origin' into xoauth

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

(cherry picked from commit 9f7d48c36f01ba29974834cbcdc24e4c2e1f07d9)

Added const to arg of m_backup().

(cherry picked from commit d42ab414c8bce17b6b437ef9ebfd194a8ac4111c)

Synched common.sh.in from master to nmh-oauth.

No longer need const cast in sendfrom.c.

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

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.

(cherry picked from commit 564b5b785c7cc709f92a83afb547267ed7418f18)

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

Plugged leak in mh_oauth_cred_load().

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

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

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

(cherry picked from commit 942561da2455ee50a1eb82c8c4f2bdb5e25133d1)

spelling-fixes ( docs, man, readme )

(cherry picked from commit fa92642a21119eda8bfc961f8d5a8d3e9ee7d494)

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

spelling-fixes ( docs, man, readme )

Added sendfrom to docs/contrib.

Added sendfrom to docs/contrib.

Added docdir to mhparam.

(cherry picked from commit 89d69fbd2ae1f9c8c989850ec7057814c655a727)

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

(cherry picked from commit 56c4ac49d7b8ad674c0a66e9999186bcb98bd048)

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

(cherry picked from commit 35fce21b692a6259138fc81caf13f4d07bb58da0)

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.

Added test of negative return code from jsmn_parse() to test-send.

Added test of empty authorization code to test-mhlogin.

Added explicit OATH_SUPPORT=1 to configure.ac when oauth support is
enabled, otherwise it didn't get set to 1.

Fix tests with oauth disabled.

Check for empty code.

Return false from parse_json() if jmsn_parse() returns a negative
(or zero, as it did before) value.

Improve oauth autoconf support.

- By default, enable OAuth if curl is found.

- If OAuth requested (--with-oauth), error if curl not found.

- If OAuth disabled (--without-oauth), don't enable it even if found.

- Sanity-check curl-config --version output so we don't send anything crazy in
  our HTTP requests if curl-config changes.

Rename files to match renamed functions

Follow-up to commit 840b3f0e45a43b8378e88c66f004b2f186662e35

Fix mhlogin(1) title.

Added -user to mhlogin man page.

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.

(cherry picked from commit fe36a7ee2f4304fd3797443d4f2f950d6dc2475d)

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.

check curl_ctx->res_len > 0 not res_body != NULL

res_body is not a pointer and is never NULL

(Thanks clang!)

oops, pass int to m_getfld not size_t

support multiple oauth users; mhlogin now requires -user

plug some memory leaks, in success cases only

(staying clean at least on non-adios paths is not hard)

fix mhlogin -browser test

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

test mhlogin -browser

Don't call sm_auth_sasl() for XAUTH2 with -sasl.  Instead,
call only sm_auth_xoauth2().

Added start_test()/finish_test() to README.developers.

Migrated the oauth tests to start_test()/finish_test() so that
they don't have any output on success, but do report the particular
test on failure.

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

Added -browser switch to mhlogin(1).

Merge remote-tracking branch 'origin' into xoauth

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.

Changed sign of minchars for the SASL-related switches from
negative to positive so the won't be hidden from help messages.
I think.

Merge remote-tracking branch 'origin' into xoauth

 merge is necessary,

Replaced -oauth service switch with -saslmech xoauth2
-authservice service, because OAUTH2 is a SASL mechanism.

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.

Merge remote-tracking branch 'origin' into xoauth

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

Added oauth support to whatnow.

Added oauth switch to whatnow.

Added use of local variable user to prevent compiler warning
when built without OAuth2 support.

Sort JSON output from oauth tests so that comparisons don't
depend on it being in a specific order.  Also, clean up test
tmp files.

Set MH_VERSION, OAUTH_SUPPORT, and CURL_USER_AGENT in tests/common.sh.in
so that oauth tests can be run individually (not just via make check).

Added OAuth2 support to build_nmh.

Merge branch 'master' into xoauth

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.