tests: minor fixes

Fix:
- Misleading output
- printf specifiers
- Missing test cases

config: allow deeper inheritance

The code assumed that an inherited HashElem had a normal HashElem
parent.  This change removes the assumption, allowing deep nesting of
Config Items.

merge: trivial fixes

 * fix folder use in mx_path_canon()
 * doxy: fix typos
 * rename local variable for clarity
 * test: tidy code
 * refactor #ifdef'd conditional
 * check includes with iwyu

check includes with iwyu

refactor #ifdef'd conditional

test: tidy code

- clang-format
- remove notes

rename local variable for clarity

doxy: fix typos

fix folder use in mx_path_canon()

Use the parameter Context, not the global.

merge: upstream fixes

 * Turn on $ssl_force_tls by default
 * Change mutt_addr_is_user() to no for a NULL address
 * Minor documentation correction
 * Fix dropped new mail notifications when an EXPUNGE_PENDING is set
 * Improve robustness of imap_append_message()
 * Enable the idata->check_status using bit operations

Enable the idata->check_status using bit operations

Commit e3f66d7e fixed dropped new mail notications, removing the
unsetting of idata->reopen IMAP_NEWMAIL_PENDING in imap_cmd_finish()
when an EXPUNGE was processed.

However, imap_cmd_finish() can be called twice by
imap_check_mailbox().  First as part of the imap_exec(), and manually
again just below.

Now that the IMAP_NEWMAIL_PENDING still exists, a second call could
overwrite idata->check_status if both reopen flags were set.

This unfortunately affects update_index(), which behaves differently
for MUTT_REOPENED.

I need to change the return value of mx_check_mailbox() in master to
preserve all the bits, so the index can both notify of new mail and
update_index() properly.

For stable, the best fix is to use bit operators to enable the
check_status flags in imap_cmd_finish() (and cmd_parse_fetch for
flags), and keep the imap_check_mailbox() priority of setting its
return value (it prioritizes IMAP_EXCHANGE_PENDING).

Co-authored-by: Richard Russon <rich@flatcap.org>

Improve robustness of imap_append_message()

First, check the imap_cmd_step() return value instead of looking at
idata->buf for "OK".  If the connection bombed and imap_cmd_step()
returned IMAP_CMD_BAD, the value of idata->buf is stale.

If the server returned "+ OK" for the command continuation request
response, the call to imap_code(idata->buf) would even end up
returning true, despite that the append failed!  (See #110, although
at the time of commit I can only hypothesize this is what is
happening.)

Second, check the status of the writes.  flush_buffer() was not
passing the rc from mutt_socket_write_n(), which was further making
the above disaster scenerio possible.

Co-authored-by: Richard Russon <rich@flatcap.org>

Fix dropped new mail notifications when an EXPUNGE_PENDING is set

Prior to the fetch_headers rework and introduction of
idata->max_msn (starting around e0376c75), cmd_handle_untagged() was
looking directly at ctx->msgcount, which isn't fixed up until
imap_expunge_mailbox().  At that time, more care had to be taken
inbetween handling the EXPUNGE message and the actual expunge of the
mailbox because of the discrepency between server state and mailbox
context state.

idata->max_msn is now decremented during the processing of EXPUNGE and
VANISHED notices from the server, so reflect "current" state.  So,
when we receive an EXISTS notice, we no longer need the checks for
expunge state and can always set the NEWMAIL_PENDING flag.

Additionally, fix imap_cmd_finish() to retain the IMAP_NEWMAIL_PENDING
flag after handling an expunge.  The expunge does not grab new
messages so dropping the flag would cause mutt to forget the new mail
status until another EXISTS command.

Since this is a stable branch fix, I'm leaving the either/or
processing of expunge versus new mail in imap_cmd_finish().  However,
I don't see why this has to be done in two calls.  I may rework that
in master to process an expunge and then the new mail one after the
other.

Co-authored-by: Richard Russon <rich@flatcap.org>

Minor documentation correction

Co-authored-by: Richard Russon <rich@flatcap.org>

Change mutt_addr_is_user() to no for a NULL address

The behavior is generating a false 'F' flag for a spam message with an
unparsable From address.

After reviewing the callers, I can't see a reason returning 'yes' in
this case.

Co-authored-by: Richard Russon <rich@flatcap.org>

Turn on $ssl_force_tls by default

Ticket #135 suggests that these days, it's better to force encryption
over all connections.  RFC8314 is recommending MUA's move in that
direction (actually even directing towards implicit TLS over
STARTTLS).

I'm enabling this at the beginning of the 1.13 development cycle to
give others time to chime in with any objections.  Personally, I've
had this option set myself for years.  The only place it could become
an issue is for a localhost IMAP server with no cert.  In that case,
it's easy enough to have an account hook unset if needed, and I think
a better idea that the user be forced to turn it off.

Co-authored-by: Richard Russon <rich@flatcap.org>

Save attachments without prompting.

If the 'attach_save_without_prompting' variable is set (to true), then on
saving a list of tagged attachments the user will not be prompted each time
for the name of file.

Conflicting filenames during save are also handled.

-=david=-

closes #1603

Update to latest autosetup, use @include feature

Up-to-date with Autosetup 4701f3e (2019-06-05)

Fix segfault on bounce

Fixes #1735

add missing translation file

show backtrace on segfault

Add a configure option `--backtrace` that uses libunwind to print a
basic backtrace if NeoMutt segfaults.

Co-authored-by: Pietro Cerutti <gahr@gahr.ch>

add funding details

merge Config and normal Accounts

merge: add basic notifications

 * debug: add notify level
 * add central NeoMutt object
 * add notify/observer
 * config notify
 * test: fix config tests

test: fix config tests

config notify

add notify/observer

add central NeoMutt object

debug: add notify level

merge: mime: use description if filename is missing

 * mime_lookup: use description if filename isn't useful
 * mime_lookup: use compose mailcap entries when replying

mime_lookup: use compose mailcap entries when replying

Use compose mailcap entries to format attachments when replying to a message

mime_lookup: use description if filename isn't useful

"fix" mime_lookup to also look into Description mime tags if the
filename tag doesn't come up with something useful

test: fix config_dump

merge: small fixes and tidying

 * clang-format
 * use enum return value
 * test: fix lcov out-of-tree build
 * test: boolify test_mutt_pattern_comp()
 * refactor: replace ctime() with strftime()
 * bug: prevent unterminated string
 * check range of uuencoded char
 * log failed atoi() calls
 * fix buffer handling
 * restore randomness to mutt_buffer_mktemp_full()
 * check state buffer
 * check for overflow in mutt_edit_content_type()
 * reduce scope of variables
 * add/remove checks
 * use param, not global, in prereq()
 * add brackets to clarify logic
 * misc small changes

misc small changes

- Initialise some variables
- Fix some formatting strings
- Remove unused variables

add brackets to clarify logic

use param, not global, in prereq()

The test function prereq() wasn't using the Context parameter, but the
Context global.

add/remove checks

Some functions needed checks.
Some others had redundant checks.

reduce scope of variables

check for overflow in mutt_edit_content_type()

Thanks to lgtm.com

check state buffer

Coverity: CID 186614

restore randomness to mutt_buffer_mktemp_full()

Coverity: CID 189775

fix buffer handling

Coverity: CID 197580, 198376, 198378, 198379

log failed atoi() calls

Coverity: CID 198492, 198493

check range of uuencoded char

Coverity: CID 180829

bug: prevent unterminated string

Coverity: CID 83483

refactor: replace ctime() with strftime()

ctime() is marked obsolete in POSIX:2008.
It's recommended to use strftime() instead.

Thanks to lgtm.com

test: boolify test_mutt_pattern_comp()

test: fix lcov out-of-tree build

use enum return value

- mutt_check_mime_type()
- mutt_lookup_mime_type()

clang-format

travis: workaround locale problems

merge: improve date tests

 * date: move defines, so they can be used for testing
 * test: improve test_mutt_date_add_timeout()
 * test: improve test_mutt_date_check_month()
 * test: improve test_mutt_date_gmtime()
 * test: improve test_mutt_date_is_day_name()
 * test: improve test_mutt_date_local_tz()
 * test: improve test_mutt_date_localtime()
 * test: improve test_mutt_date_localtime_format()
 * test: improve test_mutt_date_make_date()
 * test: improve test_mutt_date_make_imap()
 * test: improve test_mutt_date_make_time()
 * test: improve test_mutt_date_make_tls()
 * test: improve test_mutt_date_normalize_time()
 * test: improve test_mutt_date_parse_date()
 * test: improve test_mutt_date_parse_imap()

test: improve test_mutt_date_parse_imap()

test: improve test_mutt_date_parse_date()

test: improve test_mutt_date_normalize_time()

test: improve test_mutt_date_make_tls()

test: improve test_mutt_date_make_time()

test: improve test_mutt_date_make_imap()

test: improve test_mutt_date_make_date()

test: improve test_mutt_date_localtime_format()

test: improve test_mutt_date_localtime()

test: improve test_mutt_date_local_tz()

test: improve test_mutt_date_is_day_name()

test: improve test_mutt_date_gmtime()

test: improve test_mutt_date_check_month()

test: improve test_mutt_date_add_timeout()

date: move defines, so they can be used for testing

merge: improve list tests

 * test: ignore mutt_exit()
 * test: add shared list testing code
 * test: improve test_mutt_list_clear()
 * test: improve test_mutt_list_compare()
 * test: improve test_mutt_list_find()
 * test: improve test_mutt_list_free()
 * test: improve test_mutt_list_free_type()
 * test: improve test_mutt_list_insert_after()
 * test: improve test_mutt_list_insert_head()
 * test: improve test_mutt_list_insert_tail()
 * test: improve test_mutt_list_match()

test: improve test_mutt_list_match()

test: improve test_mutt_list_insert_tail()

test: improve test_mutt_list_insert_head()

test: improve test_mutt_list_insert_after()

test: improve test_mutt_list_free_type()

test: improve test_mutt_list_free()

test: improve test_mutt_list_find()

test: improve test_mutt_list_compare()

test: improve test_mutt_list_clear()

test: add shared list testing code

test: ignore mutt_exit()

doc: add example to find all maildir mailboxes

test: fix test failures

Fix list-reply regression introduced when converting Address to TAILQ

Fixes #1724

make OP_HALF_{UP,DOWN} behave symmetric.

  - the calculation of number of lines to scroll up/down should not
  depend on whether the number of rows in the pager/index/terminal is
  odd or even.
  - this patch will make the behaviour symmetric such that in both cases
  (even/odd number of rows) scrolling up and down by half a page (or the
  other way round) will get you back to the exact same line as before.

merge: improve string tests

 * test: improve test_mutt_str_adjust()
 * test: improve test_mutt_str_append_item()
 * test: improve test_mutt_str_asprintf()
 * test: improve test_mutt_str_dequote_comment()
 * test: improve test_mutt_str_find_word()
 * test: improve test_mutt_str_getenv()
 * test: improve test_mutt_str_inline_replace()
 * test: improve test_mutt_str_is_ascii()
 * test: improve test_mutt_str_is_email_wsp()
 * test: improve test_mutt_str_lws_len()
 * test: improve test_mutt_str_lws_rlen()
 * test: improve test_mutt_str_next_word()
 * test: improve test_mutt_str_pretty_size()
 * test: improve test_mutt_str_remall_strcasestr()
 * test: improve test_mutt_str_remove_trailing_ws()
 * test: improve test_mutt_str_replace()
 * test: improve test_mutt_str_rstrnstr()
 * test: improve test_mutt_str_skip_email_wsp()
 * test: improve test_mutt_str_skip_whitespace()
 * test: improve test_mutt_str_split()
 * test: improve test_mutt_str_startswith()
 * test: improve test_mutt_str_strcasecmp()
 * test: improve test_mutt_str_strcasestr()
 * test: improve test_mutt_str_strcat()
 * test: improve test_mutt_str_strchrnul()
 * test: improve test_mutt_str_strcmp()
 * test: improve test_mutt_str_strcoll()
 * test: improve test_mutt_str_strdup()
 * test: improve test_mutt_str_strfcpy()
 * test: improve test_mutt_str_stristr()
 * test: improve test_mutt_str_strlen()
 * test: improve test_mutt_str_strlower()
 * test: improve test_mutt_str_strncasecmp()
 * test: improve test_mutt_str_strncat()
 * test: improve test_mutt_str_strncmp()
 * test: improve test_mutt_str_strnfcpy()
 * test: improve test_mutt_str_substr_cpy()
 * test: improve test_mutt_str_substr_dup()
 * test: improve test_mutt_str_word_casecmp()

test: improve test_mutt_str_word_casecmp()

test: improve test_mutt_str_substr_dup()

test: improve test_mutt_str_substr_cpy()

test: improve test_mutt_str_strnfcpy()

test: improve test_mutt_str_strncmp()

test: improve test_mutt_str_strncat()

test: improve test_mutt_str_strncasecmp()

test: improve test_mutt_str_strlower()

test: improve test_mutt_str_strlen()

test: improve test_mutt_str_stristr()

test: improve test_mutt_str_strfcpy()

test: improve test_mutt_str_strdup()

test: improve test_mutt_str_strcoll()

test: improve test_mutt_str_strcmp()

test: improve test_mutt_str_strchrnul()