reduce scope of loop variables

merge: refactor/tidy mutt_index_menu()

 * refactor index's check macros
 * reduce scope of loop variables
 * reduce scope of variables
 * tidy mutt_index_menu()

tidy mutt_index_menu()

reduce scope of variables

reduce scope of loop variables

refactor index's check macros

update list of translatable files

[ci skip]

merge: trivial code fixes

 * boolify Score->exact
 * boolify is_reply()
 * boolify do_locales
 * boolify exact
 * boolify imap_set_flags()
 * doxy: fix typo
 * comp_mbox_close: swap if clauses
 * fix sprintf specifiers
 * fix lgtm alerts

fix lgtm alerts

Each of the removed tests was redundant.

fix sprintf specifiers

comp_mbox_close: swap if clauses

doxy: fix typo

boolify imap_set_flags()

boolify exact

boolify do_locales

boolify is_reply()

boolify Score->exact

lgtm: add config

fix mailbox size

When re-opening a mailbox, the size wasn't being reset.

merge: trivial code fixes

 * fix spelling
 * compare integers against 0
 * standardise naming in B64Context
 * standardise naming in ComposeRedrawData
 * standardise naming in FgetConv
 * standardise naming in Group
 * standardise naming in ImapHeader
 * standardise naming in MailboxNode
 * standardise naming in Pattern
 * standardise naming in SslSockData
 * standardise naming in Pager
 * simplify bitfields in Mailbox
 * simplify bitfields in ImapAccountData
 * simplify bitfields in smtp
 * drop bitfields
 * doxy: mark all out-params
 * tidy mutt_folder_hook
 * index.c move data
 * tidy reopen_mailbox
 * doxy: add comment blocks
 * iwyu
 * boolify variables
 * doxy: fix typo
 * fix param names
 * rename pop_adata_get
 * eliminate asymmetric quotes
 * reduce variable scope

reduce variable scope

eliminate asymmetric quotes

rename pop_adata_get

fix param names

doxy: fix typo

boolify variables

iwyu

doxy: add comment blocks

tidy reopen_mailbox

index.c move data

tidy mutt_folder_hook

doxy: mark all out-params

drop bitfields

simplify bitfields in smtp

simplify bitfields in ImapAccountData

simplify bitfields in Mailbox

standardise naming in Pager

standardise naming in SslSockData

standardise naming in Pattern

standardise naming in MailboxNode

standardise naming in ImapHeader

standardise naming in Group

standardise naming in FgetConv

standardise naming in ComposeRedrawData

standardise naming in B64Context

compare integers against 0

fix spelling

merge: more Context elimination

 * add EmailList for tagged Emails
 * refactor edit_or_view_one_message
 * mutt_ev_message: factor out Context
 * bounce_message: factor out Context
 * mutt_check_traditional_pgp: factor out Context
 * crypt_extract_keys_from_messages: factor out Context
 * mutt_link_threads: factor out Context
 * mutt_label_message: factor out Context
 * pipe_msg: add Mailbox param
 * pipe_message: factor out Context
 * mutt_save_message: factor out Context
 * mutt_change_flag: factor out Context
 * ci_send_message: factor Context out of mutt_make_string()
 * ci_send_message: factor out Context
 * ci_send_message: factor Context out of dependents
 * mutt_mailbox_check: factor out Context
 * mx_mbox_sync
 * add mx_path_resolve()
 * mx_mbox_open: drop path param
 * factor out PostContext
 * tidy code
 * tidy Email variables
 * move parse_mailboxes()

move parse_mailboxes()

This function has dependencies on Context and inotify().

tidy Email variables

tidy code

Non-functional changes

factor out PostContext

mx_mbox_open: drop path param

add mx_path_resolve()

mx_mbox_sync

mutt_mailbox_check: factor out Context

ci_send_message: factor Context out of dependents

ci_send_message: factor out Context

ci_send_message: factor Context out of mutt_make_string()

mutt_change_flag: factor out Context

mutt_save_message: factor out Context

pipe_message: factor out Context

pipe_msg: add Mailbox param

mutt_label_message: factor out Context

mutt_link_threads: factor out Context

crypt_extract_keys_from_messages: factor out Context

mutt_check_traditional_pgp: factor out Context

bounce_message: factor out Context

mutt_ev_message: factor out Context

refactor edit_or_view_one_message

add EmailList for tagged Emails

Map SASL loglevels to neomutt loglevels

fixed: sort_browser attribute modified after sorting

sort_browser has modified after calling browser_sort,
which means the first time sort is called it uses the old
sort method instead of the new one.

Fixes #1551

split up 'update' and 'resort' notifications

Fixes #1546

fix: crash with short attachment name

tags: fix multiple tags set on multiple emails

When we set multiple tags on multiple emails at once, only the first
email get all tags.

This is because we use strsep that will modify the initial buffer that
contains all tags.

This change fixes it by duplicating the buffer of tags before using
strsep on it.

pager: fix missing Mailbox reference

Allow the creation of a new mbox when saving to a local file

Fixes #1537

notmuch: implement read-only vfolder-from-query

Implemented a read-only variant of vfolder-from-query for users who want
to have an immutable window. This ensures no accidental tag changes
occur.

+install instructions in README.md

context: move message_is_tagged,message_is_visible

mutt_make_string: separate Context from Mailbox

doxy: fix headers

notmuch: remove Context in nm_read_entire_thread()

Removed the coupling between `nm_read_entire_thread()` and `struct
Context` by replacing it with `struct Mailbox`.

`Context` had two uses: `Mailbox` reference source and `ctx_update()`.

The introduction of mailbox notification code allows mailbox events to
trigger `ctx_update()`, which removes the hard dependency on `Context`.

Passing a `Mailbox` reference removes the remaining soft dependency on
`Context`.

Relax non-empty requirement on compose_format, pager_format, and status_format (#1531)

Fixes #1530

Revert "Don't override ndata with temporary context"

This reverts commit c128c131185fb076476991f477565ba115352faa.

Don't override ndata with temporary context

postponed/sendlib code can reopen an already opened mailbox.

This overrides m->ndata with a temporary context but never revert it back to
the previous one.

At first glance, when postponed/sendlib code open the mailbox they don't care
to have their context updated with the notification system. It should be
the Context used for the UI that should be updated instead.

So until we don't open a mailbox twice anymore, just set ndata when it
looks accurate to workaround the issue.

fix: check for mailbox for folder-hook

Fixes #1527

fix: saving email to fcc

Sort by UID so sequence ranges in UID STORE make sense

This fixes the use case where a bunch of messages are tagged, removed,
then the mailbox is sync'd in a thread view. In this case, the sequence
number given by email->index is not monotonic, which results in weird
sequence sets being passed to UID STORE.

fix crash in ci_send_message()

Fixes #1525

fix crash in mutt_parse_unmailboxes()

Fixes #1523

close command prompt when deleting the colon

Fixes #1505

merge: upstream changes

 * Factor out rfc2047_decode_envelope()
 * Parse and store mime headers in the BODY
 * Add mx operation save_to_header_cache
 * Add protected header received email support
 * Add rfc2047_encode_envelope helper
 * Refactor save_fcc() function
 * Move fcc after send
 * Prompt when mutt_write_fcc() fails
 * Refactor postpone_message()
 * When postponing, encode descriptions before encrypting
 * More postpone cleanup
 * Fix flags if mutt_get_postponed() has no messages
 * Display mail sent message after Fcc finishes
 * Handle improperly encoded pgp/mime octetstream part
 * Add config and data structure for protected header write support
 * Add a new mode for mutt_write_rfc822_header()
 * Finish protected header write support
 * Read in protected headers when resuming a postponed message
 * Free pgpkeylist if the send fails
 * Rfc2047 encode and fold X-Label when writing updates
 * Restore the xlabel_changed reset
 * Make a copy of x_label before encoding it for updates
 * spelling consistency
 * Updated French translation
 * Added support for the "light" color prefix (in addition to "bright")
 * Light colors: do not do the + 8 on negative values, such as COLOR_DEFAULT
 * Create envelope->changed to mark all field changes
 * Change x-label editing flag to be in the envelope
 * Add $crypt_protected_headers_save
 * Color protected subject as a header in the pager
 * Respect ignore for the protected subject display
 * fix config order
 * Remove unused MUTT_FORMAT_MAKEPRINT flag
 * Sleep $sleep_time after displaying "mail sent" message
 * Update copyright for the next stable release

Update copyright for the next stable release

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

Sleep $sleep_time after displaying "mail sent" message

In case returning the index displays some kind of error right away, to
give time to see the success message.

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

Remove unused MUTT_FORMAT_MAKEPRINT flag

The code in mutt_FormatString() that was using it was commented out 19
years ago.

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

fix config order

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

Respect ignore for the protected subject display

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

Color protected subject as a header in the pager

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