Updated Esperanto translation.

Minor touchups to documentation and UPDATING file.

Reword some of the UPDATING entries.

Thanks to Matthias Andree for his feedback and suggestions!

Add 1.8.0 entries to the UPDATING file.

docs: fix typo in notmuch example

style: don't align assignments -- too often ugly

handle sigint within socket operations (#411)

When trying to use mutt with non-SSL IMAP it intermittently throws
'Error talking to localhost (Interrupted system call)' and closes
the mailbox.

Closes #407
Closes #408
Closes #411

Use getaddrinfo_a if possible (#420)

When getaddrinfo_a is avialable we can use it to set a timeout for the
dns resolution. This avoid to have mutt that freeze on startup when the
network connection isn't working and the hosts/hostname misconfigured on
the system.

Closes #176
Closes #420

Remove ugly macros and casts from crypt-gpgme.c

fix minor reflow issues in some comments

See: https://github.com/neomutt/neomutt/pull/392#discussion_r102072729
for the discussion.

Merge branch 'translate', updates from upstream

Update a bunch of translations from upstream:
    1536c45 Updated Catalan translation.
    ffc0e1f Updated Russian translation.
    55f2b57 Updated Ukrainian translation.
    47341a9 Updated Danish translation.
    b792e8e Updated Czech translation.
    fcf7330 Updated Czech translation.
    975efd6 Updated French translation.

Updated Catalan translation.

Updated Russian translation.

Updated Ukrainian translation.

Updated Danish translation.

Updated Czech translation.

Updated Czech translation.

Updated French translation.

merge: prepare the source for clang-format

 * fix 80 column limit, align statements
 * add clang-format file
 * added comment-blocks for clang-format to ignore
 * editorconfig: use spaces to indent in *.[ch] files

closes #392

editorconfig: use spaces to indent in *.[ch] files

added comment-blocks for clang-format to ignore

add clang-format file

fix 80 column limit, align statements

notmuch: Allow to use untransformed tag for color

closes #417

merge: upstream fixes (mutt/default)

 * Change "allow_dups" into a flag at hash creation.
 * Filter expired local certs for OpenSSL verification.
 * Plug memory leak in weed-expired-certs code.
 * Fix potential cert memory leak in check_certificate_by_digest().
 * Show SHA1 fp in interactive cert check menu.
 * Prevent null pointer exception for h->ai_canonname

closes #413

Prevent null pointer exception for h->ai_canonname

The getaddrinfo call in line 54 sets &h to a struct addrinfo. If a
canonical name cannot be found for the node argument of getaddrinfo,
h->ai_canonname is set to NULL. In that case, the strchr call in line
58 can lead to segfault. This behavior was observed on a macos sierra
while the hostname was 192.168.1.3 (unfortunately this happens quite
often in macos).

The fix is simple, just check h->ai_canonname for the NULL value.

Show SHA1 fp in interactive cert check menu.

While here, fix a few compiler warnings about sign mismatch in comparison.

Fix potential cert memory leak in check_certificate_by_digest().

Thanks to Matthias Andree's debugging, it appears the cert is not
freed when PEM_read_X509() encounters EOF.  Change the return value
check to not overwrite cert.  It is already updated via the second
parameter.

Plug memory leak in weed-expired-certs code.

X509_STORE_add_cert() creates a copy of the certificate we're offering,
so we need to free our copy afterwards.  This isn't documented, but from
observed behaviour in OpenSSL 1.0.2 and its master branch source code.

Change PEM_read_X509() call to reuse cert to avoid free/reallocation
overhead.

Filter expired local certs for OpenSSL verification.

OpenSSL has trouble establishing the chain and verifying when
duplicate expired certs are loaded in from $certificate_file.  A
warning about this is mentioned in
SSL_CTX_load_verify_locations(3SSL).

Filter out expired certs when loading verify certs.  Note that the
full certicates file is still used for verification in
check_certificate_by_digest().

Change "allow_dups" into a flag at hash creation.

Instead of having an "allow_dups" parameter for hash_insert(), add a
flag, MUTT_HASH_ALLOW_DUPS, to hash_create().

Currently ReverseAlias, subj_hash, and thread_hash allow duplicate
keys.  Change those hashes to pass the flag at creation, and remove
the last parameter from all callers of hash_insert().

merge: fix coverity defects

 * fix: resource leak - CID 76962
 * fix: resource leak - CID 76963
 * fix: resource leak - CID 76965
 * fix: resource leak - CID 76966
 * fix: resource leak - CID 76967
 * fix: resource leak - CID 76968
 * fix: resource leak - CID 76972
 * fix: resource leak - CID 76975
 * fix: resource leak - CID 76976
 * fix: resource leak - CID 76977
 * fix: resource leak - CID 76978
 * fix: resource leak - CID 76979
 * fix: resource leak - CID 76980
 * fix: resource leak - CID 76982
 * fix: resource leak - CID 76983
 * fix: resource leak - CID 76984
 * fix: resource leak - CID 76985
 * fix: resource leak - CID 76986
 * fix: resource leak - CID 83477
 * fix: resource leak - CID 83478
 * fix: resource leak - CID 150921
 * fix: resource leak - CID 160977

closes: #405

fix: resource leak - CID 160977

fix: resource leak - CID 150921

fix: resource leak - CID 83478

fix: resource leak - CID 83477

fix: resource leak - CID 76986

fix: resource leak - CID 76985

fix: resource leak - CID 76984

fix: resource leak - CID 76983

fix: resource leak - CID 76982

fix: resource leak - CID 76980

fix: resource leak - CID 76979

fix: resource leak - CID 76978

fix: resource leak - CID 76977

fix: resource leak - CID 76976

fix: resource leak - CID 76975

fix: resource leak - CID 76972

fix: resource leak - CID 76968

fix: resource leak - CID 76967

fix: resource leak - CID 76966

fix: resource leak - CID 76965

fix: resource leak - CID 76963

fix: resource leak - CID 76962

docs: remove duplicate "default" in the sidebar intro

[ci skip]

correct spelling mistakes (#412)

build: fix doc generator

travis: use container build

Travis offers two build environments: fully virtualised, container.

Until recently the container version didn't have all the libraries that we
needed.  Now it does.

The container is much faster to start up: <1s rather that ~20s.

Update the german translation (#397)

Remove snprintf.c from EXTRA_DIST (#406)

Pull #402

Kill homebrew (v)snprintf implementations, as they are C99 (#402)

Merge pull request #400 from neomutt/devel/issue-336

Avoid browsing the remote spoolfile by setting MUTT_SELECT_MULTI attach

notmuch: fix crash when completing tags (#395)

Notmuch was found to sometimes return a NULL when asked for a list of
tags.  This caused mutt to crash when trying to autocomplete.

Thanks to William Pettersson (@WPettersson) for the initial patch.

Closes #393
Closes #395

Display charset + small refactoring

Add option $show_multipart_alternative

When viewing a multi-part MIME email:
    'alternative_order' determines which part is displayed
    'auto_view' determines how the part is displayed

$show_multipart_alternative has three options to control what is displayed:
    unset    Only the chosen part (default)
    'info'   The chosen part plus information about the other parts
    'inline' All the parts

build: add a few includes (prelude to clang-format)

build: slcurses.h defines its own bool type

Avoid browsing the remote spoolfile by setting MUTT_SELECT_MULTI attach

Closes #336

Fixes missing failure return of notmuch msg open (#401)

When calling the mx_ops->open_msg() method of the notmuch
implementation, if the message failed to open, it is not reported to the
user interface. This patch returns a non-zero value that will prevent a
crash.

fixes #308

Signed-off-by: Pietro Cerutti <gahr@gahr.ch>
Signed-off-by: Guyzmo <guyzmo+github+pub@m0g.net>

Do not cast or check returns from safe_calloc (#396)

Fixes #388

Fix latest Coverity issues (#387)

* Fix two use-before-check bugs

Closes #386

* Fix memory leak

Issue #386

* Treat a NULL b->subtype as a wildcard (*)

Issue #386

* Move assignment out of conditional

Closes #386

* Move assignment out of conditional (2)

Closes #386

Advance by the correct number of position even for unknown characters (#368)

Closes #245

merge: sync to mutt/default

I've been cherry-picking all of mutt's upstream changes.
This gave us a neat and tidy commit history, but GitHub thinks we're
very out-of-sync.

This merge doesn't make any changes, but shows GitHub where we are.

merge: upstream fixes (mutt/default)

 * Canonicalize line endings for GPGME S/MIME encryption. (closes #3904)
 * Add a pattern_cache_t to speed up a few repeated matches.
 * Fix imap server-side search to call uid2msgno() only once. (see #3905)
 * Convert HASH to be indexable by unsigned int. (see #3905)
 * Create a uid hash for imap. (see #3905)
 * Convert cmd_parse_search to use the uid hash. (closes #3905)
 * Fix minor documentation issues.
 * Allow initial blank lines in local mailboxes.
 * Improve error handling in mbox magic detection.
 * Add casecmp and strdup_key flags to hash_create()
 * Fix several alias hashtable issues.
 * Fix sidebar references to the "new count" to be "unread". (closes #3908)
 * Improve Reply-to vs From comparison when replying. (closes #3909)
 * Abstract the SPAM_LIST as a generic REPLACE_LIST
 * Add subjectrx command to replace matching subjects with something else.
 * Allow "unsubjectrc *" to remove all patterns.
 * Adds capability to edit x-labels inside mutt, and to sort by label.
 * Minor fixes to the x-label patch from David.
 * Add hash_find_elem to get the hash element.
 * Adds label completion.
 * Improve the label completion hash table usage.
 * Fix the x-label update code check location.
 * Fix the mutt_label_complete() pos parameter.
 * Permit tab completion of pattern expressions with ~y (labels).
 * Simplify mutt_label_complete().

Simplify mutt_label_complete().

It was derived from mutt_command_complete(), which had more complex
requirements.  For labels, we just need to skip whitespace and
complete based on the passed in buffer.

Therefore, we don't need the pos parameter, or to work backwards from
the end of the buffer.

Permit tab completion of pattern expressions with ~y (labels).

Thanks to David Champion for the original patch.  This version is
slightly different, as I couldn't get the original patch working.
This version simply scans backward for the first ~, and if it is ~y,
invokes completion.

Fix the mutt_label_complete() pos parameter.

i is the state->wbuf index, not the end of the buf.  It was "working"
only because the contents of buf past the null were not "space" most
of the time.

Fix the x-label update code check location.

The x-label comparison was outside the "beginning of header" block.

This meant that it could theoretically match a continuation line.
Additionally, the continuation lines of x-labels would not be
stripped, because the comparison was after the ignore variable was
reset.

Move the comparison inside the block and before the ignore reset.

Improve the label completion hash table usage.

Move the hash table inside the Context.  Hook message arrival/deletion
to update the label hash.

Change the label hash to strdup keys.

Use hash_find_elem when updating the counter, to reduce unnecessary
add/delete operations.

Adds label completion.

A global label hash is added, to which labels are added as they're parsed
from a mailbox file or edited manually by the user.  Reference counts are
kept in the hash table so that unused labels are removed from available
completions.  Completion is available in the label editor only, but it
may be feasible to add for search expressions if the preceding text ends
with '~y'.

Add hash_find_elem to get the hash element.

This will be used in the following patch for directly manipulating the
label counter.

Minor fixes to the x-label patch from David.

Add L10N comment to sort menu.  Mark a couple strings for localization.

Use ascii_strncasecmp() for the X-Label header comparison.

Simplify label_message() using mutt library routines.

Bind label editing to "Y" instead of "y".  "y" is already used in the
default sample muttrc to display mailboxes.

Adds capability to edit x-labels inside mutt, and to sort by label.

Allow "unsubjectrc *" to remove all patterns.

Thanks to Aaron Schrab for the original patch.

Add subjectrx command to replace matching subjects with something else.

This lets you define regular expressions-replacement pairs for subject
display.  When a Subject: matches the regular expression, the replacement
value will be displayed instead in the message index.  Backreferences
are supported.

This is especially nice for simplifying subjects that are overly wordy,
such as mailing list posts (with [Listname] tags, etc), mail from
ticketing systems or bug trackers, etc.  It lets you reduce clutter in
your mutt display without altering the messages themselves.

Abstract the SPAM_LIST as a generic REPLACE_LIST

REPLACE_LIST can be used more generally as a list of pattern
match-replace settings.  SPAM_LIST was a special case of this, so
spam handling has been been changed to use REPLACE_LIST instead, and
SPAM_LIST was removed.

A generic function for performing a REPLACE_LIST replacement has
been added in mutt_apply_replace().

Commited by Kevin McCarthy with some buffer overflow fixes in
mutt_apply_replace().

Improve Reply-to vs From comparison when replying. (closes #3909)

Prior to this patch, if the Reply-to mailbox matched the From mailbox,
mutt would always use the From address.  This was probably done to
preserve the display name, as the Reply-to address is often missing one.

Unfortunately, there are circumstances where the Reply-to display-name
has significance, such as in ticket 3909.

Change mutt so that it only uses the From address if the Reply-To has
no display-name.

Fix sidebar references to the "new count" to be "unread". (closes #3908)

%N in $sidebar_format and "new"in $sidebar_sort_method actually use
the unread message count.  Update the documentation to mention that.

Add an "unread" method to $sidebar_sort_method, but preserve "new" for
compatibility.

Change the SORT_COUNT_NEW constant to SORT_UNREAD, so the code is also
consistent with the meaning.

Thanks to cri for reporting the problem and suggesting where to fix it
in the code.

Fix several alias hashtable issues.

Convert to use the strdup keys hash.  Addresses can be converted back
and forth from intl to local forms.  This frees and recreates a new
addr->mailbox string, resulting in the hash table key being a dangling
pointer.

Change alias hash table insert/remove to ensure the address is in intl
form.  The alias menu (previously) converted address entries to local
form when performing a completion.  Even with the pointer issue fixed,
the entries may not be removed from the hash if the intl and local
forms are different.

Lastly, there is no reason for the alias menu to manually convert to
local form before writing the address to the output buffer.
rfc822_write_address() has a display parameter that will call
mutt_addr_for_display() instead when set.  Change to set the display
parameter and remove the conversion calls.

This last change obviates the first two changes, but they are a good idea
in any case.

Add casecmp and strdup_key flags to hash_create()

Aliases and (in the future), X-Label hashes will require a hash that
strdups the key.  Convert the casecmp parameter of hash_create() to a
flags parameter, and add a flag to strdup the keys.

Improve error handling in mbox magic detection.

Thanks to Simon Ruderich for pointing out several small issues with
the previous commit.

Allow initial blank lines in local mailboxes.

Some mailbox-creation tools erroneously append a blank line to a file
before appending a UNIXv7-format mail message, resulting in mailboxes
that are intended to be valid "mbox" folders but are not.  Notably old
versions of Mailman do this, making archive files that cannot be read by
mutt.

This patch causes mutt to skip leading NLs and CRs when detecting magic.

Fix minor documentation issues.

manual.xml:
Wrap line for clarity.

muttrc.man:
Remove superfluous spaces in brackets.
Add missing error object in color command.
Sort the prompt object.
Add sidebar color objects.

Convert cmd_parse_search to use the uid hash. (closes #3905)

Replace the linear scan for each result with a hash lookup.  This
should greatly improve performance for large mailboxes.

Create a uid hash for imap. (see #3905)

This hash will allow for more efficient UID SEARCH processing,
replacing a linear scan with a hash lookup.

Convert HASH to be indexable by unsigned int. (see #3905)

Convert the HASH to be usable for either string or unsigned int keys,
so that a uid hash can be added for imap.

To keep hash-usage code disruption to a minimum, this introduces new
create/insert/find/delete functions for the int hash, but keeps the
old function names for string keys.

This implementation makes the key a union.  It may have been a better
idea to introduce a whole new structure, but this way allows minimum
changes to and maximum reuse of the existing hash code.

Fix imap server-side search to call uid2msgno() only once. (see #3905)

After performing a UID SEARCH, the results are parsed in
cmd_parse_search().  This was accidentally calling uid2msgno() twice.
Since that function does a linear search, this has a noticable impact
on large search results.

Add a pattern_cache_t to speed up a few repeated matches.

Vincent Lefèvre reported experiencing an index display performance
issue.  This occurred with messages containing many recipients.  He
had many index color lines containing ~l.  The ~l ended up being run
over and over on these messages, resulting in a noticable slowdown
displaying the index.

This patch adds caching for just a few of the pattern operations (~l,
~u, ~p, ~P) that are potentially expensive and also don't have
arguments.  The caching is only enabled for operations repeatedly
matching against the same message: color, hooks, scoring.

The caching is fairly targeted, but isn't that invasive or
complicated.

Canonicalize line endings for GPGME S/MIME encryption. (closes #3904)

This matches the behavior for S/MIME classic mode: OpenSSL converts
the line endings to cr/lf before encrypting.  Although Mutt always
canonicalizes the line endings before verifying the signature, some
clients do not do this for encrypted messages.

Thanks to cooler for the patch!

Release KyotoCabinet data with kcfree() (#384)

Obtained from https://dev.mutt.org/hg/mutt/rev/09bb4a62ceb1

Closes #383