build: fix distcheck

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

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]

fill out .mailmap

[ci skip]

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

match_reference

maildir_add_to_context

link_threads

is_display_corrupting_utf8

is_day_name

is_context_available

imap_has_flag

hostname_match

get_initials

feature_enabled

crypt_is_numerical_keyid

create_hcache_dir

count_body_parts_check

compare_flags

cmd_queue_full

check_idn

check_for_mailing_list

check_certificate_expiration

check_certificate_cache

check_boundary

address_uses_unicode

addresses_use_unicode

addrcmp

add_quote_suffix

Fix memleak when attaching files.

merge: fix coverity defects

 * coverity: strfcpy
 * coverity: add variable - function arg could be NULL/invalid
 * coverity: add variable - failed function leads to invalid variable
 * coverity: add variable - Context could become NULL
 * coverity: add variable - alloc/strdup could return NULL
 * coverity: add variable - route through code leads to invalid variable
 * coverity: remove variable test
 * coverity: test functions
 * coverity: tidy switches
 * coverity: unused variables
 * coverity: refactor only
 * coverity: check for buffer underruns
 * coverity: fix leaks
 * coverity: minor fixes

merge stable

Fix memleak when attaching files.

coverity: minor fixes

Fix a few minor bugs.

coverity: fix leaks

Fix resource leaks.

coverity: check for buffer underruns

Prevent buffer underruns caused by functions that can return -1.

coverity: refactor only

Refactor code for clarity.

coverity: unused variables

Remove unused variables.

coverity: tidy switches

Simplify a few switch cases:
- Factor out trivial fall throughs
- Document any other fall throughs

coverity: test functions

Test the return value of some functions that could fail.

coverity: remove variable test

Remove a few simple tests where the condition is guaranteed to be true.

coverity: add variable - route through code leads to invalid variable

coverity: add variable - alloc/strdup could return NULL

coverity: add variable - Context could become NULL

coverity: add variable - failed function leads to invalid variable

coverity: add variable - function arg could be NULL/invalid

coverity: strfcpy

Replace strncpy with strfcpy to avoid unterminated strings.

merge: upstream fixes (mutt/default)

 * Remove redraw flag setting after mutt_endwin().
 * Change km_dokey() to pass SigWinch on for the MENU_EDITOR. (see #3877)
 * Separate out the compose menu redrawing. (see #3877)
 * Separate out the index menu redrawing. (see #3877)
 * Prepare for pager redraw separation. (see #3877)
 * Separate out the pager menu redrawing. (see #3877)
 * Don't create query menu until after initial prompt. (see #3877)
 * Silence imap progress messages for pipe-message. (see #3929)
 * Ensure mutt stays in endwin during calls to pipe_msg() (closes #3929)

Ensure mutt stays in endwin during calls to pipe_msg() (closes #3929)

The previous commit solved the problem reported in #3929: progressbar
output while downloading a message via IMAP was refreshing the mutt
ncurses session after launching the pipe program.

To ensure another place in the code doesn't inadvertantly generate
output, wrap OPTKEEPQUIET around the calls to
pipe_msg()/mutt_wait_filter() too.

Silence imap progress messages for pipe-message. (see #3929)

_mutt_pipe_message() calls endwin(), and then calls pipe_msg().  If an
imap message body hasn't already been downloaded, this can end up
calling imap_fetch_message().

The progress messages in imap_fetch_message() were restoring curses,
just after extract_url was running.  This was leading to a condition
where mutt curses didn't think the screen had changed after
extract_url exited.

There was already a check for isendwin() inside imap_fetch_message(),
but it wasn't wrapped around the progressbar creation/usage.  Add a
check for those places too.