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

modified some Japanese translations

Update lt_LT translations

One fuzzy down.

Various chages to spanish localization

Optionally use pkg-config for 3rd party dependencies

Add a new --pkgconf switch to configure.
When specified, the compiler and linker flags for most 3rd party dependencies
will be figured out using pkg-config.

Clarify header_cache behaviour on missing paths

Fixes #1752

merge: upstream fixes

 * Convert last rfc1524_expand_command caller to use a Buffer
 * Convert rfc1524_expand_command() implementation to use struct Buffer
 * Rename mutt_buffer_rfc1524_expand_command()
 * Fix send-mode printing when expand_filename() returns 1
 * Fix compose and edit attachment symlink failure code
 * Remove unnecessary strcmp for mutt_view_attachment()
 * Remove mutt_rfc1524_expand_filename() return value and checks
 * Make sure mailcap test %s is sanitized
 * Clean up mutt_print_attachment() cleanup
 * Add NULL checks to rfc1524_free_entry()
 * Detail the documentation of %l for index_format

Detail the documentation of %l for index_format

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

Add NULL checks to rfc1524_free_entry()

The existing code was fine, but make it robust like other free
functions in mutt, so the behavior isn't surprising.

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

Clean up mutt_print_attachment() cleanup

Check if mutt_save_attachment() fails and abort the print in that
case.

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

Make sure mailcap test %s is sanitized

It's not clear to me if %s is allowed as part of a test field.
However since we are passing the attachment filename, we should
sanitize it first.

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

Remove mutt_rfc1524_expand_filename() return value and checks

The return value was of dubious value.  It returned 0 only for the
case that a nametemplate was specified and it already matched the
supplied oldfile.

However, just because the nametemplate matched does not mean
attachment handling in send-mode should skip the mutt_adv_mktemp()
conversion, which includes a call to mutt_sanitize_filename().  We
didn't do so if *no* nametemplate was supplied.

Remove the return value from the function, and remove the checks and
"special handing" in mutt_attach.c calls.

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

Remove unnecessary strcmp for mutt_view_attachment()

mutt_rfc1524_expand_filename() runs the result through
mutt_adv_mktemp(), which will sanitize and relocate the filename under
$tmpdir.  The strcmp() is unneeded and distracting to the program
logic; none of the other routines perform or need this check.

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

Fix compose and edit attachment symlink failure code

In the case where safe_symlink() fail, mutt prompts to continue, but
did not properly reset the filename to be operated on.

Fix up mutt_view_attachment() to use the same flow as the others, to
allow for easier comparison.

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