merge: replace mutt_mktemp() with mutt_file_mkstemp()

 * pgp_class_application_handler()
 * pgp_class_decrypt_mime()
 * pgp_class_encrypted_handler()
 * pgp_class_encrypt_message()
 * pgp_class_traditional_encryptsign()
 * pgp_class_verify_one()
 * pgp_decrypt_part()
 * pgp_find_hash()
 * pgp_gpgme_decrypt_mime()
 * pgp_gpgme_encrypted_handler()
 * pgp_gpgme_extract_keys()
 * smime_class_build_smime_entity()
 * smime_class_decrypt_mime()
 * smime_class_invoke_import()
 * smime_class_verify_one()
 * smime_extract_certificate()
 * smime_extract_signer_certificate()
 * smime_gpgme_application_handler()
 * smime_gpgme_decrypt_mime()
 * smime_handle_cert_email()
 * smime_handle_entity()

Remove errfile from smime_handle_entity

Remove outfile from smime_handle_entity

Remove tmptmpfname from smime_handle_entity

Remove ftmpname from smime_handle_cert_email

Remove tempfile from smime_gpgme_decrypt_mime

Remove tempfile from smime_gpgme_application_handler

Remove tmpfname from smime_extract_signer_certificate

Remove tmpfname from smime_extract_certificate

Remove smimeerrfile from smime_class_verify_one

Remove tmpfname from smime_class_invoke_import

Remove tempfile from smime_class_decrypt_mime

Remove smimeerrfile from smime_class_build_smime_entity

Remove tmpfile from pgp_gpgme_extract_keys

Remove tempfile from pgp_gpgme_encrypted_handler

Remove tempfile from pgp_gpgme_decrypt_mime

Remove tempfile from pgp_find_hash

Remove pgperrfile from pgp_decrypt_part

Remove pgperrfile from pgp_class_verify_one

Remove pgperrfile from pgp_class_traditional_encryptsign

Remove pgperrfile from pgp_class_encrypt_message

Remove tempfile from pgp_class_encrypted_handler

Remove tempfile from pgp_class_decrypt_mime

Remove pgperrfile from pgp_class_application_handler

Remove pgpoutfile from pgp_class_application_handler

Put bit fields together

Use the enum constant to initialise

teach docs about true and false

merge: NeoMutt 2018-06-22

sync translations

clang-format

update contributors

changelog

version 2018-06-22

merge: add doxygen docs

 * document pattern functions
 * document sendlib functions
 * document send functions
 * document recvattach functions

document recvattach functions

document send functions

document sendlib functions

document pattern functions

merge: update translations

 * Update German translation
 * Update ja translation
 * Update zh_CN translation

Update German translation

update German translation

Update zh_CN translation

Update zh_CN translation

Update zh_CN translation

Update zh_CN translation

Update zh_CN translation

Update ja translation

Update zh_CN translation

Fix env var quoting in backticks

Issue #1262

ensure Buffers are initialised

merge: upstream fixes

 * Add $pgp_check_gpg_decrypt_status_fd.
 * Add GnuPG status fd checks for inline pgp.
 * Allow larger passphrase timeout values
 * mutt_str_atol: better error handling

mutt_str_atol: better error handling

Reset errno to 0 before calling strtol and testing it in mutt_atol.

Otherwise providing LONG_MAX+1 then LONG_MAX gave an error for LONG_MAX too.

* Detect overflow on negative numbers too (< LONG_MIN).

* Distinguish between format error and number overflow as already
  expected (e.g. for pgp_timeout and smime_timeout).

Allow larger passphrase timeout values

This came up in the comp.mail.mutt newsgroup where a user wasn't
satisfied with the SHORT_MAX seconds ~9 hours limit on passphrase
timeouts.

For the first time made it necessary for the options parser to be
able to parse numbers as long values. Also, introduced
mutt_add_timeout() to detect possible overflow before adding a
timeout to a time_t value and truncate to TIME_T_MAX instead.

Add GnuPG status fd checks for inline pgp.

The difficulty is that "BEGIN PGP MESSAGE" could be a signed and
armored part, so we can't fail hard if it isn't encrypted.

Change pgp_check_decryption_okay() to return more status codes, with
>=0 indicating an actual decryption; -2 and -1 indicating plaintext
found; and -3 indicating an actual DECRYPTION_FAILED status code seen.

Fail hard on -3, but change the message for -2 and -1 to indicate the
message was not encrypted.

Add $pgp_check_gpg_decrypt_status_fd.

If set (the default) mutt performs more thorough checking of the
$pgp_decrypt_command status output for GnuPG result codes.

Ticket #39 revealed that GnuPG (currently) does not protect against
messages that have been manipulated to contain an empty encryption
packet followed by a plaintext packet.

A huge thanks to Marcus Brinkmann for researching this issue, taking
the time to report it to us (and the GnuPG team), and taking even more
time to clarify exactly what needed to be checked for.   

tidy init.h values

strip down .gitignore

Get rid of all the things we don't see any more, like autoconf.

Refactor resize.c into separate functions

Splits up mutt_resize_screen() into its separate function to improve readability.

Closes #1149

merge: minor improvements

 * light tidy
 * coverity: fix defects
 * Put variable declaration inside case-break into braces.
 * Trailing spaces
 * Insert extern
 * One definition is enough
 * External linkage was not intended
 * Remove doubled #include
 * Redundant if clause

Redundant if clause

Remove doubled #include

External linkage was not intended

One definition is enough

Insert extern

The variable was already defined in "git_ver.h".

Trailing spaces

Put variable declaration inside case-break into braces.

coverity: fix defects

light tidy

Fix for missing fingerprint

If GPGME doesn't return a key fingerprint, don't crash.

Fix off-by-one in index jump

In an index of 'n' items, you couldn't jump to 'n', only (n-1).

Fixes #1261

acutest: sync to upstream

This file is from the Acutest project:

- https://github.com/mity/acutest

Specifically, from commit: `03885e23`

fcc_attach: Don't ask every time

Option fcc_attach used to ask in any case, if there were attachment nad if there weren't.
In case there is no attachment, there is no need to ask.

fix non-empty buffers

Insert a few `mutt_buffer_reset()`s.

Commit 9d6a2418 replaced a lot of `snprintf()`s with `mutt_buffer_printf()`.
Unfortunately, a few of the original lines were relying on `snprintf()`
writing over the beginning of the buffer.

doxygen: document mh functions

doxygen: document menu functions

drop memset for simple neomutt structs

drop memset for simple arrays

merge: more ncrypt tidying

 * drop unused param in pgp_make_key_attachment()
 * drop unused param in pgp_extract_keys_from_attachment_list()
 * boolify need_passphrase
 * boolify params
 * boolify retval

boolify retval

boolify params

boolify need_passphrase

drop unused param in pgp_extract_keys_from_attachment_list()

The function wasn't being use to look for tagged messages.
So, it's been renamed to pgp_extract_key_from_attachment()

drop unused param in pgp_make_key_attachment()

It was unused by any of the callers of the API wrapper:
crypt_pgp_make_key_attachment().

merge: mass tidy-up of ncrypt

 * drop unnec typedefs
 * fold CryptModuleFunctions into CryptModuleSpecs
 * drop redirect functions
 * tidy headers
 * doxygen: Document the encryption API
 * doxygen: document api implementors
 * unify the naming of all the ncrypt API functions
 * doxygen: document the ncrypt api wrappers

doxygen: document the ncrypt api wrappers

unify the naming of all the ncrypt API functions

doxygen: document api implementors

doxygen: Document the encryption API

tidy headers

drop redirect functions

fold CryptModuleFunctions into CryptModuleSpecs

drop unnec typedefs

merge: minor improvements

 * tidy bcache
 * boolify function params
 * doxy: add function comments
 * fix comment order
 * tidy hcache definitions

tidy hcache definitions

fix comment order

doxy: add function comments

boolify function params

tidy bcache

revert dodgy type changes

Some of the type changes I introduced in 309ecbf4 would cause problems
when strtol() and atoi() return errors.