merge: upstream fixes

 * Increase buffer sizes for some IMAP commands.
 * Change imap_rename_mailbox to use a BUFFER.
 * Convert pgp_invoke_list_keys and mix_send_message to use BUFFERs.

Convert pgp_invoke_list_keys and mix_send_message to use BUFFERs.

Both repetitively perform a lot of copying back in forth, which is
much cleaner with a BUFFER.

Note that in pgp_invoke_list_keys, if there are no hints uids->data
would be NULL.  However, the pgp_invoke() checks and wraps all the
format substitutions with NONULL.

Change imap_rename_mailbox to use a BUFFER.

Increase buffer sizes for some IMAP commands.

Use to ensure assembled IMAP commands fit in the resultant buffer to
be imap_exec()'ed.

RFC2683 suggested a limit of 1000, but asked servers to accept up to
8000.  Furthermore, RFC7162 bumped the client limit suggestion up to
8192.  So I don't believe any issues will be caused by this.

Most of these are increases are just to remove theoretical truncation
warned about by the compiler; I don't believe we've had reports of
actual command truncation due to a mailbox 1024 characters long.

Fix non-threaded $sort_aux "reverse-" sorting.

The secondary sort was looking at (Sort & SORT_REVERSE) instead
of (SortAux & SORT_REVERSE), so wasn't even performing the reverse
based on the correct flag.

Additionally, afterwards, the primary sort was improperly applying a
reverse when the secondary sort returned non-zero.

Change SORTCODE() to look at SortAux when we are inside a secondary
sort.

Change AUXSORT() to return the result if the secondary sort returns
non-zero.  It is ugly to put a return inside the macro, but the check
for non-zero needs to be performed inside the AUXSORT if branch.

If the secondary sort returns 0, then the primary sort can still
compare index and apply a reverse as needed.

merge: minor tidying

 * boolify: account_match
 * rename header_cache_t vars
 * move file_get_size
 * docs: fix example
 * doxy: comment remaining parse functions
 * doxy: comment remaining connection functions

doxy: comment remaining connection functions

doxy: comment remaining parse functions

docs: fix example

move file_get_size

rename header_cache_t vars

boolify: account_match

merge: minor tidying

 * sync mutt_parse_mailboxes to other parse functions
 * refactor mutt_account_match
 * refactor mutt_conn_new()
 * mx_path_probe: export stat info
 * tidy code
 * add comments

add comments

tidy code

mx_path_probe: export stat info

refactor mutt_conn_new()

refactor mutt_account_match

sync mutt_parse_mailboxes to other parse functions

merge: tidy menu api functions

 * rename functions
 * document api functions
 * rename functions
 * unify api function params

menu: unify api function params

menu: rename functions

menu: document api functions

menu: rename functions

merge: upstream fixes

 * Change imap_conn_find() to always return an authenticated conn.
 * Free queries and alias_queries before exiting.
 * Remove purpose checks in smime_keys.pl verify step.
 * Send imap keepalives for interactive filters.
 * Add output during smime_keys purpose flag checking.
 * Increase user/pass field sizes in auth_login.

Increase user/pass field sizes in auth_login.

account.pass was previously increased to 256 in cd421c13.  Also, the
buf should be bigger than the user+pass+"LOGIN", so bump it up too.

Add output during smime_keys purpose flag checking.

Send imap keepalives for interactive filters.

When viewing attachments externally with a (non-copiousoutput) mailcap
entry missing %s, the command is invoked as a filter, with the
attachment piped into stdin.  However, unlike a filter, the user
interacts with the command, instead of just displaying the output in
the pager.

Just as with the mutt_system() command, Mutt needs to send imap
keepalives to keep those connections from closing during the
potentially extended invocation.

Thanks to John Hawkinson for the bug report, and his suggested patch,
which this commit is based upon.

Remove purpose checks in smime_keys.pl verify step.

The purpose checks are subsequently performed, and added as a field to
the .index.  In any case, it turns out passing multiple '-purpose'
arguments isn't even correct: openssl appears to just use the last
one.

Thanks to David J. Weller-Fahy for reporting the problem, and for
including a possible patch.

Free queries and alias_queries before exiting.

Change imap_conn_find() to always return an authenticated conn.

With the flag MUTT_IMAP_CONN_NONEW, it was already ensured the
connection would be authenticated.  However, without that flag, an
error in opening the connection or authentication would still return
an idata.

The callers that didn't bother to check the state were still assuming
authenticated, because they were all subseqeuently issuing an
"authenticated state" command to the server.

Rather than add state checks to every caller, just change the function
to return NULL if the idata did not end up in an authenticated state.

Remove the now redundant state checks in imap_open_mailbox() and
imap_get_mailbox().

refactor/simplify status %f

boolify ImapMboxData data members

fix naming of ImapMboxData alloc/free functions

docs: fix 'hidden_tags' references

Fixes #1347

[ci skip]

Fix segfault when checking a mailbox after a disconnect (II)

Fix segfault when checking a mailbox after a disconnect

restore Mailbox order

rename socket-private data structs

reorg socket functions

add const to mutt_browser_select_dir

sidebar: drop unnec fn: mutt_sb_set_mailbox_stats

This used to copy stats from the Context to the Mailbox.
Now, the Context points to the Mailbox.

drop debugging code

merge: tidy private data usage

 * rename Account to ConnAccount
 * add config Account
 * tidy notmuch
 * reorg notmuch functions
 * reorg pop functions
 * tidy pop
 * reorg imap functions
 * reorg maildir functions
 * reorg mbox functions
 * reorg nntp functions
 * tidy maildir
 * rename maildir_flags()
 * rename email include
 * rename Header to Email
 * Email doesn't begin with 'h'
 * notmuch: tidy data naming
 * notmuch: distinguish between Mailbox and Email data
 * notmuch: fix header references
 * nntp: tidy data naming
 * pop: tidy data naming
 * pop: separate email data
 * imap: tidy data naming
 * imap: more data tidying
 * change free functions to use void**

change free functions to use void**

imap: more data tidying

imap: tidy data naming

pop: separate email data

pop: tidy data naming

nntp: tidy data naming

notmuch: fix header references

notmuch: distinguish between Mailbox and Email data

notmuch: tidy data naming

Email doesn't begin with 'h'

rename Header to Email

rename email include

rename maildir_flags()

tidy maildir

reorg nntp functions

reorg mbox functions

reorg maildir functions

reorg imap functions

tidy pop

reorg pop functions

reorg notmuch functions

tidy notmuch

add config Account

rename Account to ConnAccount

maildir: alloc mem for headers

Fixes #1337

Fix segfault introduced by 5e8cb63

boolify

- be_edit_header
- boolify mutt_file_lock
  The parameters were only being used as bools.
- boolify maildir_delayed_parsing

bug: check for aborted dir read

doxy: fix links

trivial fixes

Fix build

Fix check on empty strings

merge: redistribute Context and Mailbox

 * fix mxapi comments
 * move Context.mx_ops
 * move Context.ctx->data
 * move Context.{hdrs,hdrmax}
 * move Context.{size,changed,readonly,quiet,closing}
 * free_data
 * notmuch: move mailbox data
 * notmuch: let mxapi do cleanup
 * notmuch: use mailbox, not context
 * mbox: use Mailbox, not Context
 * maildir: use Mailbox, not Context
 * nntp: use Mailbox, not Context
 * pop: use Mailbox, not Context
 * compress: use Mailbox, not Context
 * imap: use Mailbox, not Context
 * add mbox private Mailbox data
 * move Context.rights
 * eliminate Context from backends
 * Mailbox doesn't begin with 'b'
 * move Context.compress
 * tidy compress
 * move Context.{id,subj,label}_hash
 * move Context.{v2r,vcount}
 * move Context.mtime
 * tidy context usage
 * imap: rename mailbox out of the way

imap: rename mailbox out of the way

tidy context usage

move Context.mtime

move Context.{v2r,vcount}

move Context.{id,subj,label}_hash

tidy compress

move Context.compress

Mailbox doesn't begin with 'b'

eliminate Context from backends

move Context.rights

add mbox private Mailbox data

imap: use Mailbox, not Context

compress: use Mailbox, not Context

pop: use Mailbox, not Context

nntp: use Mailbox, not Context

maildir: use Mailbox, not Context

mbox: use Mailbox, not Context

notmuch: use mailbox, not context

notmuch: let mxapi do cleanup

notmuch: move mailbox data

free_data

move Context.{size,changed,readonly,quiet,closing}

move Context.{hdrs,hdrmax}