Add $forward_attachments quadoption for inline-forward mode

When set or answered yes, non text-decodable parts will be attached to
the new message.  The default value is "ask-yes", because I believe
this is something people will want to use, and should be made aware of
the new possible behavior.

The option presents a nice middle ground between previous
inline-forwarding behavior (where all the non-text parts were
dropped), and $mime_forward where the entire email was included as an
attachment.

This was previously difficult to achieve, but after the
recv-attachment refactoring: (a19e5266^..faabd621)
it became possible to use the ATTACH_CONTEXT and the recvattach.c
helper mutt_generate_recvattach_list() to properly deal with nesting
and multiple file-handles.

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

Add note about IMAP browser and trailing delimiters

This is the same behavior as the regular browser, but I think still
deserves a quick mention.

Thanks to Charles for pointing out this section of the manual and
suggesting the addition.

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

Enable the use of toggle-write from the pager

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>
Co-authored-by: Richard Russon <rich@flatcap.org>

Add $imap_fetch_chunk_size to allow FETCHing new headers in chunks

For extremely large mailboxes, some implementations will time out just
while fetching the new headers, because the client doesn't send any
commands for 30 minutes while downloading the large number of headers.

Rewrite imap_fetch_msn_seqset() to return chunks of size
$imap_fetch_chunk_size.

The change requires trusting the server will follow the RFC and not
send an EXPUNGE during or between the FETCH chunks; otherwise we'll
miss MSNs between the chunks because the shift.

We could in theory continue to set "msn_begin = idata->max_msn + 1",
but that makes the assumption there are no holes in the header cache
that we are filling in during a chunk.  Personally I am dubious about
"header cache holes", but the IMAP code has explicitly mentioned and
handled them since prior to my involvement.

Since the RFC forbids the interleaving EXPUNGE I believe it's safe
enough to set "msn_begin = fetch_msn_end + 1" until proven otherwise.

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

Add note about $mailcap_path generation to the documentation

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

Convert mutt_parse_adrlist() to use struct Buffer for simple parsing

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

Change mutt_parse_mailboxes() to use the path struct Buffer argument

Instead of copying to a buf[] variable, since the various utility
methods are now available to struct Buffer too.

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

Handle the same secret key appearing in multiple public keyrings

Allow for S/MIME keys and the possibility of missing subkeys

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

Revert "Fix truncation of long filenames in attachments."

This reverts commit f476d0aecd6f88db5291427fced21a8e834ca181.

The commit allowed the generation of illegal length header lines.  A
correct fix needs to implement parameter continuations.

I will work on that, but in case I don't get it in before 1.12,
preserve the hard truncation.

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

Convert remailer to use struct Buffer for Mixmaster invocation

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

Convert compress to use mutt_buffer_quote_filename()

Because the compress expandos operate differently than the rest of
mutt, requiring manual outer quoting, add a parameter to the function
to toggle outer quoting off.

Remove the now unused escape_path() function.

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

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