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()

imap: remove ImapMbox from browse_add_list_result()

imap: remove ImapMbox from add_folder()

imap: remove ImapMbox from cmd_parse_status()

imap: create mdata in imap_prepare_mailbox()

Hidden mailbox does not call mx_ac_add(). So this change
moves the ImapMailboxData initialisation in imap_prepare_mailbox().

This ensures we always have a valid m->mdata;

imap: remove useless allocation in imap_mailbox_create()

imap: remove ImapMbox from imap_browse()

imap: remove ImapMbox from imap_hcache_open()

imap: remove ImapMbox from imap_copy_messages()

imap: remove ImapMbox from imap_append_message()

imap: remove ImapMbox from imap_adata_find()

imap: remove ImapMbox from imap_delete_mailbox()

imap: remove ImapMbox from imap_mbox_open()

imap: remove ImapMbox from imap_ac_add()

imap: remove ImapMbox from imap_fast_trash()

imap: remove get_mailbox()

imap: remove ImapMbox from imap_adata_find signature

imap: remove ImapMbox from imap_prepare_mailbox

imap: setup a mdata struct for each mailboxes

This change introduces `struct ImapMailboxeData` to mainly store
the mbox name in all its form.

Next change will use this names instead of parsing again and again the
path.

At this end we should be able to drop ImapMbox and imap_parse_path().

imap: make imap_mailbox_check aware of Mailbox

With multiple imap account not loaded at startup time, when we switch to
another account, stats are almost not updated. If check_stats if true at
this time by change, mutt crash. Because we unexpectly update the
msg_unread of Context->mailbox.

Imap is the only backend that check stats is done for all backends at
once.

This change makes it more like other backends, and it make all imap
mailboxes stats to be loaded on next check_stats=true without crash.

merge: refactor Context/Mailbox usage

 * pop: Mailbox refactor
 * mbox: Mailbox refactor
 * maildir: Mailbox refactor
 * nntp: Mailbox refactor
 * notmuch: Mailbox refactor
 * compress: Mailbox refactor
 * imap: Mailbox refactor
 * flags: Mailbox refactor
 * mx: Mailbox refactor
 * threads: Mailbox refactor
 * open_append: remove Context
 * msg_close: remove Context
 * msg_commit: remove Context
 * msg_padding_size: remove Context
 * tags_edit: remove Context
 * mxapi: unify retval of msg_open
 * path_canon: drop folder

path_canon: drop folder

mxapi: unify retval of msg_open

tags_edit: remove Context

msg_padding_size: remove Context

msg_commit: remove Context

msg_close: remove Context

open_append: remove Context

threads: Mailbox refactor

mx: Mailbox refactor

flags: Mailbox refactor

imap: Mailbox refactor

compress: Mailbox refactor

notmuch: Mailbox refactor

nntp: Mailbox refactor

maildir: Mailbox refactor

mbox: Mailbox refactor

pop: Mailbox refactor

Do not create an additional ctx for syncing a mailbox

When the additional ctx is closed, the mailbox is free'd too, which
causes a crash back in the index.

merge: add mutt_str_startswith API

 * Fix NONULL so it's usable in expressions such as `!*NONULL(str)`
 * Add mutt_str_startswith API
 * mutt_str_startswith - bcache.c
 * mutt_str_startswith - browser.c
 * mutt_str_startswith - color.c
 * mutt_str_startswith - config/sort.c
 * mutt_str_startswith - copy.c
 * mutt_str_startswith - email/from.c
 * mutt_str_startswith - email/idna.c
 * mutt_str_startswith - email/parse.c
 * mutt_str_startswith - handler.c
 * mutt_str_startswith - imap/auth_sasl.c
 * mutt_str_startswith - imap/browse.c
 * mutt_str_startswith - imap/command.c
 * mutt_str_startswith - imap/imap.c
 * mutt_str_startswith - imap/message.c
 * mutt_str_startswith - imap/util.c
 * mutt_str_startswith - init.c
 * mutt_str_startswith - maildir/mh.c
 * mutt_str_startswith - main.c
 * mutt_str_startswith - mbox/mbox.c
 * mutt_str_startswith - mutt/envlist.c
 * mutt_str_startswith - mutt/path.c
 * mutt_str_startswith - mutt_lua.c
 * mutt_str_startswith - muttlib.c
 * mutt_str_startswith - ncrypt/crypt_gpgme.c
 * mutt_str_startswith - ncrypt/pgp.c
 * mutt_str_startswith - nntp/nntp.c
 * mutt_str_startswith - pager.c
 * mutt_str_startswith - pop/pop_auth.c
 * mutt_str_startswith - pop/pop_lib.c
 * mutt_str_startswith - postpone.c
 * mutt_str_startswith - rfc3676.c
 * mutt_str_startswith - sendlib.c
 * mutt_str_startswith - mutt/date.c
 * mutt_str_startswith - mutt/list.c
 * mutt_str_startswith - mutt_header.c
 * mutt_str_startswith - config/sort.c - fix logic
 * mutt_str_startswith - ncrypt/crypt.c
 * mutt_str_startswith - ncrypt/smime.c
 * mutt_str_startswith - nntp/nntp.c
 * mutt_str_startswith - notmuch/mutt_notmuch.c
 * mutt_str_startswith - pop/pop_lib.c
 * mutt_str_startswith - postpone.c
 * mutt_str_startswith - rfc1524.c
 * Remove unneeded strlen
 * mutt_str_startswith - send.c
 * mutt_str_startswith - sendlib.c
 * mutt_str_startswith - smtp.c
 * mutt_str_startswith - terminal.c
 * mutt_str_startswith - browser.c (leftover)
 * mutt_str_startswith - mbox/mbox.c
 * mutt_str_startswith - mutt/charset.c
 * mutt_str_startswith - mutt/pop.c
 * mutt_str_startswith - rfc1524.c

mutt_str_startswith - rfc1524.c

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