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>

Fix send-mode printing when expand_filename() returns 1

It will return this when a nametemplate entry already matched the
passed in filename.  So this bug required both a print entry and an
already matching nametemplate entry to trigger.

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

Rename mutt_buffer_rfc1524_expand_command()

Remove buffer prefix now that all callers are converted.

Convert rfc1524_expand_command() implementation to use struct Buffer

Add mutt_buffer_sanitize_filename() helper.

Add a few end-of-buffer checks while iterating over command.

Convert the parameter name, paramater value, and types to use struct Buffer
instead of fixed size strings.

Convert last rfc1524_expand_command caller to use a Buffer

merge: upstream fixes

 * Improve imap_append_message() error message handling
 * Mention sources for ~p and ~P patterns
 * Allow imap_cmd_finish() to both expunge and fetch new mail
 * Improve $reverse_realname documentation
 * Add $fcc_before_send, defaulting unset

Add $fcc_before_send, defaulting unset

When set, the message will be Fcc'ed the same as sent.  $fcc_clear and
$fcc_attach will be ignored.  This is because of the difficulty of
unwinding changes, notably Protected Headers, without potentially
breaking signatures.

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

Improve $reverse_realname documentation

Make it clear that, even if set, a missing realname part of the
matching address will be filled in by $realname.

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

Allow imap_cmd_finish() to both expunge and fetch new mail

Since commit dd327606 changed check_status setting to use bit
operators, and imap_check_mailbox() can call imap_cmd_finish() twice,
there is no reason to delay the processing of new mail until a second
call.

imap_read_headers() deals with msn_end < msg_begin, so remove
the (count > idata->max_msn) check.  This will allow the reopen flag
to be reset if somehow it's not the case.

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

Mention sources for ~p and ~P patterns

The manual only mentioned alternates.  Add $from and local
account/hostname to the list.

The man page implied the only source was alternates.  Change to the
same wording as the manual.

Thanks to @rear1019 for pointing out the misleading man page.

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

Improve imap_append_message() error message handling

If the rc is IMAP_CMD_BAD, then either idata->buf is stale or an error
message has already been printed (in cmd_handle_untagged()).

Use imap_next_word() to skip over the next two words instead of
directly skipping over SEQLEN, in case the buffer is in a different
format.  We don't want to jump over the end of string.

Skip the mutt_error() if there is nothing to print.

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

test: fixes for recent updates

merge: trivial fixes

 * initialise all struct members
 * boolify Pattern.ready
 * docs: fix typo
 * fix #includes
 * unify GroupList variable naming
 * doxy: tidy compare_uid
 * move mutt_str_split() to list.c
 * reorder params for mutt_str_substr_copy()
 * rename el_free()

rename el_free()

Rename to mutt_emaillist_free() to match other functions.

reorder params for mutt_str_substr_copy()

- rename substr_cpy() to substr_copy()
- reorder params to put dest and destlen together

move mutt_str_split() to list.c

Remove List dependency from string.c

doxy: tidy compare_uid

unify GroupList variable naming

fix #includes

docs: fix typo

boolify Pattern.ready

initialise all struct members

drop mutt_dump_variables()

This function has been replaced by dump_config().