de-Context nntp_check_msgid()

drop unused maildir_update_tables()

rename mutt_emaillist_free() to _clear()

The function doesn't free the Emails, just the list, so rename it to
match similar functions.

fix whitespace

doxygen: fix comments

merge: upstream fixes

Sync to upstream: 82973a6e

 * change Command to use intptr_t
 * Don't read or save history if $history_file isn't set
 * Omit User-Agent: header by default
 * Remove unnecessary checks for strings
 * Convert $header_cache_pagesize to type DT_LONG

Convert $header_cache_pagesize to type DT_LONG

Prior to commit 4bc76c2f there was no LNUM type, and so the workaround
was to store it as a string, converting in the hcache_open_gdbm()
call.

This will not affect the user interface or config file, because DT_NUM
and DT_LNUM read in a string from the config file and convert to a
number.  Quotes are used for escaping style, not passed through to the
variable setter.

So essentially this simply moves the conversion to parse_set(), and
provides feedback for a non-numeric type immediately.

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

Remove unnecessary checks for strings

MuttVars of those types are set via safe_strdup(), which returns NULL
if the original is "".  Thus Var implies *Var.

A good portion of the code relies on that axiom, but over the years
some (Var && *Var) checks have crept in, including from me.

This was partially because of the INITVAL("") that were in the code,
which implied (incorrectly) the initial value could be "".  Commit
2f91d43e removed those to make it more clear.

This commit removes the *Var checks to make it even clearer, and help
avoid them creeping back in again.

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

Omit User-Agent: header by default

The User-Agent: header can be fun and interesting and useful for
debugging, but it also leaks quite a bit of information about the user
and their software stack.

This represents a potential security risk (attackers can target the
particular stack) and also an anonymity risk (a user trying to
preserve their anonymity by sending mail from a non-associated account
might reveal quite a lot of information if their choice of mail user
agent is exposed).

Users who want to configure `user_agent` to `yes` can still do so, but
it makes sense to have safer defaults.

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

Don't read or save history if $history_file isn't set

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

change Command to use intptr_t

fix browser attach

sync translations

fix re-opening attachments

When opening an attachment with a two part suffix, e.g. `.tar.gz`
NeoMutt created a temporary file with just `.gz`.

Hack the splitting logic to give a useful suffix

Fixes: #1699

Avoid implicit conversion between bool and quadopt

Make the conversion between `bool and `QuadOption` explicit, and avoid
depending on the underlying `enum` value.

Also state that `mutt_yesorno` works correctly for `MUTT_YES` and `MUTT_NO`

Make callback take const param

Otherwise a realloc is potentnially unsafe

Initialize the variable `index_hint`

Prior to this commit, `index_hint` is initialized if `Context && !attach_msg`.

If `query_quadoption(C_Quit, _("Quit NeoMutt?")) == MUTT_YES`,
`!Context || ((check = mx_mbox_close(&Context)) == 0)` is `false` and
`(check == MUTT_NEW_MAIL) || (check == MUTT_REOPENED)` the function `update_index`
is called with parameter `index_hint`.

As one of the condition is `query_quadoption(C_Quit, _("Quit NeoMutt?")) == MUTT_YES`,
the real current position is probably not so important.

merge: account preparations

 * split up mailbox.c
 * notify: add global notifications
 * neomutt: move AllAccounts
 * notify: add Account Events
 * refactor users of AllMailboxes
 * drop AllMailboxes
 * account: add mailbox_add()
 * notify: add Mailbox Events
 * notify: drop old Context notifications
 * config: add config subsets
 * config: convert to subset
 * mailbox: rename desc to name
 * create libcore

version: add colour for devel features

create libcore

doxygen: Address struct members

mailbox: rename desc to name

doxygen: Email struct members

config: convert to subset

gitignore: drop old test binaries

config: add config subsets

notify: drop old Context notifications

notify: add Mailbox Events

account: add mailbox_add()

drop AllMailboxes

refactor users of AllMailboxes

notify: add Account Events

neomutt: move AllAccounts

notify: add global notifications

split up mailbox.c

merge: Improve const-correctness

 * Improve const-correctness
 * Do not cast const qualifier from sort callbacks
 * Make const discards more explicit
 * Remove useless cast

Remove useless cast

Make const discards more explicit

Do not cast const qualifier from sort callbacks

If `const void*` points to an instance of type `T`, then it should be
casted to a `const T*` instance.

If `const void*` is a pointer to an instance of type `T`, then the pointer
is `const T*`.

The instance could be `const`, or could be not, but to be on the safe
side, it's casted to `T const *const *`.
In the case of pointer-to-pointer, `const` is on the right to ease
readability; from right to leaft it reads:
A pointer to a const pointer to a const T.

Improve const-correctness

of

  * convert_file_to
  * get_quote_level
  * mutt_write_one_header
  * classify_quote
  * gen_string_hash
  * gen_case_string_hash
  * mutt_buffer_enter_fname_full
  * parse_references

Avoid using reserved identifiers

Found with `clang -Wreserved-id-macro`

merge: trivial fixes

 * rename templates
 * drop old refs to struct Header
 * unify naming of Email vars/params
 * unify naming of Mailbox vars/params
 * doxy: unify MXAPI comments
 * notmuch: fix prototypes

notmuch: fix prototypes

doxy: unify MXAPI comments

unify naming of Mailbox vars/params

unify naming of Email vars/params

drop old refs to struct Header

rename templates

Rename typedef to ParseDateRangeFlags

As there is another definition of PatternFlags

merge: trivial fixes

 * fix whitespace
 * add missing flag
 * fix compress local init
 * sidebar: pass in Mailbox
 * debug: fix levels and messages
 * tags: fix leak
 * notify: fix observer retval
 * drop nm_description_to_path() - unused
 * rename locals for consistency
 * rename account_remove_mailbox()
 * monitor #define -> enum

monitor #define -> enum

rename account_remove_mailbox()

rename locals for consistency

drop nm_description_to_path() - unused

notify: fix observer retval

tags: fix leak

debug: fix levels and messages

sidebar: pass in Mailbox

fix compress local init

add missing flag

fix whitespace

merge: Avoid using c++ reserved identifiers

 * Replace `try` variable with `user`
 * Replace `virtual` variable with `vnum`
 * Replace `template` variable with `tmpl`
 * Replace `catch` variable with `restore`
 * Replace `new` variable with `has_new_mail`
 * Replace `new` variable with`new_file`
 * Replace `public` variable with `only_public_key`
 * Replace `delete` variable with `delete_original`
 * Replace `new` variable with `mark_new`
 * Replace `new` variable with `al_new`
 * Replace `new` variable with `alias`
 * Replace `new` variable with `ap`
 * Replace `new` with `pnew`
 * Unify `body_new` name instances
 * Replace `new` variable with `add`
 * Replace `new` variable with `new_mail`
 * Replace `new` variable with `new_tag`
 * Replace `new` variable with `new_label`
 * Replace `new` variable with `tnew`
 * Replace `new` variable with `new_mail`
 * Replace `delete` variable with `delete_header`
 * Replace `this` variable with `ctx_cur`
 * Replace `this` variable with `key`
 * Replace `class` variable with `qc`
 * Replace `new` variable with `new_param`
 * Replace `new` variable with `new_label`
 * Replace `class` member with `flags`
 * Replace `not` member with `pat_not`
 * Replace `or` member with `pat_or`
 * Fix functions declarations/definitions

Fix functions declarations/definitions

As the function declaration and definition did not match up

Replace `or` member with `pat_or`

It clashes with the alternate operators

Replace `not` member with `pat_not`

It clashes with the alternate operators

Replace `class` member with `flags`

As it's reserved in C++

Replace `new` variable with `new_label`

`new` is a reserved keyword in c++

Replace `new` variable with `new_param`

`new` is a reserved keyword in c++

Replace `class` variable with `qc`

`class` is a reserved keyword in c++

Replace `this` variable with `key`

`this` is a reserved keyword in c++

Replace `this` variable with `ctx_cur`

`this` is a reserved keyword in c++

Replace `delete` variable with `delete_header`

`delete` is a reserved keyword in c++

Replace `new` variable with `new_mail`

`new` is a reserved keyword in c++

Replace `new` variable with `tnew`

`new` is a reserved keyword in c++

Replace `new` variable with `new_label`

`new` is a reserved keyword in c++

Replace `new` variable with `new_tag`

`new` is a reserved keyword in c++

Replace `new` variable with `new_mail`

`new` is a reserved keyword in c++

Replace `new` variable with `add`

`new` is a reserved keyword in c++

Unify `body_new` name instances

`new` is a reserved keyword in c++
Most instances are named this way

Replace `new` with `pnew`

`new` is a reserved keyword in c++

Replace `new` variable with `ap`

`new` is a reserved keyword in c++

Replace `new` variable with `alias`

`new` is a reserved keyword in c++

Replace `new` variable with `al_new`

`new` is a reserved keyword in c++

Replace `new` variable with `mark_new`

`new` is a reserved keyword in c++

Replace `delete` variable with `delete_original`

`delete` is a reserved keyword in c++

Replace `public` variable with `only_public_key`

`public` is a reserved keyword in c++

Replace `new` variable with`new_file`

`new` is a reserved keyword in c++

Rename both `new` and `old` to `new_file` and `old_file` for consistency

Replace `new` variable with `has_new_mail`

`new` is a reserved keyword in c++

Replace `catch` variable with `restore`

`catch` is a reserved keyword in c++

Replace `template` variable with `tmpl`

`virtual` is a reserved keyword in c++

Replace `virtual` variable with `vnum`

`virtual` is a reserved keyword in c++

Replace `try` variable with `user`

`try` is a keyword in c++

While this should not affect C code, it affects unfortunately the
tooling.

This specific instance triggers a "syntax error" in Cppcheck, so it
could hide other problems.

(Cppcheck version  1.86, code parsed with `--language=c --std=c99`)

Of course it is a bug in Cppcheck, but renaming the variable is the most
pragmatic approach, as the bug in Cppcheck does not manifest itself in
a simple program.

notify: fix leak on exit

Fix saving a msg to a non-existent mailbox

Fixes #1762

Avoid UB when clearing stdin

Flushing `stdin` is undefined behaviour.

References
 http://www.open-std.org/jtc1/sc22/wg14/www/docs/n1570.pdf, chapter 7.21.5.2 (The fflush function)
 http://en.cppreference.com/w/c/io/fflush

Sync autosetup required version with what we ship

build: refactor idn headers check

build: check which idn2 headers to include

merge: update translations

 * Various chages to spanish localization
 * Update lt_LT translations
 * modified some Japanese translations
 * sync translations

sync translations