remove 'mutt_' from static functions

test: update basic address tests

doxy: fix comments

Provide mutt_addrlist_append and mutt_addrlist_prepend

These are one-liners that call TAILQ_INSERT_TAIL and TAILQ_INSERT_HEAD,
but the macros have the ugly side-effect of evaluating their arguments
multiple times, so TAILQ_INSERT_TAIL(list, new_element()); doesn't do
what you would expect.

Rename mutt_addresslist_* to mutt_addrlist_*

Merge AddressNode into Address

Remove "next" pointer from struct Address: conversion to TAILQ is complete

Cleanup Address API: mutt_addrlist_to_(intl|local) -> mutt_addresslist_to_(intl|local)

Cleanup Address API: mutt_addrlist_dedupe -> mutt_addresslist_dedupe

Cleanup Address API: unexpose mutt_addr_set_(intl|local)

Cleanup Address API: mutt_addr_remove_xrefs -> mutt_addresslist_remove_xrefs

Cleanup Address API: mutt_addr_remove_from_list -> mutt_addresslist_remove

Cleanup Address API: unexpose mutt_addr_mbox_to_udomain

Cleanup Address API: unexpose mutt_addr_is_(intl|local)

Fix match_user

Fix and refactor mutt_is_list_recipient and mutt_is_subscribed_list_recipient

Cleanup Address API: mutt_addr_write

Cleanup Address API: mutt_addresslist_qualify

Cleanup Address API: parse and free

Cleanup Address API: mutt_addr_cmp_strict -> mutt_addresslist_equal

Fix recursive alias lookup

Fix variadic function

Convert Envelope addresses to AddressList

Convert Alias to use AddressList

Convert mutt_addr_remove_from_list and mutt_addr_remove_xrefs API to AddressList

Use AddressList in mutt_addrlist_to_intl and mutt_addrlist_to_local

Use AddressList in mutt_addr_write

Tidy Address free'ing functions

Use AddressList in mutt_addr_remove_xrefs

Use AddressList in mutt_addr_search

Use AddressList in mutt_addr_has_recips

Use AddressList in mutt_addr_cmp_strict

Use AddressList in mutt_addr_append

Use AddressList in mutt_addr_qualify

Use AddressList in mutt_addr_remove_from_list

Use AddressList in mutt_addr_free, implement mutt_addresslist_free_one

Use AddressList in mutt_addr_copy_list

Implement mutt_addresslist_new

Const'ify parameter to mutt_addr_copy

Use AddressList in mutt_addr_parse_list

Use AddressList in mutt_addrlist_dedupe

Add functions to convert to/from Address/AddressList

Define an AddressList structure using TAILQ

Avoid redundant sort in entire-thread.

918885f added, indirectly, a mutt_sort_headers() call to
nm_read_entire_thread() (via mutt_mailbox_changed() ->
ctx_mailbox_changed() -> ctx_update()), but didn't remove the sort
after nm_read_entire_thread() in mutt_index_menu().

Remove the now superfluous sort.

entire-thread: determine current position before resorting.

Previously oldcur was determined after nm_read_entire_thread(),
leading to entire-thread unintentionally jumping to the first message
in the thread.

Before 918885f32db41 that used to work, because mutt_sort_headers()
wasn't called from within nm_read_entire_thread().

Fix by moving determination of pre nm_read_entire_thread() count a few
lines up. Also reuse existing oldcount variable, rather than
'oc' (which could also be oldcur).

Fixes: #1714

fix: reverse_name / alternates

Fixes: #1581

merge: minor tidying

 * clang-format
 * readme: split out contributors
 * rename fd parameters
 * increase buffer sizes to avoid truncation
 * test: add missing #include guards
 * unify char buffer inititialisation
 * boolify: locals, params, args
 * rename mailbox functions
 * light tidying
 * don't check for empty config strings
 * rename struct Event and Mailbox.notify
 * move el_free() to email
 * tidy maildir_mbox_check()
 * config: rename listener to observer
 * tidy ternary operators
 * tidy char pointers
 * initialise pointers
 * tidy pointer definitions

tidy pointer definitions

Separate multiple variable definitions which differ in type, e.g.

  char buf[16], *p = NULL, **pp = NULL;

becomes

  char buf[16];
  char *p = NULL;
  char **pp = NULL;

initialise pointers

tidy char pointers

Make char-pointer dereferencing more consistent.
Any functions that used both `*ptr` and `ptr[0]` notation have been tidied.

tidy ternary operators

- insert ()s
- expand checks against 0 or '\0'

config: rename listener to observer

This fits in with the GoF's subject/observer pattern.

tidy maildir_mbox_check()

Factor out a common pointer.

move el_free() to email

This function will be used elsewhere, soon.

rename struct Event and Mailbox.notify

Rename things out of the way of future work.
These names will be used by a general notifications system.

don't check for empty config strings

The config system stores empty strings as NULL.
Config variables must not be set directly, but using the config system
(this gives us validation and notifications).

light tidying

Lots of minor fixes, many inspired by cppcheck.

- reduce scope of variables
- return early
- NULL pointers after free
- fix typos
- correct retval type

rename mailbox functions

Fix mailbox function naming for consistency

boolify: locals, params, args

unify char buffer inititialisation

For consistency, use the format:

  `char buf[16] = { 0 };`

test: add missing #include guards

increase buffer sizes to avoid truncation

This is just to hide a compiler warning.

rename fd parameters

Rename parameters to match other functions.

readme: split out contributors

clang-format

merge: split up Buffer and the Pool

 * split out pool
 * don't use pool in libraries
 * test: tidy config tests

test: tidy config tests

The config tests don't use the buffer pool any more.

don't use pool in libraries

split out pool

fix leaks in mutt_pattern_comp()

test: fix leaks

merge: upstream fixes

 * Fix sample neomuttrc to use better quoting practices.
 * Add a "backticks in double quotes" example to the manual.
 * Avoid undefined behaviour on huge integer in a RFC2231 header
 * sync pgp_gpgme_invoke_import()

sync pgp_gpgme_invoke_import()

Avoid undefined behaviour on huge integer in a RFC2231 header

The atoi() function was called on the index, which can potentially
be huge in an invalid message and can yield undefined behavior. The
mutt_atoi() function is now used for error detection.

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

Add a "backticks in double quotes" example to the manual.

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

Fix sample neomuttrc to use better quoting practices.

Use single quotes in the password encryption example.

For password decryption, put the backquotes inside double quotes to
avoid special characters being re-interpreted.

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

Do not cache a compiled regex to avoid leaks

glibc's implementation of the regex engine is a DFA which creates nodes
on the fly under the regex_t object. This will cause leakes if we keep
the compiled regular expression around, as it was the case before.

See also https://sourceware.org/bugzilla/show_bug.cgi?id=12567

fix: counting of content-lines header

merge: fix most leaks in test code

 * check library parameters more carefully
 * test: fix leaks in config tests
 * test: fix leaks in library tests
 * clang-format test code

clang-format test code

test: fix leaks in library tests

test: fix leaks in config tests

check library parameters more carefully

Some test cases were causing library code to be run with invalid
parameters.  Reject these cases.

Fix next-unread-mailbox

e4133e5cd2 have introduced a small regression.

next-unread-mailbox does not automatically open the mailbox.

This change fixes it.

merge: update translations

 * Update lt_LT translation
 * update German translation
 * fix command string
 * fix 'too few/many' translations
 * tidy OAUTHBEARER error message
 * Update Ukrainian translation
 * Update Russian translation
 * Update Danish translation
 * sync translations

sync translations

Update Danish translation

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

Update Russian translation

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

Update Ukrainian translation

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

tidy OAUTHBEARER error message

fix 'too few/many' translations

fix command string

update German translation

Update lt_LT translation

Fixes 6 fuzzy strings.

feat: use mailbox description for spoolfile

'spoolfile' is limited to mailboxes paths so if one wants to use a
'named-mailboxes' or 'virtual-mailboxes' as the spoolfile, its path must
be duplicated in the 'spoolfile' assignment. 'virtual-mailboxes' has a
helper variable, 'virtual_spoolfile', for selecting the first
'virtual-mailboxes' entry as the spoolfile.

This commit allows one to use a mailbox description for the 'spoolfile,
which removes the need to duplicate a 'named-mailboxes' or
'virtual-mailboxes' path and makes 'virtual_spoolfile' defunct. The
configuration syntax is now more consistent.

It is important to note that mailbox descriptions are preferred to
paths.

To reflect the change, documentation was updated:
 - Add note to 'spoolfile' about using a string to specify a
   'named-mailboxes' or 'virtual-mailboxes' description.
 - Add a note to the 'virtual_spoolfile' documentation saying that it is no
   longer necessary since 'spoolfile' supports descriptions now.

merge: upstream fixes

 * Add a test for NULL parameter attribute too
 * Add a quick-starter config section to the manual.
 * Add sample starter muttrc to contrib.
 * Change sample muttrc path to match other manual references.
 * Add note about %r for the pgp_list_*_command vars.

Add note about %r for the pgp_list_*_command vars.

In this case %r is the list of search strings.

Change sample muttrc path to match other manual references.

The other parts of the manual refer to /usr/local/share/doc in their
examples.  For consistency, change the starter muttrc and manual entry
to do the same.

In the future, we may wish to have a config variable instead, so users
don't have to worry about system vs locally installed paths when
sourcing the gpg.rc, for instance.

Add sample starter muttrc to contrib.

Add a mini-section with a link to the manual instead, under
"Configuration".

Add a quick-starter config section to the manual.

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