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().

build: fix doc auto-generation

build: update to latest autosetup

This version includes the location of the Tcl interpreter used at
configure time in config.log.

merge: overhaul of the config system

 * config: allow deeper inheritance
 * tests: minor fixes
 * tidy type definitions
 * move private flags
 * move sensitive flag to type
 * tidy init.h config definitions
 * move redraw flags
 * rename the flags field to data (and move it)
 * drop dt_path
 * drop dt_command
 * drop mutt dump
 * add enum type
 * add slist type
 * mbox_type -> enum
 * drop magic type
 * hidden_tags -> slist
 * mailcap_path -> slist
 * imap_authenticators -> slist
 * smtp_authenticators -> slist
 * pop_authenticators -> slist
 * preferred_languages -> slist
 * fix makedoc

fix makedoc

Fix the `makedoc` documentation generator to understand the new types.

preferred_languages -> slist

Convert `$preferred_languages` to use the new DT_SLIST type (string list).

pop_authenticators -> slist

Convert `$pop_authenticators` to use the new DT_SLIST type (string list).

smtp_authenticators -> slist

Convert `$smtp_authenticators` to use the new DT_SLIST type (string list).

imap_authenticators -> slist

Convert `$imap_authenticators` to use the new DT_SLIST type (string list).

mailcap_path -> slist

Convert `$mailcap_path` to use the new DT_SLIST type (string list).

hidden_tags -> slist

Convert `$hidden_tags` to use the new DT_SLIST type (string list).

drop magic type

The DT_MAGIC type is unnecessary; it's an enumeration of four strings.
That's better handled by the new DT_ENUM type.

mbox_type -> enum

Convert `$mbox_type` to use the new Config enumeration.

add slist type

Add a string list type.  This manages strings delimited by space, comma
or colon.  The NeoMutt code doesn't need to parse the string any more,
it's simply presented with a STAILQ (list) of strings.

add enum type

Create a new config type for enumerations.
This will use the `ConfigDef.data` field and an `EnumDef` to define the
allowed strings.

drop mutt dump

Drop the legacy config dump code.
It's less versatile than the new code.

drop dt_command

The DT_COMMAND type is unnecessary; it behaves exactly the same as DT_STRING.
Only the NeoMutt code needs to make the distinction, so use a subclass
of DT_STRING instead of a separate type.

drop dt_path

The DT_PATH type is unnecessary; it behaves exactly the same as DT_STRING.
Only the NeoMutt code needs to make the distinction, so use a subclass
of DT_STRING instead of a separate type.

rename the flags field to data (and move it)

Deprecate the flags field of ConfigDef and create a data field.
The data field is inserted near the end of the struct as it won't be
used often.

move redraw flags

Move the redraw flags from a general flags field to the type field.
These GUI flags don't belong in the config definition, but once the
notification system is complete, the flags will be obsolete.

The `ConfigDef.flags` field is now unused.

tidy init.h config definitions

Remove all the excess whitespace.
The fields may once have been aligned, but was long ago.

move sensitive flag to type

Rename F_SENSITIVE to DT_SENSITIVE.
Move it from a general flags field, to the type field (with the other
type modifying flags).

move private flags

Move the private config flags to the 'top' of the variable.

tidy type definitions

Unify the flag definitions using the (1 << n) form.

tests: minor fixes

Fix:
- Misleading output
- printf specifiers
- Missing test cases

config: allow deeper inheritance

The code assumed that an inherited HashElem had a normal HashElem
parent.  This change removes the assumption, allowing deep nesting of
Config Items.

merge: trivial fixes

 * fix folder use in mx_path_canon()
 * doxy: fix typos
 * rename local variable for clarity
 * test: tidy code
 * refactor #ifdef'd conditional
 * check includes with iwyu

check includes with iwyu