factor out Context from mx_toggle_write()

factor out Context from link_threads()

factor out Context from mutt_messages_in_thread()

factor out Context from mbox_open_append()

factor out Context from check_subjects()

factor out Context from make_subj_hash()

factor out Context from find_subject()

factor out Context from mutt_newsgroup_uncatchup()

factor out Context from mutt_newsgroup_catchup()

factor out Context from mx_msg_commit()

move Context flags to Mailbox

light tidying

factor out Context from mx_tags_is_supported()

factor out Context from tags_edit()

factor out Context from tags_commit()

factor out Context from msg_padding_size()

notmuch: factor out Context

Use Mailbox instead of Context

This aims to remove Context from mutt_set_flag, to do so other function
signature have been changed too.

* mutt_set_flag_update
* mx_msg_open
* mx_msg_open_new
* mx_msg_close
* mutt_pattern_exec
* mutt_parse_mime_message
* mutt_set_header_color
* mutt_count_body_parts

move ctx->tagged to mailbox->msg_tagged

imap: remove unused define

Keep adata on imap-logout-all

The former call to `imap_adata_free((void **) adata);` caused an account to be around with no adata associated with it. This caused a crash when trying to access the account later on.

merge: imap: refactor status functions

 * imap: remove useless static int queued in imap_status()
 * imap: remove MESSAGES count optimization
 * imap: merge all STATUS function

imap: merge all STATUS function

This change creates only one STATUS function.

imap: remove MESSAGES count optimization

This optimization is buggy, it puts sometimes mdata->messages out of
sync with m->msg_count;

This optimization links in an not obvious way the cmd_parse_status and
mx_mailbox_check().

This change removes this useless optimization and just always asks for
MESSAGES during STATUS.

imap: remove useless static int queued in imap_status()

Only imap_mbox_open() passes true to queue parameter of imap_status().
imap_mbox_open() takes care of flushing/droppping the imap pipeline
comamnds.

So we don't need this extra check about queued STATUS.

This change removes it.

imap: move flags in ImapMboxData

Flags are a per mailbox thing.
So moves in ImapMboxData.

imap: reduce complexity of imap_exec

imap_exec flag IMAP_CMD_FAIL_OK is the only one that configure the
function return instead of the function behavior.

Also returned value of the function depends on the flag make hard to
understand what happen after imap_exec() is run.

This change removes this flags, instead we always returns the detailed
returns code, the caller have to deal with it.

Return code are always IMAP_EXEC_SUCCESS/IMAP_EXEC_ERROR/IMAP_EXEC_FATAL and
depends on if the command succeed, failure or if the imap connection
fail.

ssl: don't shutdown broken connection

Closes #1440

account.login is empty, use account.user instead

imap: tidy doc

merge: trivial fixes

 * pop: remove sleeps on error paths
 * nntp: reorg cleanup
 * fix coverity defects
 * fix typos
 * mailbox tidy
 * boolify
 * rename vars
 * close attachments before closing logging
 * mbox: refactor for clarity

mbox: refactor for clarity

close attachments before closing logging

rename vars

boolify

mailbox tidy

fix typos

fix coverity defects

nntp: reorg cleanup

pop: remove sleeps on error paths

imap: remove ImapCommandType

ImapAccountData->cmdtype can only be IMAP_CT_LIST
cmddata can only be set to an ImapList.

This change removes cmdtype and rename void *cmddata to ImapList
*cmdresult.

ImapStatus doesn't make sense since we have ImapMboxData.

Some attributes are duplicated a synced between strucuture.

This change moves all STATUS informations in ImapMboxData
and remove ImapStatus structure.

mboxcache is also removed because ImapMboxData stay in memory during the
whole Mailbox life, no more need to cache that.

nntp: remove unnec context reference

imap: move some adata attributes to mdata

nntp: fix url usage

fix leak in imap_status()

fix leak in imap_ac_add()

imap: fix browser

Avoid segfault when an IMAP connection cannot be opened (II)

Part II of 6a2e9f, which broke edit-msg. It turns out that mx_ac_add()
is not required to be successful, e.g., when opening a temporary file.
This means we must be extra cautious and check that m->account->adata is
valid before using it

Make sure error buffer is properly initialized

merge: update translations

 * Update zh_CN translation
 * Update Czech translation
 * More translations
 * Update lt_LT translations
 * Update Polish translation
 * update German translation
 * fix Polish format spec
 * Update lt_LT translation
 * Update Czech translation
 * Update Slovak translation
 * fixing typos in Polish translation
 * update filenames
 * sync translations

sync translations

update filenames

fixing typos in Polish translation

Update Slovak translation

Update Czech translation

Update lt_LT translation

(I accidentally re-translated lt.po in master, and when I tried to
cherry-pick my translation onto the translate branch I noticed that I'd
translated a couple of messages incorrectly.)

fix Polish format spec

update German translation

Update Polish translation

Update lt_LT translations

New messages, new fuzz, some old typos.

More translations

Update Czech translation

Update zh_CN translation

merge: trivial fixes

 * doxygen: sort out Email descriptions
 * boolify params/retvals
 * imap: simplify some helper functions
 * remove unused member
 * imap: tiny refactor
 * doxygen: unify docs for private data
 * rename: _create to _new for simple allocs

rename: _create to _new for simple allocs

doxygen: unify docs for private data

imap: tiny refactor

remove unused member

imap: simplify some helper functions

boolify params/retvals

doxygen: sort out Email descriptions

clang-format

rename: curs_main.c to index.c

merge: Simplify URL APIs

 * Simplify URL APIs
 * Remove unused U_DECODE_PASSWD flag
 * Add a basic unit test for URL
 * Convert NotMuch to the new URL API
 * Additional improvements to URL API and implementation
 * Additional tests for URL
 * Avoid flexible array members, improve URL test code

Avoid flexible array members, improve URL test code

Additional tests for URL

Additional improvements to URL API and implementation

Convert NotMuch to the new URL API

Add a basic unit test for URL

Remove unused U_DECODE_PASSWD flag

Simplify URL APIs

imap: ensure we can't delete the current selected mailbox

Closes #1417

imap: add missing conn_account free

Avoid segfault when an IMAP connection cannot be opened

This could be reproduced trying to open an IMAPS connection with a
NeoMutt built without SSL support.

merge: remove ImapMbox

 * imap: make imap_mailbox_check aware of Mailbox
 * imap: setup a mdata struct for each mailboxes
 * imap: remove ImapMbox from imap_prepare_mailbox
 * imap: remove ImapMbox from imap_adata_find signature
 * imap: remove get_mailbox()
 * imap: remove ImapMbox from imap_fast_trash()
 * imap: remove ImapMbox from imap_ac_add()
 * imap: remove ImapMbox from imap_mbox_open()
 * imap: remove ImapMbox from imap_delete_mailbox()
 * imap: remove ImapMbox from imap_adata_find()
 * imap: remove ImapMbox from imap_append_message()
 * imap: remove ImapMbox from imap_copy_messages()
 * imap: remove ImapMbox from imap_hcache_open()
 * imap: remove ImapMbox from imap_browse()
 * imap: remove useless allocation in imap_mailbox_create()
 * imap: create mdata in imap_prepare_mailbox()
 * imap: remove ImapMbox from cmd_parse_status()
 * imap: remove ImapMbox from add_folder()
 * imap: remove ImapMbox from browse_add_list_result()
 * imap: remove ImapMbox from imap_pretty_mailbox()
 * imap: remove ImapMbox and imap_parse_path()
 * imap: remove now useless param of imap_prepare_mailbox
 * imap: remove imap_parse_path2() from imap_hcache_open()
 * imap: remove imap_parse_path2() from imap_rename_mailbox()
 * imap: remove imap_parse_path2() from imap_append_message()
 * imap: always parse account in imap_parse_path2()
 * imap: remove old PINE url format
 * imap: handle all mailbox name with ImapMailboxData struct
 * imap: ensure account flags are zero'ed
 * imap: Check m->has_new to increase MailboxCount
 * imap: rename imap_parse_path2() imap_parse_path()
 * imap: rename imap_qualify_path2() to imap_qualify_path()
 * imap: rename ImapMailboxData to ImapMboxData

imap: rename ImapMailboxData to ImapMboxData

imap: rename imap_qualify_path2() to imap_qualify_path()

imap: rename imap_parse_path2() imap_parse_path()

imap: Check m->has_new to increase MailboxCount

imap: ensure account flags are zero'ed

imap: handle all mailbox name with ImapMailboxData struct

imap: remove old PINE url format

This format is broken since a while (when we started to use url_parse
to get the mailbox name).

This change removes it.

imap: always parse account in imap_parse_path2()

imap: remove imap_parse_path2() from imap_append_message()

imap: remove imap_parse_path2() from imap_rename_mailbox()

imap: remove imap_parse_path2() from imap_hcache_open()

imap: remove now useless param of imap_prepare_mailbox

imap: remove ImapMbox and imap_parse_path()

imap: remove ImapMbox from imap_pretty_mailbox()