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>

Add prefer-encrypt prompt during account creation

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

Add AUTOCRYPT header to imap fetch list

If $autocrypt is set, add the header to make IMAP behave the same as
other formats (parsing on initial mailbox opening).

This will also be useful if we perform an initial scan during account
creation, to make that work for IMAP too.

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

Add L10N translation message comments

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

Add autocrypt source files to POTFILES.in

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

Print an error message if an autocrypt key can't be found

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

Handle autocrypt message decryption

Try autocrypt first for crypt_pgp_decrypt_mime() and
pgp_gpgme_encrypted_handler().

Propagate the autocrypt bit if successful.  This is used when replying
to an autocrypt message, to force a reply using autocrypt.

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

Slightly improve the gossip header key update

Don't set an empty peer.gossip_keydata with a value that matches the
current peer.keydata.  This just wastes space.

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

Fix process_gossip_headers() to look at the right envelope

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

Autocrypt outgoing emails

Change crypt_get_keys() to query autocrypt.

When oppenc_mode is set, we still query the original keyring
regardless, because the compose menu can still run oppenc even if
autocrypt is on.

Since mutt_autocrypt_ui_recommendation() checks each key as part of
making the recommendation, add a keylist parameter and use that
function.

Add gpgme changes to use the autocrypt context for encryption.

Postpone work:
* Change mutt_protect() to have a postpone parameter.  Remove the
  manual toggling of the SEC_SIGN bit outside the call when postponing.

* Since autocrypt doesn't set the SEC_SIGN bit, this allows us to turn off
  signing inside mutt_protect() for both normal and autocrypt mode.

* Set the autocrypt postpone key in AutocryptDefaultKey.

Write autocrypt and gossip headers in outgoing emails.

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

Add autocrypt line to the compose menu

Remove the hardcoded HDR_ATTACH offset calcuation, and add an explicit
enum for the "-- Attachments" line to make loops and padding array
sizes easier.

Add security and recommendataion fields on the line.

Add mutt_autocrypt_ui_recommendation, following the autocrypt spec
guidelines.

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

Add another struct Email security bit for autocrypt

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

Change gossip header address comparison to use normalized addresses

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

Change autocrypt_db normalization to return struct Address

This makes reusing the code in autocrypt.c easier.

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

Add gossip header processing

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

Convert to use sqlite3_prepare_v3()

The documentation suggests the SQLITE_PREPARE_PERSISTENT flag is
helpful for long-lived prepared statements, and Mutt is using.

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

Process autocrypt headers

Create/update peer database accounts and gpg keys based on the headers.

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

Add autocrypt header parsing to mutt_parse_rfc822_line()

Convert parse_parameters() for autocrypt header usage:

  * change to use a struct Buffer to accomodate large autocrypt keydata
    attribute values.

  * Autocrypt header parameters are not rfc2231 compliant.  Rather
    than rolling another very similar function, just change the
    existing one to allow space separation.

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

Add initial autocrypt account setup

Generate gpg key and add account record to the database.

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

Factor out mutt_edit_address() in send.c

For reuse by autocrypt for address prompts.

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

Start autocrypt gpgme

Add a basic init function.

Bump up the required gpgme version to 1.8.0 if autocrypt is enabled.

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