update Makefiles

delete TODO files

see https://github.com/neomutt/neomutt/milestone/9

remove legacy files

* README (infos for mutt 1.5) doesn't contain useful infos
* NEWS (infos about mutt 1.2) is too legacy, neomutt started with mutt 1.5

don't generate vim-neomutt syntax file

doc: remove LaTeX/pdf manual generation

remove --enable-sidebar and make it persistent

It was decided to keep the USE_SIDEBAR tests in the codebase to make it
easier to find code related to that feature, but that requires
USE_SIDEBAR to be defined, or it breaks compilation because some symbol
definitions depend on that config option.

github: improve PR template and CONTRIBUTING.md

PR template:

* made structural questions bold to contrast it from the actual
  PR description.

* add note that acceptance criteria can be removed when the PR is
  submitted. Its only meant as a reminder.

* fixed manual link.

CONTRIBUTING.md:

* wrap text at 80 characters.

* add note to mention specific methods or files if the commit only
  changes things there.

[ci skip]

NeoMutt Release 2017-04-28

Fix km_error_key() infinite loop and unget buffer pollution.

'bind pager \Ch help' produces an infinite loop when an unbound key is
pressed in the pager.  The reason is because km_error_key() tries to
verify that the key sequence is really bound to the OP_HELP operation.
It does this by using km_expand_key(), tokenize_unget_string() on the
resulting buffer, then checking if the next km_dokey() returns OP_HELP.

The problem is that km_expand_key() does not always produce a string
that is properly reparsed by tokenize_unget_string().  Control-h
sequences are expanded to ^H.  tokenize_unget_string() recognizes this
as two characters '^' and 'H'.  km_error_key() checks the OP returned,
which is OP_PAGER_TOP for the '^'.  This is not OP_HELP, so it prints
a generic error and returns.  This leaves the 'H' in the input buffer!
Since 'H' (by default) is unbound in the pager, it retriggers
km_error_key(), resulting in an infinite loop.

The same issues can occur without control sequences:
  bind generic ? noop
  bind generic dq help
In the index, hitting an unbound key will end up leaving 'q' in the unget
buffer, because 'd' is bound in the index menu and will be read by km_dokey().

A simple approach to fix this would be to just use the same code as in
mutt_make_help(), which has no double-check.  This would be no worse
than the help menu, but can generate an inaccurate error message (e.g
if '?' were bound to noop)

This patch instead uses OP_END_COND as a barrier in the unget buffer.
It directly inserts the keys in the OP_HELP keymap, instead of using
km_expand_key() + tokenize_unget_string().  After calling km_dokey()
it flushes the unget buffer to the OP_END_COND barrier.

Thanks to Walter Alejandro Iglesias for reporting the bug.

Fix error message when opening a mailbox with no read permission. (closes #3934)

ctx->mx_ops ends up being NULL for both the case that ctx->magic is 0
and -1.  This meant the mutt_perror() error message was never being
printed, because the check for ctx->mx_ops == NULL was taking place
first.

Move the "ctx->magic == -1" check first, so mutt will print out an
appropriate perror message in that case.

docs: fix remaining TLS typos

Fix typo in README.neomutt (#548)

Fix and simplify handling of GPGME in configure.ac (#545)

* Fix
When GPGME is not installed, the m4 macro AM_PATH_GPGME is not defined,
which causes autoconf to fail. The fix adds an `ifdef` around the block
to skip expanding the macro, if it's not defined.

* Enhancement
The enhancement is to assume that we have at leat GPGME 1.2.0 (released
in 2009). This allows us to simplify the configure.ac code block by
avoiding checks for functionalities we know to be there.

Fixes #542

NeoMutt Release 2017-04-21

sync translation to upstream changes

merge: upstream release 1.8.2

 * automatic post-release commit for mutt-1.8.2
 * Added tag mutt-1-8-2-rel for changeset c6ea4aed6bec
 * mutt-1.8.2 signed

automatic post-release commit for mutt-1.8.2

fix: various minor fixes

- nroff typo in docs
- fix function prototype
- add spaces inside comments
- add missing name to mailmap

build: add keymap_alldefs.h to BUILT_SOURCES

keymap_alldefs.h needs to be generated by the main build so that
targets, "update-po", "dist" and "distcheck" succeed.

build: lower the gettext requirement 0.18 -> 0.17

RHEL6 only has gettext 0.17.

fix broken from_chars behaviour

Upstream adopted and extended the from_chars feature.
A couple of changes to the behaviour were introduced by the change:
- '\r' wasn't being recognised
- An unwanted space was being displayed

Fixes #533

mutt_envlist_set: change 1/0 int parameter to boolean (#537)

Remove -Iimap from CFLAGS and include imap/imap.h explicitely (#536)

fix decode of encrypted mail

This one-char typo caused mutt to lock up when reading encrypted email.

merge stable

mutt-1.8.2 signed

Added tag mutt-1-8-2-rel for changeset c6ea4aed6bec

automatic post-release commit for mutt-1.8.2

Fix GPG_TTY to be added to envlist. (closes #3931)

Changeset 37209157e33c converted filters to use the envlist.
Unfortunately, I missed that pgp.c sets GPG_TTY when using the GnuPG
agent.  Convert to add GPG_TTY to the envlist too.

Revert sort prompt labels. (see #3930)

Using the (s)ort style may look a bit better, but it makes the prompt
over 80 columns.  The multichoice prompt supports multiple lines now,
but it is better to have it fit on one line if possible.

Revert back to the "capital letter" method, but leave the ticket open,
to explore other ideas that Vincent and chdiza have for better ways to
present a long choice like this.

merge stable

Fix GPG_TTY to be added to envlist. (closes #3931)

Changeset 37209157e33c converted filters to use the envlist.
Unfortunately, I missed that pgp.c sets GPG_TTY when using the GnuPG
agent.  Convert to add GPG_TTY to the envlist too.

merge: upstream fixes (mutt/default)

 * Add ifdefs around new mutt_resize_screen calls.
 * Add multiline and sigwinch handling to mutt_multi_choice. (see #3877)
 * Set pager's REDRAW_SIGWINCH when reflowing windows.
 * Add multiline and sigwinch handling to mutt_yesorno. (closes #3877)
 * Change the sort prompt to use (s)ort style prompts.
 * Handle the pager sort prompt inside the pager.

Handle the pager sort prompt inside the pager.

Display the prompt in the pager, so a multiline prompt or resize
doesn't refresh the index menu.  Then, bounce back through the index
to handle resorting and status line updates.

This also fixes cancelling to stay in the pager.

Change the sort prompt to use (s)ort style prompts.

This is consistent with the rest of mutt, and in my opinion, more readable.

Add multiline and sigwinch handling to mutt_yesorno. (closes #3877)

Most of the yes/no and query_quadoption prompts are pretty short, but
for completeness add handling for those too.

Set pager's REDRAW_SIGWINCH when reflowing windows.

So that all external reflow handling functions don't have to remember
to set the flag too.

Add multiline and sigwinch handling to mutt_multi_choice. (see #3877)

Resize the message window up to three lines to fix wide prompts.

Enable sigwinch processing and redraw the current menu as needed.

Add ifdefs around new mutt_resize_screen calls.

Changeset 231fa2eff206 added sigwinch handling to _mutt_enter_string()
but neglected to add an ifdef check.

build: fix distcheck

Fix the location of keymap_alldefs.h so the translations can be created.

Handle the pager sort prompt inside the pager.

Display the prompt in the pager, so a multiline prompt or resize
doesn't refresh the index menu.  Then, bounce back through the index
to handle resorting and status line updates.

This also fixes cancelling to stay in the pager.

Change the sort prompt to use (s)ort style prompts.

This is consistent with the rest of mutt, and in my opinion, more readable.

resync translations

Some translations got lost in the move.
This commit restores them and updates all the file references.

build: restore po/mutt.pot

This file is autogenerated from the the files listed in po/POTFILES.in
However, if it's missing it causes the build to regenerate ALL the .po
files every time there's a change to the code.  This is annoying.

If you want to update the .po files, try:
    make -C po update-po

fix: build of imap/auth_sasl.c

Header file <sasl/sasl.h> doesn't seem to include a header defining
'size_t'.  Make sure we include <stddef.h> before it.

Closes #525

docs: delete trailing whitespace

This was left by tidy when the docs were reformatted.

use w3m as default for generating UTF8 manual.txt (#491)

The input for manual.txt is manual.html which is UTF8 encoded. Now
forcing manual.txt to be ascii leads to all sorts of "?" in the ascii
file, which is not nice. If we have UTF8 input we should also use UTF8
for the output.
Moved w3m to be the default to generate UTF8 manual.txt. If you prefer
manual.txt to be ascii the fallbacks lynx or elinks will still produce
plain ascii.

Signed-off-by: Stefan Assmann <sassmann@kpanic.de>

add requets to be patient in IRC and eliminate warnings (#523)

* README.md: add note to be patient when asking in IRC
* CONTRIBUTIONS.md: Please eliminate warnings the compilation tools produce.

[ci skip]

Add multiline and sigwinch handling to mutt_yesorno. (closes #3877)

Most of the yes/no and query_quadoption prompts are pretty short, but
for completeness add handling for those too.

fill out .mailmap

[ci skip]

Set pager's REDRAW_SIGWINCH when reflowing windows.

So that all external reflow handling functions don't have to remember
to set the flag too.

Add multiline and sigwinch handling to mutt_multi_choice. (see #3877)

Resize the message window up to three lines to fix wide prompts.

Enable sigwinch processing and redraw the current menu as needed.

Add ifdefs around new mutt_resize_screen calls.

Changeset 231fa2eff206 added sigwinch handling to _mutt_enter_string()
but neglected to add an ifdef check.

merge: upstream release 1.8.1

 * automatic post-release commit for mutt-1.8.1
 * Added tag mutt-1-8-1-rel for changeset f44974c10990
 * mutt-1.8.1 signed

update translation files

apply clang-format to the source code

The source and headers have been treated slightly differently.

The source files (*.c) have had clang-format applied and it's expected
that this will become an automatic process in the future.  A few blocks
of source have been protected from change by special comments
    // clang-format off
    // clang-format on

The header files contain a lot of #define'd lists which contain a lot of
manually placed whitespace.  clang-format doesn't have an option to
align these as we'd like.

The compromise was to run clang-format on the headers, but cherry-pick
the formatting that was more human-friendly.  clang-format won't be
automatically run on the headers.

This commit also sorts out a few other formatting issues:
- All tabs have been expanded
- Tabs in translated strings have been expanded
  (this makes it easier to align translated strings)

convert .ja translation to utf-8

build: rename files hyphen to underscore

revert broken fix

gpgme returns 0 on EOF which we shouldn't pass to fwrite().

revert broken fix

This IS the site of a possible bug, but this isn't the way to fix it.
Some paths through parse_set() will lead to (idx == -1) which will cause a
buffer underrun when used as an index to MuttVars.

closes #520

merge stable

mutt-1.8.1 signed

Added tag mutt-1-8-1-rel for changeset f44974c10990

automatic post-release commit for mutt-1.8.1

merge: boolify functions

These functions returned -1 on failure but are essentially boolean.
Convert them to use false/true.
 * check_all_msg()
 * check_msg()
 * compare_certificates()
 * compare_stat()
 * crypt_key_is_valid()
 * is_bound()
 * line_compare()
 * mutt_check_charset()
 * rfc822_valid_msgid()
 * smtp_code()
 * url_parse_query()
 * user_in_addr()

user_in_addr

url_parse_query

smtp_code

rfc822_valid_msgid

mutt_check_charset

line_compare

is_bound

crypt_key_is_valid

compare_stat

compare_certificates

check_msg

check_all_msg

merge: boolify functions

These functions all returned 0/1 on failure/success.
Convert them to use false/true.
 * add_quote_suffix()
 * addrcmp()
 * addresses_use_unicode()
 * address_uses_unicode()
 * check_boundary()
 * check_certificate_cache()
 * check_certificate_expiration()
 * check_for_mailing_list()
 * check_idn()
 * cmd_queue_full()
 * compare_flags()
 * count_body_parts_check()
 * create_hcache_dir()
 * crypt_is_numerical_keyid()
 * feature_enabled()
 * get_initials()
 * hostname_match()
 * imap_has_flag()
 * is_context_available()
 * is_day_name()
 * is_display_corrupting_utf8()
 * link_threads()
 * maildir_add_to_context()
 * match_reference()
 * mbox_has_new()
 * mh_valid_message()
 * mutt_addr_is_user()
 * mutt_comp_can_append()
 * mutt_comp_can_read()
 * mutt_group_match()
 * mutt_limit_current_thread()
 * mutt_matches_list()
 * mutt_match_spam_list()
 * mutt_needs_mailcap()
 * mutt_pop_list()
 * mutt_ts_capability()
 * perform_and()
 * perform_or()
 * pgpring_string_matches_hint()
 * pgp_use_gpg_agent()
 * prepare_sidebar()
 * select_next()
 * select_next_new()
 * select_prev()
 * select_prev_new()
 * string_is_address()

string_is_address

select_prev_new

select_prev

select_next_new

select_next

prepare_sidebar

pgp_use_gpg_agent

pgpring_string_matches_hint

perform_or

perform_and

mutt_ts_capability

mutt_pop_list

mutt_needs_mailcap

mutt_match_spam_list

mutt_matches_list

mutt_limit_current_thread

mutt_group_match

mutt_comp_can_read

mutt_comp_can_append

mutt_addr_is_user

mh_valid_message

mbox_has_new