Delete Attachments upon OP_QUIT

use 'enum MailboxType magic' consistently

pop: tidy

Do not use disconnected socket.

If an IMAP connection was never set up, do not try to write a LOGOUT
command through that connection or to close that socket.

Both results in segmentation faults.

Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>

mbox: recognise empty file

path: fix expansion of bare + and =

imap: home is now '+'

imap: check for null connection

imap: fix subscription list

trans: fix source list

move notmuch test macro

merge: refactor backends to use Account object

 * clang-format
 * rename Email private data
 * rename Mailbox private data
 * misc
 * add Account
 * mxapi: unify naming
 * mxapi: add Account
 * mxapi: add Mailbox to mbox_open
 * mxapi: refactoring
 * Notmuch: add Account
 * Notmuch: move private data
 * Notmuch: unify naming
 * Notmuch: move db to Account
 * Notmuch: separate db functions
 * Maildir: add Account
 * Maildir: unify naming
 * Maildir: move private data
 * Pop: add Account
 * Pop: unify naming
 * Pop: move data to Account
 * Compress: unify naming
 * Compress: add Account
 * Nntp: add Account
 * Nntp: unify naming
 * Nntp: add new/free/get functions
 * Mbox: add Account
 * Mbox: unify naming
 * Mbox: move data to Account
 * Imap: tidy
 * Imap: add Account
 * Imap: use Mailbox ptr
 * Imap: unify naming
 * Imap: refactor current message
 * Imap: move Account data

Imap: move Account data

Imap: refactor current message

Imap: unify naming

Imap: use Mailbox ptr

Imap: add Account

Imap: tidy

Mbox: move data to Account

Mbox: unify naming

Mbox: add Account

Nntp: add new/free/get functions

Nntp: unify naming

Nntp: add Account

Compress: add Account

Compress: unify naming

Pop: move data to Account

Pop: unify naming

Pop: add Account

Maildir: move private data

Maildir: unify naming

Maildir: add Account

Notmuch: separate db functions

Notmuch: move db to Account

Notmuch: unify naming

Notmuch: move private data

Notmuch: add Account

mxapi: refactoring

mxapi: add Mailbox to mbox_open

mxapi: add Account

mxapi: unify naming

add Account

misc

rename Mailbox private data

rename Email private data

clang-format

Revert "notmuch: count messages in queried threads"

The change was causing severe performance problems with large mailboxes.

This reverts commit b71ac2bae5d471b2a32452d9103d4164f27f729f.

Issue #1359

rename nntp structs

Try adding status indicator R for my address in Reply-To

It adds a new "R" flag for the 3rd position of the index %Z field, for when user's own address is in the Reply-To header unless none of the other flags for that field is applicable.

Some mailing lists now do the following transformation of the headers: rewrite the From header to the list address, and add a Reply-To header with the poster address.  This feature helps highlight my own posts in folders dedicated to such lists.  Of course if you are _subscribed_ to the list (in the Mutt sense) you'll get the "L" flag instead, but in that case you don't care about hightlighting your posts anyway.

rename mbox_get_mdata

add pop_get_mdata

manual: document `named-mailboxes` for neomuttrc

add MaildirMboxData

fix some mailbox logic

merge: add <named-mailboxes> command

 * mailbox: try to find existing description
 * format: add `%D` - mailbox description
 * Add `named-mailboxes` command
 * browser: `%i` - folder_format description expando

browser: `%i` - folder_format description expando

`%i` will display the description for a folder and fallback to the
folder's name if a description does not exist.

Add `named-mailboxes` command

No additional work is necessary since this command reuses the same code
as `mailboxes` and `virtual-mailboxes`. This was surprisingly simple.

format: add `%D` - mailbox description

Added the `%D` expando, which displays the mailbox's description, to
sidebar_format and status_format. If the mailbox does not have a
description, fallback to the path.

This behavior is same as `%f` and `%B` for `virtual-mailboxes`. This
was done to prevent breaking existing configurations as `notmuch`
queries have unsightly paths.

This lays the ground work for introducing `named-mailboxes`.

mailbox: try to find existing description

`mx_mbox_open(...)` creates a new `Context`, which requires creating new
`Mailbox` instances. Modified `mailbox_new(...)` to try and find an
existing description that matches the mailbox `path`.

fix mailbox crashes

Fixes #1360
Fixes #1361

Thanks to Sergey Alirzaev (@l29ah)

doxy: fix docs

[ci skip]

rename config Account

test: drop duplicate account code

use mutt_buffer_strcpy

notmuch: count messages in queried threads

`count_query` did not handle counting all messages in `type=threads`
queries. It returned the number of messages matching the `query=`
qualifier. Refactored `count_query` into two methods handling
`type=threads` and `type=messages`, respectively.

This makes `sidebar_format`'s `%S` display the same value as
`pager_format`'s `%m`.

imap: factor out get_adata

fix imap log - buffer_add_printf

tidy code

rename things

fix comments

merge: upstream fixes

 * Fix gnutls (a)lways to properly save for all certerr values.
 * Change gnutls certificate_file processing and interactive prompts.
 * Add a comment about gnutls date bits in certstat.

Add a comment about gnutls date bits in certstat.

It's easy to miss the call disabling date checking and wonder why the
certstat bits are not set.

Fix IMAP auth w/o IR to not hang on failed authentication

Change gnutls certificate_file processing and interactive prompts.

Accept a saved cert at any point in the chain, except for those
revoked or with invalid dates.

The INSECUREALG processing was a bit goofed up.  It was skipping the
prompt for every chain except the host, and was only displaying an
error for entry 1 (the host being 0, root being len-1).  Instead
prompt at any point, and display a warning as with the other errors.

Add a CERTERR_OTHER type to capture other bits.  Recently GnuTLS has
added additional certificate_status_t types.  We may want to improve
the prompts in the future for those, but for now at least make sure
it's recorded in certerr.

Fix gnutls (a)lways to properly save for all certerr values.

For the case of SIGNERNOTCA, INSECUREALG, or a newer unhandled value,
the "(a)ccept always" prompt was allowed, but the cert saving was
prevented by a check only against NOTTRUSTED.  This ended up giving a
strange error message saying the cert was not saved.

Fix to save the cert for all errors except HOSTNAME (which is handled
separately).

merge: upstream refactoring

 * Add buffer pool functions.
 * Convert imap_rename and pgp_invoke_list_keys to use buffer pool.
 * Create mutt_buffer_add_printf().
 * Convert mix_send_message to use buffer pool.
 * Convert buffy_maildir_check_dir to use struct Buffer.
 * Convert maildir_parse_dir to use struct Buffer.
 * Convert _maildir_commit_message to use struct Buffer.
 * Convert maildir_sync_message to use BUFFERs.
 * Convert maildir_canon_filename() and callers to use struct Buffer.

Fix segfault when checking a mailbox after a disconnect (III)

Issue #1349

tidy

Convert maildir_canon_filename() and callers to use struct Buffer.

Callers are maildir_check_mailbox(), _maildir_open_find_message(), and
maildir_open_find_message().

Convert maildir_sync_message to use BUFFERs.

Convert _maildir_commit_message to use struct Buffer.

Convert maildir_parse_dir to use struct Buffer.

Note: the uses of buf and tmp don't overlap, so we only use a single
struct Buffer.

Convert buffy_maildir_check_dir to use struct Buffer.

Convert mix_send_message to use buffer pool.

Create mutt_buffer_add_printf().

Change mutt_buffer_printf() to overwrite the buffer contents, like
mutt_buffer_strcpy().

mutt_buffer_add*() functions append to the buffer contents.

Convert imap_rename and pgp_invoke_list_keys to use buffer pool.

Add buffer pool functions.

Also add a few helper functions: mutt_buffer_clear(),
mutt_buffer_strcpy(), and a macro mutt_b2s to grab the buffer data as
a const char *.

ImapAccountData

Even though it's attached to a Mailbox, this is Account data.

notmuch: use global settings for vfolder-from-query

notmuch: simplify uri creation logic

If `mdata` does not exist in `mailbox`, try to create a default instance
of `NmMboxData`. Using a default instance removes two conditionals from
`nm_uri_from_query(...)`, moving them to `nm_get_default_data(...)`.
This reduces the number of `snprintf(...)` to maintain, and allows query
parsing for type meta-data.

An extra variable keeps track of default data usage for freeing
allocated memory.

notmuch: use global settings for vfolder-from-query

When the <vfolder-from-query> command is run while already in a vfolder,
it takes the current vfolder as context.  That means the global settings
(nm_query_type, nm_db_limit, nm_default_uri) are ignored in favor of the
current vfolder's settings.  This is confusing behavior because the new
vfolder is otherwise independent from the old one.

Set the nm_uri_from_query() context to NULL so that it uses the global
settings instead.

fix index allocation

The `vcount` wasn't being reset, so the index overflowed later.

notmuch: fix variable used after its scope

The progress bar keeps a reference to `msgbuf`.

notmuch: reconstruct thread on-the-fly upon entire-thread and no vfolder

Before this change <entire-thread> worked only if invoked from a virtual
folder.

Now <entire-thread> works also from regular folders, with the following
semantics:

- a notmuch search is done on the fly using as query the message-id of the
  currently selected message

- mutt switches to the query result (a vfolder)

- <entire-thread> is invoked as before, expanding the thread of the original
  message

Closes #1345

merge: trivial fixes

 * clang-format
 * cppcheck: reduce variable scope
 * scan-build: fix errors
 * replace strncpy with strfcpy

replace strncpy with strfcpy

strncpy isn't guaranteed to NUL-terminate strings

scan-build: fix errors

cppcheck: reduce variable scope

merge: notmuch: Allow type= statements in vfolder-from-query

 * Implement case-insensitive substring search
 * Implement substring removal function
 * Testing: mutt_str_strcasestr
 * Notmuch: Parse `type=` statements from query

Closes #1214
Closes #1319

Notmuch: Parse `type=` statements from query

Implemented a function that will parse `type=` statements from a notmuch
query and set the appropriate query type in `NmCtxData`.

This change allows a user to change the query type for a single query
without affecting other queries. The primary use is for executing
`vfolder-from-query`.

This commit ensures notmuch integration behavior conforms to NeoMutt's
notmuch documentation.