Remove conversion of nbsp to space

Keep the behavior of it not line-breaking, but don't change it to a
space character when displaying.  This is so copy-paste from the pager
will preserve the nbsp, which is semantically important in some
locales.

add Buffer fixup functions

This is split out of another upstream commit

Always print mutt_buffer_pool_free() size debug output

So I can easily see the resulting size of the pool at the end of a
run.

Add mutt_buffer_adv_mktemp() transition function

Add mutt_buffer_mktemp() transition function

This is self-contained and easy to translate, so instead just create
an alternate implementation using struct Buffer.

clang-format

Rename arguments of mutt_file_read_line for consistency

Add more convenient functions to map file lines

check for Context

merge: doc: fixes

 * doc: correct pattern asterisk counts
 * doc: use non-breaking spaces before footnote asterisks
 * doc: add or remove missing or excess spacing respectively

doc: add or remove missing or excess spacing respectively

Signed-off-by: Joey Pabalinas <joeypabalinas@gmail.com>

doc: use non-breaking spaces before footnote asterisks

Signed-off-by: Joey Pabalinas <joeypabalinas@gmail.com>

doc: correct pattern asterisk counts

The patterns ~b, ~B, and ~h read each message in and should have 4
asterisks instead of 3 (3 refers to patterns with message number range
arguments). Add the missing asterisk to these patterns.

Signed-off-by: Joey Pabalinas <joeypabalinas@gmail.com>

doc: change instances of GMail to Gmail for consistency

Signed-off-by: Joey Pabalinas <joeypabalinas@gmail.com>

split up maildir/mh code

move timespec functions to library

How to run shell commands in global-hooks

Addition is based on issue https://github.com/neomutt/neomutt/issues/1353

factor out Context from maildir functions

move backend-specific code

imap: handle EXPUNGE and EXISTS during one pipeline

Receiving EXPUNGE, VANISHED and EXISTS during the same imap pipelined
command is perfectly valid.

Currently when EXPUNGE/VANISHED is handled we ignore EXISTS, making
msn_index not updated. Then MUTT_REOPEN refreshes the mailbox where last
received messages are missing.

Don't open the same mailbox twice

doxygen: fix comments

[ci skip]

merge: factor out Context

 * notmuch: factor out Context
 * factor out Context from msg_padding_size()
 * factor out Context from tags_commit()
 * factor out Context from tags_edit()
 * factor out Context from mx_tags_is_supported()
 * light tidying
 * move Context flags to Mailbox
 * factor out Context from mx_msg_commit()
 * factor out Context from mutt_newsgroup_catchup()
 * factor out Context from mutt_newsgroup_uncatchup()
 * factor out Context from find_subject()
 * factor out Context from make_subj_hash()
 * factor out Context from check_subjects()
 * factor out Context from mbox_open_append()
 * factor out Context from mutt_messages_in_thread()
 * factor out Context from link_threads()
 * factor out Context from mx_toggle_write()
 * factor out Context from mutt_score_message()
 * factor out Context from maildir_update_flags()
 * factor out Context from mutt_check_rescore()
 * factor out Context from mutt_prepare_template()
 * factor out Context from append_message()
 * factor out Context from mutt_append_message()
 * factor out Context from mutt_set_flag_update()
 * factor out Context from mutt_copy_message_ctx()
 * factor out Context from mutt_save_message_ctx()
 * factor out Context from mutt_make_message_attach()
 * factor out Context from trash_append()
 * factor out Context from edit_or_view_one_message()

factor out Context from edit_or_view_one_message()

factor out Context from trash_append()

factor out Context from mutt_make_message_attach()

factor out Context from mutt_save_message_ctx()

factor out Context from mutt_copy_message_ctx()

factor out Context from mutt_set_flag_update()

factor out Context from mutt_append_message()

factor out Context from append_message()

factor out Context from mutt_prepare_template()

factor out Context from mutt_check_rescore()

factor out Context from maildir_update_flags()

factor out Context from mutt_score_message()

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