mutt_str_startswith - mutt/pop.c

mutt_str_startswith - mutt/charset.c

mutt_str_startswith - mbox/mbox.c

mutt_str_startswith - browser.c (leftover)

mutt_str_startswith - terminal.c

mutt_str_startswith - smtp.c

mutt_str_startswith - sendlib.c

mutt_str_startswith - send.c

Remove unneeded strlen

mutt_str_startswith - rfc1524.c

mutt_str_startswith - postpone.c

mutt_str_startswith - pop/pop_lib.c

mutt_str_startswith - notmuch/mutt_notmuch.c

mutt_str_startswith - nntp/nntp.c

mutt_str_startswith - ncrypt/smime.c

mutt_str_startswith - ncrypt/crypt.c

mutt_str_startswith - config/sort.c - fix logic

mutt_str_startswith - mutt_header.c

mutt_str_startswith - mutt/list.c

mutt_str_startswith - mutt/date.c

mutt_str_startswith - sendlib.c

mutt_str_startswith - rfc3676.c

mutt_str_startswith - postpone.c

mutt_str_startswith - pop/pop_lib.c

mutt_str_startswith - pop/pop_auth.c

mutt_str_startswith - pager.c

mutt_str_startswith - nntp/nntp.c

mutt_str_startswith - ncrypt/pgp.c

mutt_str_startswith - ncrypt/crypt_gpgme.c

mutt_str_startswith - muttlib.c

mutt_str_startswith - mutt_lua.c

mutt_str_startswith - mutt/path.c

mutt_str_startswith - mutt/envlist.c

mutt_str_startswith - mbox/mbox.c

mutt_str_startswith - main.c

mutt_str_startswith - maildir/mh.c

mutt_str_startswith - init.c

mutt_str_startswith - imap/util.c

mutt_str_startswith - imap/message.c

mutt_str_startswith - imap/imap.c

mutt_str_startswith - imap/command.c

mutt_str_startswith - imap/browse.c

mutt_str_startswith - imap/auth_sasl.c

mutt_str_startswith - handler.c

mutt_str_startswith - email/parse.c

mutt_str_startswith - email/idna.c

mutt_str_startswith - email/from.c

mutt_str_startswith - copy.c

mutt_str_startswith - config/sort.c

mutt_str_startswith - color.c

mutt_str_startswith - browser.c

mutt_str_startswith - bcache.c

Add mutt_str_startswith API

The function mutt_str_startswith(str, p, cs) checks whether str starts
with p. The case-sensitivity of the comparison is done according to cs.

Fix NONULL so it's usable in expressions such as `!*NONULL(str)`

Avoid closing a mailbox prematurely

fix some msg_new update

This changes fixes some case where mailboxe->msg_unread is ++/-- without
changing mailbox->msg_new.

imap: simplify getter around ImapAccountData

Remove now useless attributes of imap_prepare_mailbox()
Rename imap_ac_data_find() to imap_adata_find() and move it in util.c
aside other adata related function.
Parse path within ImapMbox in imap_adata_find() to remove some common
code.
Create ImapAccountData resource during imap_ac_add()

Make sure imap_subscribed is always called with an expanded path

imap: fix imap list subscribe

mutt_list_find doesn't modify the list, so it should take a const*

Centralize Context cleanup in a dedicated function

Fix adding / removing mailboxes

Do not duplicate mailboxes in the AllMailboxes list. Reset Context when
the last mailbox goes away.

Fix pointer dereference

Simplify account cleanup

Always do mailbox_check() the first time we see a mailbox

When multiple accounts are used, stats are all 0 until the next
MailCheckStatsInterval is reached.

This change does an initial check when we see a mailbox for the first
time.

Also delete account if needed during unmailboxes

When we do:

    mailboxes imaps://foo
    unmailboxes *
    mailboxes imaps://foo

* account->mailboxes is not cleaned and have pointer to freed mailboxes
* account is still there even if not used

This changes cleanup account->mailboxes and account if needed.

Avoid FREE'ing a non-malloc'd block

The mx.mbox member variable could remain uninitialized if imap_prepare_mailbox
failed, for example because a mailbox in the list doesn't have an account
associated with it.

remove mailbox.closing

mx_mbox_close() does not always put back mailbox.closing to false.

This make next imap_sync_mailbox() during mx_mbox_sync() closing the
imap connection when it should not.

Since this attribute is only used by imap backend, this change removes
it and handle the imap specific shutdown within the imap backend.

move Context.new (leftover)

move Context.deleted

merge: manage lifespan of mailcap tmp files

 * Implement mailcap option x-neomutt-keep
 * Unlink temporary attachment files on mutt_timeout_hook
 * doc: document 'x-neomut-keep'
 * Delete temporary file upon signal reception

Delete temporary file upon signal reception

doc: document 'x-neomut-keep'

Unlink temporary attachment files on mutt_timeout_hook

Implement mailcap option x-neomutt-keep

The option x-neomutt-keep instructs neomutt not to remove the attachment
after the view-command has quit. This is helpful with mime types handled
by commands such as firefox, which fork before opening the attachment in
the main instance. Example:

application/pdf; evince; x-neomutt-keep;

merge: imap: remove imap_conn_find

 * remove imap_conn_find in cmd_handle_fatal
 * remove imap_conn_find in imap_mbox_open_append
 * remove imap_conn_find from imap_delete_mailbox
 * remove imap_conn_find from imap_subscribe
 * remove imap_conn_find from imap_browse
 * remove imap_conn_find from imap_completes
 * replace all imap_conn_find with MUTT_IMAP_CONN_NONEW
 * remove unused imap_conn_find
 * rename imap_conn_find2 to imap_login

imap: rename imap_conn_find2 to imap_login

imap: remove unused imap_conn_find

imap: replace all imap_conn_find with MUTT_IMAP_CONN_NONEW

imap: remove imap_conn_find from imap_completes

imap: remove imap_conn_find from imap_browse

imap: remove imap_conn_find from imap_subscribe

imap: remove imap_conn_find from imap_delete_mailbox

imap: remove imap_conn_find in imap_mbox_open_append

All code using imap_conn_find is currently broken.

This change is the first part of moving out imap_conn_find.

To do so, a new function mail_prepare_mailbox() will setup the
ImapAccountData structure.

This method will be used in imap_access()/imap_mbox_open()/imap_mbox_open_append().

imap: remove imap_conn_find in cmd_handle_fatal

imap_conn_find() is broken and always return NULL if MUTT_IMAP_CONN_NONEW
flag is passed.

This change removes it from cmd_handle_fatal().

merge: notmuch refactoring

 * replace hard-coded protocol with constant
 * replace hard-coded maildir flags
 * remove nm_path_probe(...) calls
 * single point for closing database
 * remove all unneeded #ifdef USE_NOTMUCH
 * remove nm_normalize_uri(...)
 * move version-specific loading to one spot

notmuch: move version-specific loading to one spot

Moved loading of messages and threads to into their own method,
respectively. This helper method abstracts the loading for the caller.
All version-specific calls are in one spot for easier maintenance.

notmuch: remove nm_normalize_uri(...)

`nm_normalize_uri(...)` had one usage in `mutt_parse_unmailboxes(...)`.
Removing it had no affect on the `unmailboxes` command. I was able to
remove mailboxes without error.

notmuch: remove all unneeded #ifdef USE_NOTMUCH

Removed all unnecessary #ifdef USE_NOTMUCH. The remaining #ifdef
USE_NOTMUCH statements gate notmuch-specific calls that are not covered
by an interface.

The notmuch implementation of `mx_path_canon(...)` has been replaced with a
call to `nm_normalize_uri(...)` since that already canonizes notmuch
queries.

notmuch: single point for closing database

There are three points where a database is closed. Each point duplicates
the code for different versions of notmuch. Implemented a new method for
the version-specific notmuch code is handled. This method takes in a
database and closes it.

notmuch: remove nm_path_probe(...) calls

Replaced `nm_path_probe(...)` with `mx_path_probe(...)` to make notmuch
be completely compliant with the MX api.

notmuch: replace hard-coded maildir flags

Replaced the hard-coded maildir flags unread, flagged, and replied with
variables NmUnreadTag, NmFlaggedTag, and NmRepliedTag.

Since unread and flagged may be different from the maildir flag, we do
not want to limit to using "unread", "flagged", or "unread" when
modifying tags. The user can specify which tags they use for the maildir
flags, and the subsystem will update maildir flags correctly.

notmuch: replace hard-coded protocol with constant

Removed all instances of `notmuch://` with a constant, NmUriProtocol.
This makes it easier to change to protocol if necessary.

clang-format

imap: when opening mailbox, reset Mailbox attrs

We currently only reset vcount and msg_count.

But imap_read_headers() call from imap_mbox_open() reload all
messages and then call mx_update_context(). So attributes that have not
been reseted will grow from previous values.

This does the reset of size, msg_unread, msg_flagged, msg_new.

Remove unused ImapAccountData->account

move Context.new

flags: change confusing var name

mailbox->msg_flagged was stored in mailbox variable.

This change rename mailbox to flagged.

mailbox: drop unnec function

Fixes #1388

Thanks to @alyssais and @puckipedia.

mxapi: mx_path_canon should not double / uri

At least for imap if "folder=imap://example.com" and mailboxes contains "=Foo"

mx_path_canon will construct imap://example.com//Foo

This change ensures we do not add the extra /.