tidy code

- rename variables: b to buf
- fix Buffer check
- remove redundant conditionals

merge: refactor Buffer handling

 * Tidy some Buffer API, keep most buffers on the stack
 * Keep spam member of struct Envelope by value
 * Kill mutt_buffer_from
 * Move buffer_new / buffer_free to static functions in pool.c
 * Add mutt_buffer_make to create a Buffer on the stack
 * Use mutt_buffer_is_empty instead of mutt_buffer_len where it makes sense

Use mutt_buffer_is_empty instead of mutt_buffer_len where it makes sense

Add mutt_buffer_make to create a Buffer on the stack

Move buffer_new / buffer_free to static functions in pool.c

Kill mutt_buffer_from

Keep spam member of struct Envelope by value

Tidy some Buffer API, keep most buffers on the stack

boolify 'optional' variables

Co-authored-by: Pietro Cerutti <gahr@gahr.ch>

rename Pattern members

doxy: tidy list comments

rename lists for consistency

- Move `entries` to end of the struct
- Separate declaration from use
- Rename `ColorLineHead`      to `ColorLineList`
- Rename `LogList`            to `LogLineList`
- Rename `MyVarHead`          to `MyVarList`
- Rename `ObserverHead`       to `ObserverList`
- Rename `PatternHead`        to `PatternList`
- Rename `RegexListNode`      to `RegexNode`
- Rename `ReplaceListNode`    to `Replace`
- Rename `TagHead`            to `TagList`
- Rename `TagNode`            to `Tag`
- Rename `UrlQueryStringHead` to `UrlQueryList`
- Rename `UrlQueryString`     to `UrlQuery`

unify time handling

- Add mutt_date_epoch_ms API, convert gettimeofdays to use the new API
- Add mutt_date_sleep_ms as a general sleep API

Remove useless abstraction

merge: light tidying

 * clang-format
 * pager: rename PagerRedrawData.index to menu
 * drop: unused sha1 code
 * rename two duplicate `struct Entry`
 * rename two duplicate `struct Syntax`

rename two duplicate `struct Syntax`

rename two duplicate `struct Entry`

drop: unused sha1 code

pager: rename PagerRedrawData.index to menu

clang-format

Make sure Progress doesn't reference an out-of-scope buffer

Cleanup mutt_progress_init's callers

Fixes: #1819

Add: mailbox_path() to tidy Mailbox usage

Remember last visited folder when browsing

Fixes #1781

clear up different redraw flags

The Menu and field-entering code had very similarly-named redraw flags.

compose: factor out global Window

sidebar: fix crash with no mailbox

menu: use enum for menu type

- Use `enum MenuType` rather than `int`
- Rename `Menu.menu` to `Menu.type`
- Light tidying

window: externalise global Windows

Many functions use a global `MuttWindow` pointer.
Change them to use a parameter version instead.

Currently, this is the same Window.
When the nested Windows arrive, it won't be.

window: add copy_size()

Create a dedicated copy function for `MuttWindow`, rather than using
`memcpy()`.

This is in preparation for when `MuttWindow` becomes more complex.

fix: alias expansion in compose

This fixes two bugs when expanding aliases, of the form "alias1, alias1"
in compose.

- only the first alias was expanded
- alias address was leaked

Refactor for clarity:

- Rename local variable
- Reduce scope of variable
- Add comments

fix: memory leak in mx_mbox_close()

Freeing a Context isn't enough; use ctx_free().

fix: memory leak in mutt_smtp_send()

Free the Connection after sending the Email.

fix: memory leak in parse_menu()

pager: change summaries to use mutt_do_pager()

By using mutt_do_pager(), the summary pages now honour "$pager".

Let my_hdr override MIME headers

Fix segfault in bounce

Fixes #1735

check for empty Context

We'd already checked for an empty Context at the beginning of the `case`
statement, but we didn't check after any functions that might have
closed the Context.

Fixes: #1799

Add envelope's Organization field to hcache

This ensures using %W in index_format while having
header cache enabled works as expected.

Note that as we don't modify the header files that
hcachever.sh is looking for, but the serial_dump_envelope
and serial_restore_envelope functions instead, we have
to increment BASEVERSION to ensure that cache built with
previous versions gets invalidated in all cases.

Fixes #1808

doxy: fix typo

merge: trivial fixes

 * rename MENU_POST to MENU_POSTPONE
 * imap: rename the command response constants
 * add more Context checks
 * add Context checks to status_format_str()
 * define flags for mutt_pattern_comp()
 * remove unnecessary goto/label
 * reorder params in driver_tags_get_transformed_for()
 * move data/structs to beginning
 * boolify some variables
 * rename mutt_menu_destroy()
 * rename ParameterList variables
 * fix parameter usage of calloc()
 * hcache: unify naming in free()
 * tidy pre-processor defs
 * refactor pre-processor
 * docs: rename mutt to neomutt
 * Update autocrypt keyring documentation
 * Clean up the autocrypt doc a bit more
 * tidy some comments/whitespace

tidy some comments/whitespace

Clean up the autocrypt doc a bit more

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

Update autocrypt keyring documentation

Recommend setting $autocrypt_dir to your normal keyring directory if
you want to use your existing key.

Trying to copy it over leads to signature verification issues.  Even
if I reversed the order (which is much less clean), that would just
lead to missing key signature errors for Autocrypt messages instead.

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

docs: rename mutt to neomutt

refactor pre-processor

tidy pre-processor defs

hcache: unify naming in free()

Ensure all the hcache backends clean up in the same way.

fix parameter usage of calloc()

rename ParameterList variables

For consistency, call ParameterLists `pl`.

rename mutt_menu_destroy()

All the other free functions are called *_free().

boolify some variables

move data/structs to beginning

reorder params in driver_tags_get_transformed_for()

Change the parameter order to match all the other tags functions.

remove unnecessary goto/label

define flags for mutt_pattern_comp()

add Context checks to status_format_str()

add more Context checks

Check for the condition where the Context exists, but `Context->mailbox`
is NULL.

Fixes: #1747

imap: rename the command response constants

Rename command responses to `IMAP_RES_*` to avoid clashing with the
command flags `IMAP_CMD_*`.

rename MENU_POST to MENU_POSTPONE

'POST' implies sending an email, but the menu deals with postponed
emails.

Fix segfault in PGP key selection

Fixes #1809

coverity: fix defects

Clarify hcache invalidation mechanism

doxy: add autocrypt pages

merge: update translations

 * Update lt_LT translatons
 * Update pt_BR translation
 * Update pt_BR translation
 * Polish translate update
 * sync translations

sync translations

Polish translate update

Update pt_BR translation

Update pt_BR translation

Update lt_LT translatons

Fix the fuzzies.

merge: upstream autocrypt feature

 * address: add mutt_addr_to_intl()
 * build: add autocrypt option
 * Initial autoconf and makefile setup for autocrypt
 * Add autocrypt config vars
 * Add database and schema initialization
 * Start autocrypt gpgme
 * Factor out mutt_edit_address() in send.c
 * Add initial autocrypt account setup
 * Add autocrypt header parsing to mutt_parse_rfc822_line()
 * Process autocrypt headers
 * Convert to use sqlite3_prepare_v3()
 * Add gossip header processing
 * Change autocrypt_db normalization to return struct Address
 * Change gossip header address comparison to use normalized addresses
 * Add another struct Email security bit for autocrypt
 * Add autocrypt line to the compose menu
 * Autocrypt outgoing emails
 * Fix process_gossip_headers() to look at the right envelope
 * Slightly improve the gossip header key update
 * Handle autocrypt message decryption
 * Print an error message if an autocrypt key can't be found
 * Add autocrypt source files to POTFILES.in
 * Add L10N translation message comments
 * Add AUTOCRYPT header to imap fetch list
 * Add prefer-encrypt prompt during account creation
 * Add account->enabled checks
 * Improve gen-map-doc work on multi-word maps
 * Convert peer_update to just pass the peer object
 * Basic autocrypt account menu
 * Add more translation string comments for autocrypt
 * Add option to scan mailboxes during first run
 * Add documentation for the autocrypt feature
 * Add autocrypt README file
 * Don't try to decrypt autocrypt messages if the option is off
 * fix test cases
 * iwyu: tidy #includes
 * rename local variable rv to rc
 * doxy: add comment blocks
 * rename local variable hdr to e
 * rename local variable msg to e
 * add braces to unclear 'if' statements
 * explicitly check retval for strcmp, buffer_len
 * add whitespace for clarity
 * tidy code
 * Add ability to create autocrypt account from an existing key
 * Automatically clear screen when popping the last menu
 * Add $autocrypt_reply
 * Allow nested setting of OptIgnoreMacroEvents
 * Turn off macro processing during autocrypt initialization
 * Enabled writing protected subject with autocrypt too
 * Minor autocrypt manual section updates
 * Add autocrypt documentation about shared key and keyring strategies
 * Add a note about autocrypt first-run and macros
 * Add L10N comment about autocrypt menu help line abbreviations
 * Mention base64 keydata being stored in the autocrypt database

Mention base64 keydata being stored in the autocrypt database

Add L10N comment about autocrypt menu help line abbreviations

Add a note about autocrypt first-run and macros

Add autocrypt documentation about shared key and keyring strategies

Minor autocrypt manual section updates

Mention key selection during account creation and $autocrypt_reply for
controlling autocrypt mode setting during replies.

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

Enabled writing protected subject with autocrypt too

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

Turn off macro processing during autocrypt initialization

The muttrc can push events into the macro buffer, with the assumption
that mailbox will be opened next.  This will interfere with the
prompts uses during first run.  The only issue is that macros won't
work inside the folder browser, if invoked.

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

Allow nested setting of OptIgnoreMacroEvents

This will be needed for the first-run of autocrypt initialization.
The initialization involves a number of prompts, and can even use the
folder browser.  The browser could in turn connect to IMAP which might
invoke a certificate prompt, login prompt, etc.

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

Add $autocrypt_reply

This allows turning off the force-autocrypt mode when replying to an
autocrypt email.

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

Automatically clear screen when popping the last menu

This removes the need to manually clear the screen during pre-menu
operations, such as autocrypt first run initialization.

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

Add ability to create autocrypt account from an existing key

This is useful when adding accounts, or for users who want to use an
existing key from the keyring during initial account creation.

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

tidy code

- boolify variables, members
- reduce variable scope
- initialise pointers
- bracket logic
- name parameters in prototypes

add whitespace for clarity

explicitly check retval for strcmp, buffer_len

add braces to unclear 'if' statements

rename local variable msg to e

rename local variable hdr to e

doxy: add comment blocks

rename local variable rv to rc

iwyu: tidy #includes

fix test cases

Don't try to decrypt autocrypt messages if the option is off

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

Add autocrypt README file

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

Add documentation for the autocrypt feature

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

Add option to scan mailboxes during first run

The scanner will disable the header cache, allowing scanning all
messages in the mailbox.

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

Add more translation string comments for autocrypt

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

Basic autocrypt account menu

Provide ability to create, delete, and toggle the prefer-encrypt and
enabled flag for an account.

Hook into the index via 'A' <autocrypt-acct-menu>.

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

Convert peer_update to just pass the peer object

Unlike get and insert, there is no need for an addres parameter that
needs to be normalized, since the address is already in the peer
object.

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

Improve gen-map-doc work on multi-word maps

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

Add account->enabled checks

Add to ui recommendation, keylist generator, and autocrypt header
writing routines.

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