Make NotMuch hcache-aware

Closes: #1600

Fix table in man page

Previously:

    /usr/bin/tbl:<standard input>:824: unrecognised format `\'
    /usr/bin/tbl:<standard input>:824: giving up on this table

It seems the period after table headers must be on its own line.

merge: trivial fixes

 * trivial ()s and NUL
 * pop: add typedef for pop_fetch_data()
 * doxy: add links
 * more NULs
 * fix typos
 * add checks to imap_account_match()
 * add check to imap_adata_find()
 * maildir: rename context functions
 * add flags to maildir_mbox_check()

add flags to maildir_mbox_check()

maildir: rename context functions

These functions now work with Mailboxes:
- maildir_add_to_context()
- maildir_move_to_context()

add check to imap_adata_find()

add checks to imap_account_match()

fix typos

more NULs

doxy: add links

pop: add typedef for pop_fetch_data()

trivial ()s and NUL

Do not prepend "attach_save_dir" to an absolute path

This happens when saving multiple attachment. Assume `attach_save_dir = ./`.
When saving multiple attachments to `/tmp/att`, the proposed path
becomes `.//tmp/att`.

hcache: fix hash generator

`struct Header` was renamed to `struct Email`

Fixes: #1595

added: localtime_r/gmtime_r wrappers

Fixes: #1571

Fix incorrect IMAP message purging bug.

Thanks to Ivan Middleton @imiddle for the awesome bug report and
suggested fix.

The bug is most easily generated using Gmail with the $trash variable
set.

Deleted messages are first copied to the $trash folder.  If this is
set to "[Gmail]/Trash", then Gmail inteprets the copy as a "delete"
and sends EXPUNGE messages back for the messages.

cmd_parse_expunge() and cmd_parse_vanished() set the hdr->index to
INT_MAX, which subsequently an imap_expunge_mailbox() will use to
remove the messages from the local mailbox.

If we close the mailbox instead of sync it, Mutt will end up executing
the 'Deleted' flag setting before processing the expunge (because
"imap_check_mailbox() -> imap_cmd_finish()' doesn't set check_status
when we are closing).  The expunged messages will then be included in
the set of 'Deleted' flags.

Unfortunately, because the messages are sorted by *index* before
msgset generation, an incorrect range of UIDs will be sent, which
could easily include messages that should not be deleted.

This fix is a minimal fix for a stable bug fix  excluding
messages with the index set to INT_MAX from all msg sets.  Other
things that should be investigated in master are:

- sorting by UID instead of index before msgset generation
- unsetting the 'active' flag in cmd_parse_expunge() and
  cmd_parse_vanished() instead of waiting until imap_expunge_mailbox()
  to do so.

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

Fix opening IMAP messages when the body cache is disabled

Fixes #1574

mx: fix mbox double free

Fixes: #1597

Fix pointer assignment to '\0' introduced in 495153

merge: merge translations

 * Corrected several minor errors in the spanish translation
 * Update Czech translation
 * More miscellaneous improvements to spanish localization
 * Added a few miscellaneous translations
 * Update lt_LT translation
 * Polish translation update
 * update po files list
 * sync translations

sync translations

update po files list

Polish translation update

Update lt_LT translation

Fixed three new fuzzy entries and a lot of typos in old translations
because I decided to do a spell check on a whim.

Added a few miscellaneous translations

More miscellaneous improvements to spanish localization

Update Czech translation

Corrected several minor errors in the spanish translation

iwyu: optimise #includes

use SecurityFlags more

Some functions missed out on the earlier upgrade.

merge: Use NUL or NULL rather than 0

 * Use NUL or NULL rather than 0 when setting
 * Test string contents against NUL (==)
 * Test string contents against NUL (!=)

Test string contents against NUL (!=)

```diff
- while (*buf)
+ while (*buf != '\0')
```

```diff
- if (bestname[0] != 0)
+ if (bestname[0] != '\0')
```

Test string contents against NUL (==)

```diff
- if (!bestname[0])
+ if (bestname[0] == '\0')
```

```diff
- if (dest[0] == 0)
+ if (dest[0] == '\0')
```

Use NUL or NULL rather than 0 when setting

This helps to reinforce the variables' types when reading the code.

```diff
 char buf[10];
-buf[0] = 0;
+buf[0] = '\0';
```

```diff
 void *ptr;
-ptr = 0;
+ptr = NULL;
```

merge: trivial tidying

 * Rename variables: string -> str
 * docs: fix typos
 * fix comments
 * tidy L10N comments
 * fix block comments
 * fold comments

fold comments

fix block comments

tidy L10N comments

fix comments

docs: fix typos

Rename variables: string -> str

Consistent variable naming makes the code easier to read.

fix pre-increment in for loops

In for loops, there's no difference in behaviour between:

  `for (int i = 0; i < 10; ++i)`
  `for (int i = 0; i < 10; i++)`

Use the C form, `i++`, for consistency.

move switch/case break

Move the `break` inside the {}s of a case statement (for consistency).

move group to email

group.[ch] have dependencies on Address, so they should live in libemail.

Disable all history operations if history is not enabled

Fixes #1592

Fix composing a new message (again)

The old code had crept in again in 028c34.

add parentheses to conditionals

(flatcap discovers uncrustify)

ifdef: add support for environment variables

This allows the user to make config decisions based on the existence of
environment variables.  e.g.

  ifdef PS1 source interactive.rc

drop devel script

Committed by accident

[ci skip]

merge: lots of tidying

 * fp prefix
 * return r
 * return ret
 * return result
 * rename SHA1 contexts
 * rename MD5 contexts
 * create enum SortType
 * add typedef for PatternFlags
 * use enum QuadOption for query_quadoption(), mutt_yesorno()
 * separate enum for mutt_view_attachment()
 * separate enum for mailcap lookups
 * separate enum for mutt_save_attachment()
 * separate enums for pattern types
 * create TreeChar enum for tree characters

create TreeChar enum for tree characters

separate enums for pattern types

separate enum for mutt_save_attachment()

separate enum for mailcap lookups

separate enum for mutt_view_attachment()

use enum QuadOption for query_quadoption(), mutt_yesorno()

add typedef for PatternFlags

create enum SortType

rename MD5 contexts

Distinguish MD5 contexts from Context contexts.

rename SHA1 contexts

Distinguish SHA1 contexts from Context contexts.

return result

return ret

return r

fp prefix

Let ctx_update_tables update the mx size for IMAP emails

Issue #1577

Decreasing mbox size is done at ctx_update_tables, !active is not used

Issue #1577

Fix crash when an email disappears from IMAP

Issue #1577

Update to latest autosetup

Fix composing a new message

Fixes #1582

merge: fix coverity defects

 * defect 180847: Pointer to local outside scope
 * defect 189753: Inferred misuse of enum
 * defect 190188: Unchecked return value from library
 * defect 190189: Unchecked return value
 * defect 190190: Dereference before null check
 * defect 190853: Dereference after null check
 * defect 190854: Logical vs. bitwise operator
 * defect 190855: Dereference after null check
 * defect 190856: Overlapping buffer in memory
 * defect 190857: Dereference after null check

defect 190857: Dereference after null check

defect 190856: Overlapping buffer in memory

defect 190855: Dereference after null check

defect 190854: Logical vs. bitwise operator

defect 190853: Dereference after null check

defect 190190: Dereference before null check

defect 190189: Unchecked return value

defect 190188: Unchecked return value from library

defect 189753: Inferred misuse of enum

defect 180847: Pointer to local outside scope

merge: trivial tidying

 * clang-format
 * rename variables: descrip -> desc
 * rename variables: buffer -> buf
 * rename variables: command -> cmd
 * rename variables: progressbar -> progress
 * rename variables: thepid -> pid
 * rename variables: synch -> sync
 * name macro variables
 * lowercase local variables
 * expand meaningless R_BOTH macros
 * tidy mutt_edit_content_type()
 * tidy generate_body()
 * tidy mutt_alias_create()
 * small code tidying
 * boolify variables/params
 * reduce variable scope

reduce variable scope

boolify variables/params

small code tidying

- fix typo
- drop once-used define
- rename old_msg_count
- drop unnec parentheses

tidy mutt_alias_create()

- return early, reducing indent
- rename variables for clarity

tidy generate_body()

- fix variable type

tidy mutt_edit_content_type()

- boolify
- parenthesise

expand meaningless R_BOTH macros

Both what?

- R_BOTH -> R_INDEX | R_PAGER
- R_RESORT_BOTH -> R_RESORT | R_RESORT_SUB

lowercase local variables

Local variables should be lowercase and snake_case.

Consistent variable naming makes the code easier to read.

name macro variables

Change macros from:

`#define function(A,B,C) other(A,B,C)`

to

`#define function(ctx,buf,buflen) other(ctx,buf,buflen)`

rename variables: synch -> sync

Consistent variable naming makes the code easier to read.

rename variables: thepid -> pid

Consistent variable naming makes the code easier to read.

rename variables: progressbar -> progress

Consistent variable naming makes the code easier to read.

rename variables: command -> cmd

Consistent variable naming makes the code easier to read.

rename variables: buffer -> buf

Consistent variable naming makes the code easier to read.

rename variables: descrip -> desc

Consistent variable naming makes the code easier to read.

clang-format

merge: Define types for more flags/bitfields

 * imap: rename constants to match capability strings
 * smtp: convert capability enum to flags
 * add typedef for ImapCapFlags
 * add typedef for SmtpCapFlags
 * add typedef for AnsiFlags
 * add typedef for CheckFlags
 * add typedef for MhSeqFlags
 * add typedef for AclFlags
 * add typedef for HashFlags
 * add typedef for MixCapFlags
 * add typedef for CliFlags
 * add typedef for SelectFileFlags

add typedef for SelectFileFlags

add typedef for CliFlags