RFC2047-decode mailto header values. (closes #3879)

RFC 6068 specifies that the header values (with the exception of body)
may contain RFC 2047-encoded values.

Reset invalid parsed received dates to 0.  (closes #3878)

The actual problem in the ticket would be solved by d3f31cf9239e (see
#3798).  However there is still the bug that Mutt considers a
(hdr->received != 0) to be set and usable, despite not checking the
return value of mutt_parse_date().

Change mutt_read_rfc822_header() to unset an invalid received value
back to 0.  We don't do this inside mutt_read_rfc822_line() because
that would cause the next received line to be parsed.

Clear pager position when toggling headers.

It doesn't make sense to try to preserve the pager position when
toggling headers: the purpose of toggling headers is to see the
headers in full or weeded state.  So, reset the position back to the
top.

Don't abort the menu editor on sigwinch. (closes #3875)

getch() will return ERR on sigwinch when timeout() is called with a
positive value.  mutt_getch() will therefore return ch==-2 for both a
timeout and a sigwinch in this case.

The imap code in km_dokey() exits out of the ImapKeepalive loop for a
SigWinch, and was skipping past the check for MENU_EDITOR and
tmp.ch==-2.  Move this check below the gotkey: label so the
ImapKeepalive loop behaves the same as the Timeout code.

Thanks to nicop for reporting the problem and for the initial patch!

Mark some gpgme pgp menu keybinding translations as fuzzy. (closes #3874)

Some translations for crypt-gpgme.c are marked as fuzzy but the keybindings
attached to these translations are not, this creates confusions for the users
who see the english message but have the keybindings for a message in their own
language available.

As long as the translations are fuzzy, the keybindings should stay fuzzy.

Check for NULL mx_ops in mx.c

Eike Rathke reported this happening when in an IMAP index view the
underlying connection was terminated, ctx->mx_ops was NULL and thus
accessing ctx->mx_ops->check segfaulted.

Thanks also to Eike Rathke for the initial patch, for which I expanded
the checks to other functions.

Use body color for gpgme output. (closes #3872)

When switching from pgp_* commands to crypt_use_gpgme=yes, Peter
Colberg noticed that the output was colored 'brightyellow'.

The issue is that crypt-gpgme.c uses state_attach_puts in various
places where it should use state_puts to maintain compatibility with
the previous behavior in pgp.c.

Fix gpgme segfault when querying candidates with a '+' in the address. (closes #3873)

list_to_pattern() was not allocating enough space for the '+' to '%2B'
transformation.

docs: fix notmuch vim syntax file

build fix for strndup / malloc

Some systems don't have strndup() and rely on our version.
I changed it to use Mutt's safe_malloc(), but unfortunately it gets
compiled into some tools which don't have that function.

travis: install the kyoto-cabinet dev files

Now, we will be able to test the kyoto build automatically.

docs: fix the alignment in a 'nested-if' example

fix a typo in the version string

fix: change gcc build options to prevent crashes

A GCC optimisation "-O2" enables "-fdelete-null-pointer-checks" which
was causing some pointer checks to be optimised out of the code.
This was causing some crashes, most notably in the pager.

fix: bulletproof the pager

Put some more checks into the pager, just in case the Context
disappears, e.g. when a network connection goes down.

build source before docs

The Makefile tried to build the docs before the IMAP source.
Change the build order to prioritise the code.

fixed typo of devel mailinglist name

added Mailinglist mailto links to "Where is NeoMutt" section

quasi-delete: check there's a selection

If you created a vfolder with no contents, then tried to quasi-delete
something, it would crash.

docs: fix the layout of the syntax file

docs: make the license clear to github

Fixes repaint problem with $pager_index_lines #159

Commit 6b4c4ca changed the behaviour of mutt_pager to build the
pager index even if pager_index_lines was set to 0 to (fixing #143).

The code redrawing the pager status line was relying on the pager index
being NULL to determine if a redraw should take place. Now
index_window->rows is used as an additional safeguard.

Also added a check if pager index is set to the terminal status line
update code as it was unguarded and old default behaviour might be
introduced later.

Fixes #159

merge: feature compose-to-sender

 * add ident to PATCHES
 * feature: compose-to-sender
 * Add compose-to-sender documentation

Closes #30

Add compose-to-sender documentation

feature: compose-to-sender

Compose a new email to the sender.
This works on tagged messages too.

add ident to PATCHES

Optimize LMDB's hcache backend.

Thanks to Clemens Lang for his help with the fix.

See: https://dev.mutt.org/trac/ticket/3691
Closes: #77

merge: feature kyoto-cabinet

 * add ident to PATCHES
 * feature: Add Kyoto Cabinet header cache
 * doc: Document the Kyoto cabinet hcache backend

doc: Document the Kyoto cabinet hcache backend

Additionally, fix a couple of references in other parts of the
documentation that now also work with Kyoto Cabinet (and LMDB).

feature: Add Kyoto Cabinet header cache

Tokyo Cabinet's website now "strongly recommends" using Kyoto Cabinet
instead of Tokyo Cabinet because it claims to be superior. Add Kyoto
Cabinet as an option.

In some preliminary testing, the performance seems to be on-par with
Tokyo Cabinet, but not better. Kyoto Cabinet seems to need a little more
space, but the difference is probably not significant.

Signed-off-by: Clemens Lang <neverpanic@gmail.com>

add ident to PATCHES

Change 'attach_keyword' option from string to regex

A regular expression is much more useful than a single word to check for
forgotten attachments, especially when writing in multiple languages.

This is inspired by the muttng patchset abort_noattach [1], which I use
for years, but newly implemented to make minimal changes to the neomutt
source code.

The keyword is no longer mentioned in the error message as regular
expressions tend to be longer.

The option 'attach_keyword' could be renamed 'attach_remind_regex' as in
the muttng patch.

[1] https://github.com/codito/muttng-patchset/blob/master/patches/muttng.abort_noattach.diff

Closes: #138

docs: Add badges to README.neomutt

Added travis build status, coverity status and license badges to
README.neomutt

Sensible-browser behaviour fixes

 * Feedbacks highlighted issues in menu position when people switch
   between mailboxes mode and browser mode.
 * A preferred behaviour is to have 'c?' display $folder in "local" mode
   if existant on startup. For IMAP, we keep using the current directory
   in any case.
 * A '=' binding have been added:
   When in browser mode, not in mailboxes mode, '=' allows to jump back to
   $folder from anywhere one is. It stores in a variable the place the
   user was. Another hit on '=' will bring him back. When a new directory
   is explored, the variable is purged.
 * Closes #146

fix build of neomutt-syntax.vim

NeoMutt 2016-09-16 (1.7.0)

fix notmuch tag completion

Notmuch tags that contained a dash (-) confused the auto-completion.

Given tags of "neo-red", "neo-green", "neo-blue", tests:
    n<tab>      Generates: neo-
    neo-<tab>   Cycles through the three tags

The +/- before the tag are optional.
With neither, notmuch will assume +

Fixes #124

fix sidebar sort order

Collate the sidebar mailboxes for a more natural sort order.
A sorted sidebar will now match a sorted browser.

e.g.
    Old Sort    New Sort
    --------    --------
    _apple      Apple
    _Banana     _apple
    _Cherry     banana
    _damson     _Banana
    Apple       Cherry
    banana      _Cherry
    Cherry      damson
    damson      _damson

fix: make the pager more robust

Make sure that the Context is always checked before use.
Zero a HEADER struct allocated on the stack.

nntp: close message handles

Possible fix for #125

doc: Removes bug entry in new-mail docs

Bug #19 (new-mail doesn't fire if pager) was fixed upstream on commit
(4ed2b6f). We remove their mention of known-bugs in the documents

Revert "doc: Removes bug entry in new-mail docs"

This reverts commit 49f2effbce887a2d1e7cc88146dd48cfd987a4cb.

This commit accidentally included a stray patch file.

doc: Removes bug entry in new-mail docs

Bug #19 (new-mail doesn't fire if pager) was fixed upstream on commit
(4ed2b6f). We remove their mention of known-bugs in the documents

fix some translations in crypt-gpgme.c

Some translations for crypt-gpgme.c are marked as fuzzy but the keybindings
attached to these translations are not, this creates confusions for the users
who see the english message but have the keybindings for a message in their own
language available.

As long as the translations are fuzzy, the keybindings should stay fuzzy.

Debian bug: http://bugs.debian.org/837673
Upstream mutt issue: http://bugs.mutt.org/3874

merge: upstream fixes (mutt/default)

 * Remove the $locale configuration variable.
 * Add $attribution_locale configuration variable.
 * Add missing include <locale.h> to send.c and edit.c.
 * Filter out zero width no-break space (U+FEFF).
 * Update a confusing and obsolete comment.
 * Moves mutt_copy_list to muttlib.c, where it belongs.
 * Redraw screen after an SSL cert prompt
 * Preserve message-id and mft headers for recalled messages. (closes #3081)
 * Fix openssl 1.1 compilation issues. (closes #3870)

Fix openssl 1.1 compilation issues. (closes #3870)

With these changes, Mutt will no longer compile for versions less than
0.9.6.

Preserve message-id and mft headers for recalled messages. (closes #3081)

Git patch creates a patch series mailbox, including the Message-ID.
Using this as draft files was removing the Message-ID, and thus
breaking the threaded structure.

The second part of the ticket has already been addressed by
95a2230ef889 (for ticket 3653).

Thanks to Chris Webb for the original patch.

Redraw screen after an SSL cert prompt

Moves mutt_copy_list to muttlib.c, where it belongs.

Update a confusing and obsolete comment.

This 2004 comment in sort.h predicted what has recently come to pass, so
I'm reframing it just to document for future devs what's going on with
this oddball flag.

Filter out zero width no-break space (U+FEFF).

Add missing include <locale.h> to send.c and edit.c.

Add $attribution_locale configuration variable.

$attribution_locale replaces the just removed $locale, but is only
used for customizing the LC_TIME locale used for dates in
$attribution.

This could be useful in conjunction with folder or send-hooks for
recipients in different locales.

Remove the $locale configuration variable.

$locale was only used to set the LC_TIME locale.  Unfortunately, Mutt
previously defaulted to using "C".  This overrode the user's locale
setting and forced them to re-specify their locale inside their
.muttrc.

Remove $locale and instead use the locale specified by the
environment.  Mutt still allows "C locale" dates by using a leading
"!" in $date_format, ${}, etc.

Another use of $locale was to customize attribution dates using hooks.
The next commit will introduce $attribution_locale, which can be used
for this instead.

Thanks to Derek Martin for the original patch!

fix another crash in the pager

This commit prevents the crash.
A better solution will require more thought.

Fixes: #144

fix crash when exiting the pager

Make sure we've always got a menu pointer.
This is a fix for commit:
    "check for new mail while in pager when idle"

Fixes: #143

Fix off-by-one in error message

Closes: #139

Fix buffer overrun in search for attach keyword

Regression introduced by 395911ea4c20e57f121cb169e0f2083ff5e1ae33.

In the process of tidying

    char* lowerKeyword = malloc(strlen(AttachKeyword)+1);
    ...
    for (i=0; i <= strlen(AttachKeyword); i++)

was changed to

    int klen = mutt_strlen (AttachKeyword) + 1;
    ...
    char *lowerKeyword = safe_malloc (klen);
    ...
    for (i = 0; i <= klen; i++)

Now that klen is the actual allocation length, instead of the string
length, the loop comparison needs to be adjusted accordingly.

Closes: #142

fix AC_INIT tarname parameter

By specifying 'neomutt' as the tarname, the docs and translations were
being installed in wrong places.

Note: distdir is overridden in Makefile.am so that we still get a
neomutt tarfile on "make dist".

Add unsidebar_whitelist command.

This pairs with the sidebar_whitelist command, and operates like the
other "un..." list commands.

Fix sidebar documentation a bit. (closes #3859)

Sidebar_whitelist is a command, not a variable.  Also add a blurb about what it does.

Fix the sort order for $sidebar_divider_char and $sidebar_delim_chars.

docs: mass tidy up

 - tidy muttrc.*
 - tidy vimrc.*
 - sync muttrc to main manual
 - add better xml escaping
 - fix whitespace in muttrc/vimrc/manual
 - refer to features rather than patches
 - fix lots of spelling mistakes
 - sort manual sections
 - add README.notmuch to dist
 - drop unnecessary sample.colors.attach_headers
 - install sample muttrc files
 - add mutt-1.7.0-syntax.vim from vim repo
 - generate neomutt-syntax.vim from features' vimrc files
 - install neomutt-syntax.vim

Avoid segfault when listing mailboxes on startup.

When mutt is started with the -y flag, _mutt_select_file() is called while
CurrentFolder is a null pointer. If that's the case, we shouldn't try to use
it.

Closes: #137

NeoMutt 2016-09-10 (1.7.0)

align the nntp code with mutt

Specifically, change mutt_strcasecmp() -> ascii_strcasecmp().

proposed fix for clearing labels

Debian bug #832971 -- The use claimed he couldn't reset labels.
I couldn't repeat the problem using his config.  He offered a patch that
he claimed fixed his problem.  Only one part of the patch actually did
anything (and only in a section of code dealing with SETTING labels).

Allow the user to interrupt slow IO operations

use fixed version strings

docs: fix incorrect link

merge: forgotten-attachment

 * add ident to PATCHES
 * Forgotten attachment
 * tidy code to mutt standards
 * Add docs for forgotten-attachments

Add docs for forgotten-attachments

tidy code to mutt standards

Forgotten attachment

add ident to PATCHES

fix configure check for fmemopen

To use fmemopen you now need the presence of fmemopen() and
open_memstream() AND you have to pass --enable-fmemopen to configure.

Note: fmemopen is still DISABLED due to an unresolved bug.

restore the pager keymapping 'i' to exit

The mapping was removed and overridden by the NNTP patch.
This commit restore the Mutt default behaviour.

Fixes: #133

unnecessary document

docs: kill tabs that had crept into the manual

notmuch: sync vfolder_format to folder_format

The expandos used by NotMuch's vfolder_format didn't match those of
Mutt's folder_format.  This caused confusion.

vfolder_format expandos:
    %C  current file number
    %f  folder name (description)
    %m  number of messages in the mailbox *
    %n  number of unread messages in the mailbox *
    %N  N if mailbox has new mail, blank otherwise

Default setting:
    set vfolder_format = "%2C %N %?n?%4n/&     ?%4m %f"

merge: upstream stable

 * Added tag mutt-1-7-rel for changeset a4e83f60e42f
 * mutt-1.7.0 signed
 * Increase date buffer size for $folder_format.  (closes #3863)
 * Disable ~X when message scoring.  (closes #3861)
 * Fix pgpring reporting of DSA and Elgamal key lengths.  (closes #3867)
 * Autoconf: always check for getaddrinfo().
 * Stub out getdnsdomainname() unless HAVE_GETADDRINFO.
 * Add missing sidebar contrib sample files to dist tarball.

docs: update list of features, authors

docs: sort manual sections

merge: attach-headers-color

 * Add support for colouring attachments with regexp
 * Add sample colour file
 * Add docs on attach_headers
 * add ident to PATCHES

add ident to PATCHES

Add docs on attach_headers

Add sample colour file

This file colours the results of GPGME checks.

Add support for colouring attachments with regexp

Attachments can now be coloured in the same way as the body using
`attach_regexp`. The `attachment` colour can still be defined and is
used when no regexp match.

add ident to PATCHES for sensible-browser

docs: fix typo

merge: sensible-browser

 * sensible browser patch
 * Whatever the sort method, .. is always on the top
 * Reset menu->current when one re-sort the browser
 * Folder tracking enabled only with SORT_(SUBJECT|DESC)
 * Some documentation on sensible-browser patches
 * tweak the behaviour of the sensible-browser code
 * sort mailboxes, with incomplete info, last
 * docs: try to explain the behaviour of sensible-browser

docs: try to explain the behaviour of sensible-browser

sort mailboxes, with incomplete info, last

tweak the behaviour of the sensible-browser code

* fix crash when used with notmuch
* drop unnecessary breaks
* enabled for unsorted mailboxes
* don't select '..' if sorted by count, etc

Some documentation on sensible-browser patches

Folder tracking enabled only with SORT_(SUBJECT|DESC)

Reset menu->current when one re-sort the browser

Whatever the sort method, .. is always on the top

sensible browser patch

 In the browser, when you go to a folder's parent, the folder you were in
 should be selected.  This patch works for IMAP folders.

Add missing sidebar contrib sample files to dist tarball.

I previously added the files, but neglected to add them to the
contrib/Makefile.am file.

Thanks to isdtor for pointing out the problem and for the original
patch.

Stub out getdnsdomainname() unless HAVE_GETADDRINFO.

It seems unlikely there are systems without it (given that this
mistake has been in since 1.6.0), but for correctness we should stub
out the function for those without it.