libmutt: fix buffer code for degenerate cases

libmutt: fix base64 code for degenerate cases

test: add url tests for degenerate cases

test: templates for url functions

test: add thread tests for degenerate cases

test: templates for thread functions

test: add tags tests for degenerate cases

test: templates for tags functions

test: add rfc2231 tests for degenerate cases

test: templates for rfc2231 functions

test: add rfc2047 tests for degenerate cases

test: templates for rfc2047 functions

test: add parse tests for degenerate cases

test: templates for parse functions

test: add parameter tests for degenerate cases

test: templates for parameter functions

test: add from tests for degenerate cases

test: templates for from functions

test: add envelope tests for degenerate cases

test: templates for envelope functions

test: add email tests for degenerate cases

test: templates for email functions

test: add body tests for degenerate cases

test: templates for body functions

test: add attach tests for degenerate cases

test: templates for attach functions

test: add signal tests for degenerate cases

test: templates for signal functions

test: add sha1 tests for degenerate cases

test: templates for sha1 functions

test: add regex tests for degenerate cases

test: templates for regex functions

test: add memory tests for degenerate cases

test: templates for memory functions

test: add md5 tests for degenerate cases

test: templates for md5 functions

test: add mbyte tests for degenerate cases

test: templates for mbyte functions

test: add mapping tests for degenerate cases

test: templates for mapping functions

test: add logging tests for degenerate cases

test: templates for logging functions

test: add list tests for degenerate cases

test: templates for list functions

test: add history tests for degenerate cases

test: templates for history functions

test: add hash tests for degenerate cases

test: templates for hash functions

test: add file tests for degenerate cases

test: templates for file functions

test: add envlist tests for degenerate cases

test: templates for envlist functions

test: add date tests for degenerate cases

test: templates for date functions

test: add charset tests for degenerate cases

test: templates for charset functions

test: add buffer tests for degenerate cases

test: templates for buffer functions

test: add base64 tests for degenerate cases

test: templates for base64 functions

merge: trivial tidying

 * add address/lib.h to .clang-format
 * doxy: tidy/standardise lots of comments
 * docs: tidy guide
 * test scalars against zero
 * boolify variables/parameters
 * rename variables/parameters
 * reduce scope of variables

reduce scope of variables

- Move variable declarations closer to their first use
- Move parameter checks as early as possible in functions

rename variables/parameters

Rename variables for consistency or clarity.

boolify variables/parameters

test scalars against zero

- test int/size_t against `0`
- test char against `'\0'`
- test enum against 'zero' symbol

docs: tidy guide

- Add missing whitespace
- Remove unnecessary backslashes

doxy: tidy/standardise lots of comments

add address/lib.h to .clang-format

Do not rely on idn2 to provide an idn1 compat layer

Do not rely on idn2 to provide a compat layer to idn1, as Ubuntu doesn't
provide it. Instead, really code address/idna.c to use the different
APIs provided by the two versions of libidn. Systems with an old version
of idn2 which does not provide the needed functionality will fail at
configure time now, and users of such systems will see an error message
with instructions how to build with libidn1. While at it, polish a bit
the auto.def sections for idn and idn2.

Fix getting back to index after saving an edited message

Fixes #1674

build: fix a bunch of idn2-related problems

- hide code if IDN is disabled
- fix build tests
- add travis dependency

merge: add basic address tests

 * test: templates for address functions
 * test: add address tests for degenerate cases
 * libaddress: fix address code for degenerate cases
 * test: templates for group functions
 * test: add group tests for degenerate cases
 * libaddress: fix group code for degenerate cases
 * test: templates for idna functions
 * test: add idna tests for degenerate cases

test: add idna tests for degenerate cases

test: templates for idna functions

libaddress: fix group code for degenerate cases

test: add group tests for degenerate cases

test: templates for group functions

libaddress: fix address code for degenerate cases

test: add address tests for degenerate cases

test: templates for address functions

keybindings: restore nntp change-newsgroup keybindings

This restores a subset of the keybindings found in the original vvv.nntp
patch that don't clash with any of neomutt defaults.

Specifically:
  i to change-newsgroup
  ESC-i to change-newsgroup-readonly

Add RECURSIVEMATCH LIST-EXTENDED selection option to query

LSUB is required to include mailboxes with subscribed children, but
the just added "LIST (SUBSCRIBED)" by default does not do that.

To match previous behavior, add RECURSIVEMATCH to make sure the
children are included.  Fix the parser to trim off the CHILDINFO
suffix in the response.

Parse \NonExistent LIST attribute the same as \NoSelect.

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

Improve hierarchy information accuracy in IMAP browser

Currently the IMAP browser relies on LIST and LSUB (for listing subscribed
folders) which may not provide the required hierarchy information.

RFC3348 section 3 goes as far as stating that a client mustn't rely on LSUB
for hierarchy information.

This patch implements the LIST command extensions specified in RFC5258
requiring that a server must respond with hierarchy information for
listed folders (whether or not filtering on subscribed folders).

merge: add string tests

 * test,tidy: mutt_str_sysexit()
 * test,tidy: mutt_str_atos()
 * test,tidy: mutt_str_atoi()
 * test,tidy: mutt_str_atol()
 * test,tidy: mutt_str_atoui()
 * test,tidy: mutt_str_atoul()
 * test,tidy: mutt_str_atoull()

tidy: mutt_str_atoull()

test: mutt_str_atoull()

tidy: mutt_str_atoul()

test: mutt_str_atoul()

tidy: mutt_str_atoui()

test: mutt_str_atoui()

tidy: mutt_str_atol()

test: mutt_str_atol()

tidy: mutt_str_atoi()

test: mutt_str_atoi()

tidy: mutt_str_atos()

test: mutt_str_atos()

tidy: mutt_str_sysexit()

test: mutt_str_sysexit()

imap: fixing race when syncing mailbox

imap_check_mailbox return an detailled state not only 0/-1.

imap_sync_mailbox assumes that the function was returning only 0/-1.

This change fixes that.

browser: show "Subscribed" in IMAP browser