Simplify nested smime handling in the attachment menu

Since changeset 2fd6f99b allows nested encryption handling, there is
no need to deal with the nesting directly.  Instead, just recursive as
with other nested handling.

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

Improve attachment menu for s/mime parts

Don't prompt for passphrase or getkeys, or set the ENCRYPT flag, for
OPAQUE types.

Don't recurse on "text" output from decrypt_mime().  There is no
reason to recurse on a text type.  Additionally, the
mutt_read_mime_header() will return an empty text type even if the
decode doesn't generate mime output.  In those cases, we want to show
the original attachment.

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

Fix tunnels to also retry and write full buffer

Change the tunnel_socket_read() and tunnel_socket_write() as the raw
sockets were adjusted in the previous commit.  Retry on EINTR, and
complete a full write so all the implementations have the same behavior.

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

Fix gnutls tls_socket_write() to properly retry

Retry on GNUTLS_E_AGAIN and GNUTLS_E_INTERRUPTED.  This prevents an
aborted send due to a SIGWINCH, for instance.

Change tls_socket_read() to follow the same flow.  Don't bother
checking gnutls_error_is_fatal() because return codes besides AGAIN
and INTERRUPTED end up closing the connection regardless.  (We don't
handle handshakes and negotations during send/receive).

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

Change $pgp_use_gpg_agent to default set

GnuPG 2.1.0, released in 2014-11-06, automatically spawns an agent.
After 4+ years, it has reached wide enough usage to merit changing the
default.

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

Wrap ssl init calls for LibreSSL too

It looks like LibreSSL does not perform automatic initialization of
the library and error strings.

Since LibreSSL defines OPENSSL_VERSION_NUMBER as a "version 2",
add a check if LIBRESSL_VERSION_NUMBER is defined and call the
initialization functions for that case.

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

Add attributes support on color declarations

color now accepts zero or more attributes words before the foreground.

Also more or less resolves the issue that setting the color of an object
which defaults to underline/reverse is irreversible.

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

Fix compilation with LibreSSL <2.7.0

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

clang-format

Reword "postpone" message to avoid confusion

Replace "Postpone this message?" with "Save (postpone) draft message?"

Issue #1312

Update neomuttrc manual page (via init.h documentation) so that the
actual text of the $postpone quadoption prompt appears in the blurb, so
users can grep for it.

added: :bind, :macro interactive commands

Display all the active bindings/macros.

Issue #162

notmuch: delegate msg_open_new to maildir backend

Notmuch delegates several functions to the maildir backend since they
are connected. This commit delegates the notmuch msg_open_new() to the
maildir backend's implementation.

The primary benefit is being able to delete attachments from emails.
Previously, if an attachment was marked for deletion then it was
impossible to exit a virtual mailbox since a write operation could not
complete. This bug has been in Neomutt for a while.

Fixes #673

Replace integer parameters with booleans

Several methods used integer parameters to represent boolean values;
replacement with a boolean improves readability.

Affects function/macro(s):
 - mutt_emails_set_flag()
 - mutt_thread_set_flag()
 - mutt_set_flag()

merge: refactor/tidy mutt_enter_string_full()

 * boolify variables
 * use NUL rather than zero
 * reduce scope of variables
 * tidy mutt_enter_string_full()

tidy mutt_enter_string_full()

reduce scope of variables

use NUL rather than zero

boolify variables

merge: refactor/tidy mutt_compose_menu()

 * reduce scope of variables
 * boolify variables
 * fix scoping
 * merge non-overlapping buffer use
 * use NUL rather than zero
 * var scope
 * tidy mutt_compose_menu()

tidy mutt_compose_menu()

var scope

use NUL rather than zero

merge non-overlapping buffer use

`buf` and `fname` were never used at the same time, so just use one
temporary buffer.

fix scoping

- move some }s
- return early, where possible

boolify variables

reduce scope of variables

merge: refactor/tidy mutt_select_file()

 * use NUL rather than zero
 * boolify variables
 * separate op
 * reduce scope of variables
 * tidy mutt_select_file()

tidy mutt_select_file()

reduce scope of variables

separate op

boolify variables

use NUL rather than zero

merge: refactor/tidy mutt_pager()

 * reduce scope of loop variables
 * reduce scope of variables
 * tidy mutt_pager()

tidy mutt_pager()

reduce scope of variables

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