replace STRING with 256

replace SHORT_STRING with 128

config: add 'C_' prefix

Each config variable has a global variable backing it.
The name of the global is the MixedCase transform of the config name.
While this is useful for remembering their names, it makes some
variables hard to search for, giving many false positives.

- Copy
- From
- Help
- Move
- Sort

Prefixing the globals with `C_` prevents this problem.

use EmailList for piping and printing messages

Fix mutt_pipe_message() and mutt_print_message()

A discrepancy in the prototype and the function lead to these functions
crashing.

Fixes: #1579

build: fix fmemopen

Thanks to @vinpublic

unify the naming of chflags/cmflags

Most functions used `chflags` for header flags and `cmflags` for message
flags.  Now they all do.

merge: trivial fixes

 * insert space to help parsing
 * fix backing global variable for config
 * doxy: fix typos
 * drop dupe prototype
 * lowercase variable
 * doxygen: update docs for #defines
 * add a 'SEC_' prefix to the security flags
 * rename State's members for better legibility
 * rename enum MenuTypes to MenuType
 * convert config dump defines into an enum
 * convert format flags enum into an int
 * tidy mutt_menu_new()

tidy mutt_menu_new()

Rename the function parameter and the local variable to be clearer.

convert format flags enum into an int

This wasn't really an enumeration, but a set of flags.

convert config dump defines into an enum

rename enum MenuTypes to MenuType

When dealing with an enum, you have a single value.

rename State's members for better legibility

add a 'SEC_' prefix to the security flags

Symbols like 'SIGN' and 'INLINE' are hard to search for.

doxygen: update docs for #defines

lowercase variable

Local variables should be lowercase.

drop dupe prototype

doxy: fix typos

fix backing global variable for config

Global name didn't match config name.

insert space to help parsing

release: fix build tests

merge: Use symbolic log levels

 * Factor out awful inline code for logging binary buffer
 * Align numeric log level arguments
 * Adjust symbolic logging levels to avoid level 4, to be PROGRESS
 * Fix wrong calculation of remaining length in nntp_log_binbuf
 * Start replacing numeric loglevels with symbolic.

Start replacing numeric loglevels with symbolic.

This commit moves `mutt_debug(1, ...)` to `mutt_debug(LL_DEBUG1, ...)`

Fix wrong calculation of remaining length in nntp_log_binbuf

An even better way to write nntp_log_binbuf

Adjust symbolic logging levels to avoid level 4, to be PROGRESS

Adjust remaining uses of symbolic loglevels.

Mostly this means changing mutt_socket_readln to mutt_socket_readln_d
with MUTT_SOCK_LOG_FULL in cases where (in my very limited understanding)
private information *might* be logged.

Align numeric log level arguments

Factor out awful inline code for logging binary buffer

merge: upstream changes

 * Fix compilation with LibreSSL <2.7.0
 * Add attributes support on color declarations
 * Wrap ssl init calls for LibreSSL too
 * Change $pgp_use_gpg_agent to default set
 * Fix gnutls tls_socket_write() to properly retry
 * Fix tunnels to also retry and write full buffer
 * Improve attachment menu for s/mime parts
 * Simplify nested smime handling in the attachment menu
 * Show top-level decoded smime text/plain parts
 * Allow descending into maildir and mh directories in file browser
 * Add an error message for <descend-directory>
 * Updated French translation
 * Create <group-chat-reply> function
 * Updated French translation
 * Add mutt_buffer_len() helper
 * Add mutt_buffer helpers for base64 conversion
 * Change auth_gss to to use buffers and the buffer pool
 * Check the base64 decoding retval in auth_gss

Check the base64 decoding retval in auth_gss

Abort if the value is not decodable.

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

Change auth_gss to to use buffers and the buffer pool

Ticket #100 revealed that even a bufsize of 8192 isn't guaranteed to
be big enough.  Convert those large buffers to use the buffer pool
instead.

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

Add mutt_buffer helpers for base64 conversion

Add mutt_buffer_from_base64() mutt_buffer_to_base64() to help with
transitioning to buffers.

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

Add mutt_buffer_len() helper

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

Updated French translation

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

Create <group-chat-reply> function

This function differs from <group-reply> only in that it preserves To
recipients in the original email as To recipients in the reply.

The merits of this function aren't without controversy; therefore it
is left unbound by default.  Those who care about such things are free
to bind it.

Combine reply operation handlers in the pager and curs_main, since the
code was the same with the exception of the flags used.

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

Updated French translation

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

Add an error message for <descend-directory>

Don't allow the operation on a non-directory.  Print a message
indicating it's not a directory.

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

Allow descending into maildir and mh directories in file browser

Some (maybe most) IMAP mail systems allow folders to be nested inside
folders, so you can have, say, an Inbox with more folders inside it.
However, in the file browser, mutt will only open a maildir/mh
directory as a mailbox, there is no way to get to the child mailboxes.

This change adds a function <descend-directory> that forces mutt to
descend into the directory.  It is unbound by default.

Signed-off-by: Corey Minyard <minyard@acm.org>
Co-authored-by: Richard Russon <rich@flatcap.org>

Show top-level decoded smime text/plain parts

Commit 331d9d5a attempted to fix a problem with an attachment having
extension .p7m.  The attachment menu tried to decode and failed, and
the part ended up being replaced by a bogus text/plain part.

The problem is that crypt_smime_decrypt_mime() returns a text/plain
part if the decode fails, meaning we can't distinguish failure from
success in this case.

As a compromise, only use a text/plain resulting from a single
top-level application_smime part.  This will allow for the case of an
text/plain encoded email, but won't end up hiding attachments that
were not decoded.

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

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