Fix Coverity CID 198656 / 198649

fix: improve function checks

fix: stop CUR_EMAIL underflow in entire-thread

entire-thread will underflow CUR_EMAIL macro if no mailbox is
configured. To prevent this, call 'prereq()' before any other logic.
'prereq()' will ensure we have a mailbox thus preventing the underflow.

flatcap deserves credit for identifying the solution.

Fixes: #1719

Pass the test for mutt_addrlist_to_intl when building wihtout idn

merge: update transations

 * Updated Catalan translation.
 * sync translations

sync translations

Updated Catalan translation.

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

coverage: exclude 3rd party files

We don't care about system header files.

merge: Convert Address to use TAILQ

 * Define an AddressList structure using TAILQ
 * Add functions to convert to/from Address/AddressList
 * Use AddressList in mutt_addrlist_dedupe
 * Use AddressList in mutt_addr_parse_list
 * Const'ify parameter to mutt_addr_copy
 * Implement mutt_addresslist_new
 * Use AddressList in mutt_addr_copy_list
 * Use AddressList in mutt_addr_free, implement mutt_addresslist_free_one
 * Use AddressList in mutt_addr_remove_from_list
 * Use AddressList in mutt_addr_qualify
 * Use AddressList in mutt_addr_append
 * Use AddressList in mutt_addr_cmp_strict
 * Use AddressList in mutt_addr_has_recips
 * Use AddressList in mutt_addr_search
 * Use AddressList in mutt_addr_remove_xrefs
 * Tidy Address free'ing functions
 * Use AddressList in mutt_addr_write
 * Use AddressList in mutt_addrlist_to_intl and mutt_addrlist_to_local
 * Convert mutt_addr_remove_from_list and mutt_addr_remove_xrefs API to AddressList
 * Convert Alias to use AddressList
 * Convert Envelope addresses to AddressList
 * Fix variadic function
 * Fix recursive alias lookup
 * Cleanup Address API: mutt_addr_cmp_strict -> mutt_addresslist_equal
 * Cleanup Address API: parse and free
 * Cleanup Address API: mutt_addresslist_qualify
 * Cleanup Address API: mutt_addr_write
 * Fix and refactor mutt_is_list_recipient and mutt_is_subscribed_list_recipient
 * Fix match_user
 * Cleanup Address API: unexpose mutt_addr_is_(intl|local)
 * Cleanup Address API: unexpose mutt_addr_mbox_to_udomain
 * Cleanup Address API: mutt_addr_remove_from_list -> mutt_addresslist_remove
 * Cleanup Address API: mutt_addr_remove_xrefs -> mutt_addresslist_remove_xrefs
 * Cleanup Address API: unexpose mutt_addr_set_(intl|local)
 * Cleanup Address API: mutt_addrlist_dedupe -> mutt_addresslist_dedupe
 * Cleanup Address API: mutt_addrlist_to_(intl|local) -> mutt_addresslist_to_(intl|local)
 * Remove "next" pointer from struct Address: conversion to TAILQ is complete
 * Merge AddressNode into Address
 * Rename mutt_addresslist_* to mutt_addrlist_*
 * Provide mutt_addrlist_append and mutt_addrlist_prepend
 * doxy: fix comments
 * test: update basic address tests
 * remove 'mutt_' from static functions
 * Add tests for mutt_addr_cmp
 * Add tests for mutt_addr_cat, explain what it does does in the Doxygen block
 * Add tests for mutt_addr_copy
 * Add tests for mutt_addr_free
 * Add tests for mutt_addr_valid_msgid
 * Add test for mutt_addrlist_append and mutt_addrlist_prepend
 * Add tests for mutt_addrlist_copy
 * Add test for mutt_addrlist_dedupe, fix mutt_addrlist_parse
 * Run clang-format over test/address
 * Add tests for mutt_addrlist_equal
 * Add tests for mutt_addrlist_free_all
 * Add tests for mutt_addrlist_free_one
 * Add tests for mutt_addrlist_free
 * Boolify uses of Address' group
 * tidy code
 * Revert queue.h formatting
 * Rename mutt_addrlist_has_recips to mutt_addrlist_count_recips and add tests
 * Kill duplicate copyright line
 * Rename mutt_addrlist_free_all into mutt_addrlist_clear, remove freeing functions
 * Add tests for mutt_addrlist_count_recips
 * Add tests for mutt_addrlist_parse + slight refactor and fix return value
 * Slightly refactor mutt_addrlist_parse2
 * Avoid leak in the test for mutt_addrlist_parse
 * Add tests for mutt_addrlist_parse2
 * Add tests for mutt_addrlist_qualify
 * Sort address functions by name
 * Add helper to create and populate an Address, avoid leaks in tests
 * Simplify mutt_default_from
 * Add tests formutt_addrlist_remove_xrefs
 * Add tests for  mutt_addrlist_remove
 * Add (basic) tests for mutt_addrlist_to_(intl|local)
 * Add tests for mutt_addrlist_write + make it more readable
 * Fix test for mutt_addrlist_write and avoid leak
 * Add mutt_addr_to_local
 * Test addresses with comments in mutt_addrlist_parse
 * Run clang-format over test/address

Run clang-format over test/address

Test addresses with comments in mutt_addrlist_parse

Add mutt_addr_to_local

Fix test for mutt_addrlist_write and avoid leak

Add tests for mutt_addrlist_write + make it more readable

Add (basic) tests for mutt_addrlist_to_(intl|local)

Add tests for  mutt_addrlist_remove

Add tests formutt_addrlist_remove_xrefs

Simplify mutt_default_from

Add helper to create and populate an Address, avoid leaks in tests

Sort address functions by name

Add tests for mutt_addrlist_qualify

Add tests for mutt_addrlist_parse2

Avoid leak in the test for mutt_addrlist_parse

Slightly refactor mutt_addrlist_parse2

Add tests for mutt_addrlist_parse + slight refactor and fix return value

Add tests for mutt_addrlist_count_recips

Rename mutt_addrlist_free_all into mutt_addrlist_clear, remove freeing functions

No facilities are provided for the heap allocation and freeing of
AddressLists. The suggested usage is to to have AddressLists as
variables on the stack.

Kill duplicate copyright line

Rename mutt_addrlist_has_recips to mutt_addrlist_count_recips and add tests

Revert queue.h formatting

tidy code

- NULL pointers
- variable case
- rename function
- sprinkle credit

Boolify uses of Address' group

Add tests for mutt_addrlist_free

Add tests for mutt_addrlist_free_one

Add tests for mutt_addrlist_free_all

Add tests for mutt_addrlist_equal

Run clang-format over test/address

Add test for mutt_addrlist_dedupe, fix mutt_addrlist_parse

Add tests for mutt_addrlist_copy

Add test for mutt_addrlist_append and mutt_addrlist_prepend

Add tests for mutt_addr_valid_msgid

Add tests for mutt_addr_free

Add tests for mutt_addr_copy

Add tests for mutt_addr_cat, explain what it does does in the Doxygen block

Add tests for mutt_addr_cmp

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.