remove unused variables

Remove variables that are set, but not used again.

sort colour fields

sort option definitions

sort switch cases

There are no code changes here.
The 'case' statements have been sorted where:
- There's a clear order, e.g. alphabetical
- Where they all have break/return statements

docs: update README

- links contributors
- add shields

build: fix dependencies for test dir

convert init.h

Another (small) step towards the new config system

lua: fix variable getting/setting

acutest: update with latest upstream (documentation fixes)

fix Notmuch deprecation warning

We need to distinguish between two versions of `notmuch.h`.
Both declare that they are v5.0.0.  They are:

- libnotmuch 5.0 (Notmuch 0.25)
  - `notmuch_database_add_message()` is a normal function
  - `notmuch_database_index_file()` doesn't exist

- libnotmuch 5.1 (Notmuch 0.26)
  - `notmuch_database_add_message()` is **deprecated**
  - `notmuch_database_index_file()` exists

To work around this, we check for `notmuch_database_index_file()` specifically.

fix return code for 'neomutt -Z'

Fixes #1112

merge: upstream fixes

 * sync to upstream abort_noattach
 * Improve the error message when a signature is missing.
 * Change mutt_error call in mutt_gpgme_set_sender() to dprint.
 * Fix comparison of flags with multiple bits set.
 * Increase account.user/login size to 128.

Increase account.user/login size to 128.

Like the pass field in commit 52949004, the existing size was
insufficient for the reporter of issue #18.  For now, just bump up the
size, but I'll move up looking into making these dynamically sized.

Fix comparison of flags with multiple bits set.

PGPENCRYPT, PGPKEY, SMIMEENCRYPT, and SMIMEOPAQUE are all combination
flags, with multiple bits set.  In a few places these flags were
bitwise-and'ed incorrectly: expecting a non-zero result to indicate
all the bits in the flag were set.  Change those to explicitly compare
the result against the original flag.

Change mutt_error call in mutt_gpgme_set_sender() to dprint.

The message is not an error message, and flies by so quickly it has
little informative content.  Change it to use dprint instead.

Thanks to Darac Marjal for reporting the issue.

Improve the error message when a signature is missing.

Thanks to Vincent Lefèvre for reporting the issue and for the wording
suggestion!

sync to upstream abort_noattach

NeoMutt 2018-03-23

sync translations

tidy

- clang-format
- sort makefile lists
- tidy whitespace

new contributors

changelog

version 2018-03-23

Update Czech translation

Update Czech translation

error: Don't pause after duplicate messages

Fixes #1100

error: use sleep_time as error wait time

Also fix the conversion from us to ns.

doxygen: drop/tidy simple comments

merge: update translations

 * fix file list
 * remove unnec CRs
 * too few
 * too many
 * misc dupe strings
 * tidy grave quotes
 * remove un-discoverable options
 * update-po
 * update German translation
 * update German translation
 * Update Turkish translation
 * update Spanish translation
 * Polish translation update
 * Polish translation update
 * Update Portuguese (BR) translation
 * Update Portuguese (BR) translation
 * Update Portuguese (BR) translation
 * Update Portuguese (BR) translation
 * Update Czech translation
 * Update Czech translation
 * Update Czech translation
 * Update Czech translation
 * Update Czech translation
 * Update Czech translation
 * Update Czech translation
 * Update Czech translation
 * Update Portuguese (BR) translation
 * Update Portuguese (BR) translation
 * Update Portuguese (BR) translation
 * Polish translation update
 * Update Portuguese (BR) translation
 * Update Portuguese (BR) translation
 * Update Czech translation
 * Update Portuguese (BR) translation
 * Update Portuguese (BR) translation
 * update English (British) translation

merge: sort code to make docs more legible

 * docs: Sort commands
 * init.h: Sort configuration variables
 * opcodes.h: Sort opcodes
 * functions.h: Sort functions by name

merge: fix coverity defects

 * resource leak - mutt_log_start
 * range check - reset_value
 * range check - set_default_value
 * dead code - do_database_open
 * resource leak - mutt_log_set_file
 * always true - rfc2047_encode
 * null ptr deref - print_attachment_list
 * drop testing code - resolve_types
 * range check - parse_set
 * calloc size - mutt_select_file
 * check WithCrypto against zero

check WithCrypto against zero

calloc size - mutt_select_file

range check - parse_set

drop testing code - resolve_types

null ptr deref - print_attachment_list

always true - rfc2047_encode

resource leak - mutt_log_set_file

dead code - do_database_open

range check - set_default_value

range check - reset_value

resource leak - mutt_log_start

update English (British) translation

[ci skip]

Update Portuguese (BR) translation

Update Portuguese (BR) translation

functions.h: Sort functions by name

To achieve a correct sorting for all function names, it is necessary to
split some single preprocessor directives into multiple.

opcodes.h: Sort opcodes

init.h: Sort configuration variables

docs: Sort commands

Tidy space-before-tabs and multiple spaces

Support —with-sysroot configure arg (#1101)

* If --sysroot is given, prepend it to prefix for search paths

Issue #869

* Prefix is not used inside Makefiles

Issue #869

* No need to specify the empty string as opt-val's default return

Issue #869

* s/--sysroot/--with-sysroot/

Issue #869

fix: set the DebugFile

Make sure that the DebugFile is always set on startup, otherwise
`neomutt -d2` will fail and stop the app.

restore ignore_linear_white_space option

This is a dummy option -- it's not needed any more.
In the future this and several other options will be deprecated and
removed.

merge: unify logging/messaging

 * refactor cli handling of cc/bcc
 * endwin: separate curses from the message
 * sleep after every error message
 * reorganise startup to return through main()
 * refactor exits in mutt_randbuf()
 * use dedicated set_default() and reset_value()
 * redirect all messages through a log dispatcher
 * update messages (remove newline)
 * add log messages page
 * add comments markers for CLI test cases
 * fix crash with incomplete template

fix crash with incomplete template

add comments markers for CLI test cases

When starting NeoMutt, there are nearly 40 routes through main().
Add a comment for each one with a suggested command line.

add log messages page

The new command `<show-messages>` will show a backlog of 100 messages
from:
- mutt_message()
- mutt_error()
- mutt_perror()
- mutt_debug() if enabled

update messages (remove newline)

Many of the startup messages have been migrated from printf() to
mutt_error().  They no longer need a '\n'.

redirect all messages through a log dispatcher

use dedicated set_default() and reset_value()

refactor exits in mutt_randbuf()

Move the exit()s so we can call it during startup.

reorganise startup to return through main()

The single exit from main() makes it easier to clean up curses and
logging.

sleep after every error message

Mark the time of an error message.
Do not allow an error to be cleared or overwritten by a normal message
within one second.

endwin: separate curses from the message

refactor cli handling of cc/bcc

Expose EXTRA_CFLAGS_FOR_BUILD and EXTRA_LDFLAGS_FOR_BUIlD (#1098)

This allows distros to set flags that must be used both for building target and host executables.

Issue #980

lua: fix crash when setting a string

Generally, for strings `Option.var` is a `char **` (a pointer to a
string pointer).  The Lua setting code uses it differently.  There, it's
just a `char *`.

Fixes #1096

Make sure paths created by header/body/news cache are POSIX-compliant (#1080)

* Make sure paths created by header/body cache are POSIX-compliant

Issue #1073

* Make sure single-file header cache is sanitized

Issue #1073

* Avoid double path normalization

Issue #1073

* Make sure bcache_path works for a NULL mailbox (see POP)

Issue #1073

* Encode news cache dir path

Issue #1073

merge: trivial improvements

 * contrib: drop obsolete files
 * sort main switch options
 * doxygen: drop tables
 * fix odd case of Charset_is_utf8
 * docs: tidy homepage url
 * docs: light tidying
 * minor code tidy

minor code tidy

docs: light tidying

- substitute mutt with neomutt
- fix openSUSE capitalisation

docs: tidy homepage url

fix odd case of Charset_is_utf8

doxygen: drop tables

These are now auto-generated from the function comments.

sort main switch options

contrib: drop obsolete files

Make sure git_ver.h doesn't eat random 'g's out of tag names

Update Czech translation

Update Portuguese (BR) translation

Update Portuguese (BR) translation

Polish translation update

Update Portuguese (BR) translation

Update Portuguese (BR) translation

Update Portuguese (BR) translation

Update Czech translation

Update Czech translation

Update Czech translation

Update Czech translation

Update Czech translation

Update Czech translation

Signed-off-by: David Sterba <dsterba@suse.com>

Update Czech translation

Update Czech translation

Update Portuguese (BR) translation

Update Portuguese (BR) translation

Update Portuguese (BR) translation

Update Portuguese (BR) translation

Polish translation update

added some non-translated phrases and messages

Add subjectrx section to the muttrc man page.

Fix subjectrx example in the manual.

Polish translation update

Closes #1086