test: improve test_mutt_str_remall_strcasestr()

test: improve test_mutt_str_pretty_size()

test: improve test_mutt_str_next_word()

test: improve test_mutt_str_lws_rlen()

test: improve test_mutt_str_lws_len()

test: improve test_mutt_str_is_email_wsp()

test: improve test_mutt_str_is_ascii()

test: improve test_mutt_str_inline_replace()

test: improve test_mutt_str_getenv()

test: improve test_mutt_str_find_word()

test: improve test_mutt_str_dequote_comment()

test: improve test_mutt_str_asprintf()

test: improve test_mutt_str_append_item()

test: improve test_mutt_str_adjust()

merge: improve Buffer tests

 * test: improve test_mutt_buffer_add_printf()
 * test: improve test_mutt_buffer_addch()
 * test: improve test_mutt_buffer_addstr()
 * test: improve test_mutt_buffer_addstr_n()
 * test: improve test_mutt_buffer_alloc()
 * test: improve test_mutt_buffer_concat_path()
 * test: improve test_mutt_buffer_fix_dptr()
 * test: improve test_mutt_buffer_free()
 * test: improve test_mutt_buffer_from()
 * test: improve test_mutt_buffer_increase_size()
 * test: improve test_mutt_buffer_init()
 * test: improve test_mutt_buffer_is_empty()
 * test: improve test_mutt_buffer_len()
 * test: improve test_mutt_buffer_new()
 * test: improve test_mutt_buffer_printf()
 * test: improve test_mutt_buffer_reset()
 * test: improve test_mutt_buffer_strcpy()
 * test: improve test_mutt_buffer_strcpy_n()
 * reorganise buffer header

reorganise buffer header

test: improve test_mutt_buffer_strcpy_n()

test: improve test_mutt_buffer_strcpy()

test: improve test_mutt_buffer_reset()

test: improve test_mutt_buffer_printf()

test: improve test_mutt_buffer_new()

test: improve test_mutt_buffer_len()

test: improve test_mutt_buffer_is_empty()

test: improve test_mutt_buffer_init()

test: improve test_mutt_buffer_increase_size()

coverity

test: improve test_mutt_buffer_from()

move the null logger into libmutt

This logger is used by the test code (for quietness).

test: improve test_mutt_buffer_free()

remove unnecessary test in mutt_mem_calloc()

There's no point in making a calculation.
Either calloc() succeeds, or it doesn't.

test: improve test_mutt_buffer_fix_dptr()

fix idn macro

gcc9 isn't happy with pre-proc conditionals inside a macro.

test: improve test_mutt_buffer_concat_path()

test: improve test_mutt_buffer_alloc()

test: improve test_mutt_buffer_addstr_n()

test: improve test_mutt_buffer_addstr()

test: improve test_mutt_buffer_addch()

test: improve test_mutt_buffer_add_printf()

Fix Coverity CID 198652

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