rename FlagChars enums

merge: nntp: refactoring

 * doc: newsrc expando examples
 * nntp: move browser code
 * nntp: tidy headers
 * nntp: move auto-complete code

nntp: move auto-complete code

nntp: tidy headers

nntp: move browser code

doc: newsrc expando examples

more Context refactoring/protection

travis: move to xenial

sync translations

imap: login on startup

This ensure we always have a valid adata.

Closes #1438

Fix unsub from the selected mailbox by just hiding it

Fixes #1460

postpone: fix Postcount when we are in current selected mailbox

postpone: Don't reopen Postponed context

If the selected Mailbox is already the Postponed one, no need to open a
new context.

This change reuse the current context.

merge: factor out mx_update_context()

 * mxapi: add Mailbox to mbox_open
 * reset message counts
 * maildir: factor out mx_update_context()
 * mbox: factor out mx_update_context()
 * pop: factor out mx_update_context()
 * nntp: factor out mx_update_context()
 * notmuch: factor out mx_update_context()
 * imap: factor out mx_update_context()

imap: factor out mx_update_context()

notmuch: factor out mx_update_context()

nntp: factor out mx_update_context()

pop: factor out mx_update_context()

mbox: factor out mx_update_context()

maildir: factor out mx_update_context()

reset message counts

mxapi: add Mailbox to mbox_open

imap: handle case where adata->mailbox is NULL

When selected mailbox in unsubscribed, adata->mailbox may be NULL.

make imap_cmd_finish() handles it.

Closes #1460

Replace git_ver.h with a combination of git_ver.{c,o}

fix crash in eat_date()

notmuch: don't overwrite nm_default_uri

In commit 1040935, I accidentally removed a check for using `Folder` as
the `db_filename` if `db_filename` is null. My change made it so that it
always overwrote the value.

merge: upstream changes

 * Add mutt_buffer_mktemp() transition function
 * Add mutt_buffer_adv_mktemp() transition function
 * Always print mutt_buffer_pool_free() size debug output
 * add Buffer fixup functions
 * Remove conversion of nbsp to space
 * Ensure a resized empty buffer is null-terminated
 * Fix memleaks of saslconn on error paths
 * Fix memory leak on error in eat_regexp() and eat_date()
 * Document send-hooks are not executed for resumed messages
 * Fix a few memory leaks for idna conversion
 * Allow relative date hour/min/sec offsets
 * Add some brief documentation on OAUTH support
 * Update $index_format cross-references in manual
 * Add L10N comments to several unclear messages
 * Switch color chunk matching to use bsearch

Switch color chunk matching to use bsearch

Emails with extremely long lines, combined with a large number of
color matches, can cause a performance hit so severe that mutt appears
to have hung.

Switching out to using a binary search speeds things up noticably.

The for loops being replaced were a bit strange.  resolve_types()
ensures there are no overlaps, and that they are ordered.  A match is
defined by (first <= cnt < last).  I can't see any reason for the
strange comparison of (cnt > last) follow by (cnt != last).  Perhaps
at one time it made sense.

Add L10N comments to several unclear messages

Updating Japanese translation, I found that some of the updated messages
were a little hard to understand without reading the code.

Add comments to some messages I nearly translated in a wrong way.

Update $index_format cross-references in manual

Add index-format-hook to the list of hooks.

Add the list of configuration variables that use the expandos.

Add a note to $post_index_string that it uses the expandos.

Add some brief documentation on OAUTH support

These are just pulled from Brandon's commit message.

Allow relative date hour/min/sec offsets

These might be useful for index-format-hook pattern matching for some
people.

Fix a few memory leaks for idna conversion

Document send-hooks are not executed for resumed messages

Fix memory leak on error in eat_regexp() and eat_date()

Fix memleaks of saslconn on error paths

If mutt_sasl_client_new returns an error, the callers would ignore the
allocated saslconn resource from sasl_client_new. Be sure to release
these with sasl_dispose as documented in sasl.h. Likewise, let callers
(POP/IMAP) dispose the resource on their error paths. SMTP was already
taken care of. Found with LeakSanitizer in IMAP.

Ensure a resized empty buffer is null-terminated

The new buffer code is using the pool, which ensures its buffers are
null-terminated.

However, if a "new" buffer from another part of the code were passed
to one of the temporary interfaces that resizes the buffer, it's
possible a non-terminated string might end up being passed through.

It's reasonable to expect mutt_b2s() for a "new" buffer should be the
same as after it is resized larger.  So ensure the resulting buf->data
is properly terminated to avoid surprises.

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