Fix AUXSORT usage in compare_spam

AUXSORT already checks if the first parameter is zero.

Additionally, the macro expands into more than one statement, so
putting inside an unbracketed if is misleading and generates a
warning on the latest gcc.

Fortunately, both statements checked if the first parameter was zero,
so no actual bug resulted.

Add message padding to ctx->vsize computation

Use the mx_msg_padding_size() from the previous commit to be more
precise about the correct vsize.

This avoids strange situations where all the message are displayed in
a limit, but the vsize is not equal to the size.

Add mx_ops.msg_padding_size to return the padding for a mx type

Mbox pads with a 1 byte, while mmdf pads with 10.  Because compress
depends on the child type, we create a mx_ops, which allows compress.c
to delegate to the child ops.

Skip sort in mbox_sync_mailbox() when new/reopen occurs

Callers of mx_sync_mailbox() and mx_close_mailbox() already check for
those cases and call update_index().  So remove the need_sort flag
setting when mbox_check_mailbox() returns new/reopen inside
mbox_sync_mailbox().

Handle reopen/new_mail when closing mailbox in <imap-logout-all>

Other callers, such as OP_QUIT, OP_MAIN_CHANGE_FOLDER, and
OP_MAIN_SYNC_FOLDER handle the case where a sync or close_mailbox
fails due to new mail or a reopen.

OP_MAIN_LOGOUT_ALL appears to have been accidentally missed.  Add a
call to update_index() to properly sort and update vcounts.

Add ctx->vsize = 0 in a couple of places

It is not clear whether this is needed, but in both cases, the old value
(if not 0) is obsolete. If there is a bug somewhere else about vsize, it
will be easier to notice it.

Improved limit-to-all detection by skipping leading spaces of the pattern

There are many equivalent limit-to-all patterns, but in practice, one
may want to limit to " ~A" in order to prevent the pattern from being
recorded in the history. Thus it is important to detect at least this
pattern and similar ones.

Bug fix: reset ctx->vsize in update_index_unthreaded on reopened mailbox

This fixes a bug observed when doing the following:
1. Start Mutt on some mailbox.
2. Limit the view.
3. Set sort by date (thus unthreaded).
4. Set $status_format to display the size of the messages shown (%L).
5. Copy a message matching the limit pattern to this mailbox.
6. With another Mutt instance, remove this message from this mailbox.
7. Move the cursor to update the view.
Result: one was getting a value equal to twice the initial size.

Display matching new messages in a thread-limited index

Previously, the index performed pattern matching first, and then
resorted new mail.  The problem was that thread-limiting patterns,
e.g. ~(pattern), require threading data to properly match against the
new messages.

We already save new messages for the purposes of uncollapsing threads.
To keep the code cleaner, split off update_index() into
update_index_threaded()/unthreaded().  Then for threaded mode, save
the new messages first.  We can then sort (before pattern matching),
and use the save_new array to pattern match the new messages
afterwards.

The $uncollapse_new loop was unnecessarily performing a n^2 search.
Simplify to just iteratate over the save_new instead.

Minor documentation fix

Mention $pgp_decode_command for $pgp_check_gpg_decrypt_status_fd

It scans $pgp_decode_command for inline and application/pgp mime
types.

smime_handle_entity() cleanup

Remove redundant NULL assignments after safe_fclose() calls.

Remove some trailing spaces.

smimeout can't be NULL, so remove the check around the output
translation block.

Void passphrase on s/mime decryption error

Model this after pgp's pgp_decrypt_part(), which checks if fpout is
empty.

crypt-gpgme: prevent crash on bad S/MIME signature

Inform the user about the fingerprint being unavailable instead
of crashing if the S/MIME signature is bad.

clang-format

add lots of consts

merge: minor fixes

 * warn when logging at the highest level
 * boolify locals of mutt_select_file()
 * buf,buflen for menu_make_entry()
 * fix comment for nntp_expand_path()
 * doxygen: mailbox backends
 * doc: add maildir note for new mail feature
 * doxy: fix imap config descriptions

doxy: fix imap config descriptions

doc: add maildir note for new mail feature

doxygen: mailbox backends

fix comment for nntp_expand_path()

buf,buflen for menu_make_entry()

boolify locals of mutt_select_file()

warn when logging at the highest level

release build fixes

merge: simplify mx_mbox_open, mx_mbox_close

 * don't use mx_open's Context parameter
 * drop mx_open's Context parameter
 * free on mx_mbox_close

free on mx_mbox_close

drop mx_open's Context parameter

don't use mx_open's Context parameter

very minor fixes

- put listener prototype in the right header
- update list of message files
- fix comments

fix config error reporting

clean up on exit

Attempt to fix the code path to create a new mbox

Fix IMAP probe

merge: add path functions to mailbox API

 * add name and magic to MxOps
 * add MxOps:path_probe()
 * improve mutt_path_canon
 * stubs for mx_path_{canon,pretty}
 * add mutt_str_inline_replace()
 * delegate mx_path_canon()
 * factor Folder out of imap_pretty_mailbox
 * add mutt_path_abbr_folder
 * fill out mx_path_pretty()
 * improve mutt_path_parent
 * add MxOps:path_parent()
 * tidy mx
 * use mx_path_probe()
 * probe notmuch
 * probe nntp
 * probe pop
 * probe imap
 * probe maildir

probe maildir

probe imap

probe pop

probe nntp

probe notmuch

use mx_path_probe()

tidy mx

add MxOps:path_parent()

improve mutt_path_parent

fill out mx_path_pretty()

add mutt_path_abbr_folder

factor Folder out of imap_pretty_mailbox

delegate mx_path_canon()

add mutt_str_inline_replace()

stubs for mx_path_{canon,pretty}

Chase a coverity wraning by using the correct STAILQ macro

________________________________________________________________________________________________________
*** CID 187781:  Control flow issues  (DEADCODE)
/mutt/regex.c: 142 in mutt_regexlist_add()
136         mutt_buffer_printf(err, "Bad regex: %s\n", str);
137         return -1;
138       }
139
140       /* check to make sure the item is not already on this rl */
141       struct RegexListNode *np = NULL;
>>>     CID 187781:  Control flow issues  (DEADCODE)
>>>     Execution cannot reach the expression "np" inside this statement: "np = (np ? np : rl->stqh_fi...".
142       STAILQ_FOREACH_FROM(np, rl, entries)
143       {
144         if (mutt_str_strcasecmp(rx->pattern, np->regex->pattern) ==
0)
145           break; /* already on the rl */
146       }
147

improve mutt_path_canon

- Handle relative paths
- Lots of logging

add MxOps:path_probe()

add name and magic to MxOps

factor out struct MyVar

drop unused functions

fix Group's use of RegexList

merge: simple code refactoring

 * kill unnecessary != NULL
 * kill unnecessary == NULL
 * drop header-sized ifdefs
 * unify buf,buflen
 * unify buf,buflen, with reordering
 * kill assign-assign

kill assign-assign

unify buf,buflen, with reordering

unify buf,buflen

drop header-sized ifdefs

kill unnecessary == NULL

kill unnecessary != NULL

merge: conversions to use STAILQ

 * Convert RegexList to use STAILQ
 * Refactor RegexList changes
 * Convert ReplaceList to use STAILQ

Convert ReplaceList to use STAILQ

Refactor RegexList changes

Convert RegexList to use STAILQ

merge: tidy docs

 * doxy: add missing docs
 * drop unnecessary helper function
 * refactor parse functions
 * doxy: add missing comments

doxy: add missing comments

refactor parse functions

drop unnecessary helper function

doxy: add missing docs

config: don't reused constants

Fixes #1316

prevent buffer underrun in mutt_path_tidy_slash

Fixes #1315

fix leak in rfc2047_decode

merge: trivial fixes

 * tidy comments
 * buf,buflen
 * boolify params
 * use mailbox constant
 * pgpring is dead
 * doc: explain (un)virtual-mailboxes

doc: explain (un)virtual-mailboxes

pgpring is dead

use mailbox constant

boolify params

buf,buflen

tidy comments

merge: minor config fixes

use config code for neomutt -Q

config: fix display of changed config

config: fix dump

config: workaround for mailboxes

return success on neomutt -D

bug: check pointer

move address function to library

move expand functions to library

move realpath to library

group path functions

add pretty/canonical path functions

add path manipulation functions

doxy: Config docs

config: use better failing regex test

test/address.c: remove duplicated include

merge: update translations