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.

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