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.

fix trash mailbox

The second `for` loop was using the value of `i` as a starting place.
By changing the first loop, the second loop wasn't doing any work.

Fixes #1253

merge: update translations

 * add missing 'Plural-Forms' statement
 * sync translations
 * trans: de-dupe an ssl string
 * Update Czech translation
 * Update ja translation
 * Update Slovak translation
 * Update zh_CN translation
 * use consistent expression for memory streams

sync translations

Update zh_CN translation

use consistent expression for memory streams

Update zh_CN translation

Update zh_CN translation

Update zh_CN translation

Update ja translation

Update ja translation

trans: de-dupe an ssl string

Update Slovak translation

add missing 'Plural-Forms' statement

Update ja translation

Update zh_CN translation

Update zh_CN translation

Update zh_CN translation

Update Czech translation

Update zh_CN translation

merge: increase and unify buffer sizes

 * adjust the size of some general buffers
 * Use sizeof() rather than constants
 * Create a large buffer for command strings
 * Enlarge buffers from _POSIX_PATH_MAX to PATH_MAX
 * Enlarge buffers from LONG_STRING to PATH_MAX

Enlarge buffers from LONG_STRING to PATH_MAX

Every buffer that has a filesystem path copied into it

Enlarge buffers from _POSIX_PATH_MAX to PATH_MAX

Every buffer that has a filesystem path copied into it

Create a large buffer for command strings

Use sizeof() rather than constants

merge: small improvements

 * tidy handler.c
 * tidy for loops
 * insert {}s
 * doxy: document buffy.c
 * tidy return statements
 * move new functions to reduce header deps
 * add enum for 'set' commands
 * tidy the regex types
 * minor tidying

minor tidying

- rename parameters
- change member types
- fix doxygen typo
- make variable const