Convert pager help string to use struct Buffer

Remove the awkward string truncation warnings, and remove helpstr and
tmphelp char arrays from the stack.

Because the pager is fairly long lived, allocate the helpstr instead
of using the pool.

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

Convert certficate prompts to use menu dialog helper and struct Buffer

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

Add menu dialog helper to add rows

Remove the manual max calculation and dialog row allocation.

Add a NONULL check because the helper uses safe_strdup() to add a row.

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

Convert migrated lib.c functions to use struct Buffer

Fix mutt_getcwd() to return NULL on failure.

Change mutt_rmtree(), mutt_mkwrapdir(), safe_open(), and
safe_symlink() to use struct Buffer so they don't have filename length
limitations.

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

Increase prompt size to remove warning

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

Convert change folder operations to use struct Buffer

Store the folder name inside a struct Buffer and use the various struct Buffer
enhanced functions.

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

Add mutt_buffer_enter_fname()

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

Convert other users of struct Mailbox->pathbuf to use Buffers

A few functions in browser.c, buffy.c, and monitor.c were using
struct Mailbox->pathbuf but were potentially truncating via fixed size buffers.
Convert those to use Buffers too.

buffy_get() was creating epath and expanding it, apparently to match
against expanded struct Mailbox list entries, was wasn't using the epath.  I
believe this is a bug, and have switched the comparison to epath.

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

Convert struct Buffer->path to a struct Buffer

Rename to struct Buffer->pathbuf to make it clear the field is a struct Buffer, and
to make sure to catch and review all usages.

There are still uses of pathbuf that are truncating and need to be
fixed, for example in browser.c and buffy.c.

Fix up sidebar usage in one place, that was pointing inside the
struct Mailbox->path with a char *.  At the same time, change their "short
folder" computation to also use a struct Buffer.

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

Change struct Mailbox->realpath to be const char *

struct Mailbox->path is a fixed array (which will be converted to a struct Buffer in
the next commit).  This is needed to call mutt_expand_path().

However, struct Mailbox->realpath has no such need, and so it is a bit
wasteful (not to mention not big enough) to store as such.

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

Fix utf7/utf8 conversion of mailbox names in LIST response

Because of a wrong argument passed into `imap_unmunge_mbox_name`, mailbox names
encoded in utf7 are not properly converted into utf8. This causes two problems:

1. mailbox names are mangled in the UI (M&AOY-lstr&APY-m instead of Mælström)

2. mailbox names are re-encoded to utf7 before being sent to the server,
resulting in "Mailbox does not exists" errors in commands such as DELETE

Avoid identity conversion

Avoid useless reallocations in IMAP utf8/utf7 conversion

merge: unit test updates

 * test: quieten successful tests
 * test: merge tests
 * build: disable testing by default

build: disable testing by default

Change `configure` to not build the unit tests by default.

```
--testing   Enable Unit Testing
--coverage  Enable Coverage Testing (enables Unit Testing)
```

Co-authored-by: Pietro Cerutti <gahr@gahr.ch>

test: merge tests

test: quieten successful tests

Fix crash when trying to collapse a limit not matching any messages

This is a follow-up of #1685

Do not link in a new Account after a failed login

Fixes #1690

merge: move existing tests

 * test: move base64 tests
 * test: move file tests
 * test: move rfc2047 tests
 * test: move md5 tests
 * test: move url tests
 * test: move address tests
 * test: move buffer tests

test: move buffer tests

test: move address tests

test: move url tests

test: move md5 tests

test: move rfc2047 tests

test: move file tests

test: move base64 tests

merge: upstream Buffer changes

 * Change autoview_handler() to use struct Buffer
 * Convert mutt_check_simple() to accept a struct Buffer parameter
 * Add mutt_buffer_quote_filename()
 * Convert mutt_expand_file_fmt() to accept struct Buffer dest parameter
 * Convert mutt_get_tmp_attachment to use struct Buffer
 * Convert mutt_compose_attachment to use struct Buffer
 * Convert mutt_edit_attachment to use struct Buffer
 * Convert mutt_view_attachment to use struct Buffer
 * Convert mutt_print_attachment to use struct Buffer
 * Convert mutt_expand_fmt() and rfc1524_expand_filename() to struct Buffer
 * Rename to mutt_rfc1524_expand_filename()
 * Convert remaining mutt_adv_mktemp() calls to use buffer version
 * Rename mutt_buffer_adv_mktemp()
 * test: update for buffer changes

test: update for buffer changes

Rename mutt_buffer_adv_mktemp()

Remove buffer prefix since all callers now use this version.

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

Convert remaining mutt_adv_mktemp() calls to use buffer version

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

Rename to mutt_rfc1524_expand_filename()

Now that all callers pass a struct Buffer, the buffer prefix is not needed to
distinguish the parameter.

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

Convert mutt_expand_fmt() and rfc1524_expand_filename() to struct Buffer

These two functions were tied together and so are converted at the
same time.

Note that rfc1524_expand_filename() had an off-by-one error for the
left hand size of the name template.  It was only copying i-1 instead
of the i characters before %s.

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

Convert mutt_print_attachment to use struct Buffer

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

Convert mutt_view_attachment to use struct Buffer

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

Convert mutt_edit_attachment to use struct Buffer

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

Convert mutt_compose_attachment to use struct Buffer

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

Convert mutt_get_tmp_attachment to use struct Buffer

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

Convert mutt_expand_file_fmt() to accept struct Buffer dest parameter

mutt_expand_fmt() will be converted in the next commit, at the same
time as rfc1524_expand_filename().

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

Add mutt_buffer_quote_filename()

Convert almost all the callers to use the new function.  alias.c usage
is a bit involved, so leave that for now.

Remove unneeded index while converting from mutt_quote_filename().

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

Convert mutt_check_simple() to accept a struct Buffer parameter

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

Change autoview_handler() to use struct Buffer

This will allow the full conversion of rfc1524_expand_filename(), and
thereafter mutt_adv_mktemp() too.

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

merge: fix bind/macro translations

 * icommands.c: Unify error messages about temp file creation
 * icommands.c: Add hints about '%s' for the translators
 * icommands.c: I18N: Fix piecemeal constructed translation string
 * po: Update PO files due to icommands.c change

po: Update PO files due to icommands.c change

icommands.c: I18N: Fix piecemeal constructed translation string

Do not translate individual parts of a message and then combine them in
a printf-style way. This might work for English but miserably fails for
other languages. Instead use two or more complete sentences and pass
each of the sentences to the translator.

Fixes issue #1680

icommands.c: Add hints about '%s' for the translators

Add comments which describe what the '%s' parameter in the translation
string is.

icommands.c: Unify error messages about temp file creation

Unify all the messages stating that the temporary file could not be
created to include the filename of the temporary file.

Respect collapse_all after limit (#1685)

Issue #1667

Fix nullptr dereference

Fixes #1659

merge: add basic library tests

Add tests for:
- attach    functions
- base64    functions
- body      functions
- buffer    functions
- charset   functions
- date      functions
- email     functions
- envelope  functions
- envlist   functions
- file      functions
- from      functions
- hash      functions
- history   functions
- list      functions
- logging   functions
- mapping   functions
- mbyte     functions
- md5       functions
- memory    functions
- parameter functions
- parse     functions
- regex     functions
- rfc2047   functions
- rfc2231   functions
- sha1      functions
- signal    functions
- tags      functions
- thread    functions
- url       functions

test: add missing include

test: fix compound initialization

test: fix compound initialization

libemail: fix url code for degenerate cases

libemail: fix thread code for degenerate cases

libemail: fix rfc2231 code for degenerate cases

libemail: fix rfc2047 code for degenerate cases

libemail: fix parse code for degenerate cases

libemail: fix parameter code for degenerate cases

libemail: fix envelope code for degenerate cases

libemail: fix body code for degenerate cases

libemail: fix attach code for degenerate cases

libmutt: fix sha1 code for degenerate cases

libmutt: fix regex code for degenerate cases

libmutt: fix memory code for degenerate cases

libmutt: fix md5 code for degenerate cases

libmutt: fix mbyte code for degenerate cases

libmutt: fix logging code for degenerate cases

libmutt: fix list code for degenerate cases

libmutt: fix history code for degenerate cases

libmutt: fix hash code for degenerate cases

libmutt: fix file code for degenerate cases

libmutt: fix envlist code for degenerate cases

libmutt: fix date code for degenerate cases

libmutt: fix charset code for degenerate cases

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