free resources

history: free resources

history: tidy/sync functions

- sync format_str and menu_entry functions
- reduce scope of variables
- document functions

history: move static functions first

comment and reorder params

comment list functions

rename OPT_ variables

add missing extern

Clarify the relation between NeoMutt and Mutt

Fixes a discrepancy between the website and this README on the topic of "is it a fork?".

 Let mutt_ch_choose report conversion failure (#1137)

* Let mutt_ch_choose report conversion failure

Issue #1136

* Let mutt_ch_convert_string return errno if iconv(..) fails

Previously, mutt_ch_convert_string returned error only if any arguments
failed the sanity check or iconv_open() failed.

With this change, an error value is returned even if the iconv()
conversion fails, such as when the input string contains an invalid
character in the source character set (EILSEQ).

I have looked at each caller and it seems to me that this change won't
introduce any misbehaviour, but I'm far from being 100% confident.

Issue #1136

* test int functions against 0

* Better error reporting in mutt_ch_iconv

Issue #1136

doc: Describe logging capabilities of NeoMutt

Describe the logging capabilities of NeoMutt in the User Guide. The
section currently covers the different log message types (perror, error,
warning, informational debug), where they are shown and how to enable
debug log messages.

Rename <show-messages> to <show-log-messages>

Rename the function <show-messages> to <show-log-messages> as well as
its symbolic constant OP_SHOW_MESSAGES to OP_SHOW_LOG_MESSAGES.

Background is that 'messages' (in the context of neomutt) refers usually
to emails instead of log messages.

Describe <show-messages> more precisely

The term 'message' is used for log-messages as well as for ordinary
mails. In the help 'message' usually means an email. Adjust the
description of <show-message>/OP_SHOW_MESSAGE to clarify that log and
debug messages are meant.

Remove reminiscent of txt2c

txt2c is replaced by a routine in the configure/autosetup script.
Update .gitignore accordingly.

Generate .clang_complete

For users of https://github.com/Rip-Rip/clang_complete, this allows to
always keep the .clang_complete file in sync with CFLAGS.

Build: restructure ncurses/tinfo checks (#1119)

* Build: restructure ncurses/tinfo checks

Issue #1118

* Fallback to ncurses library for tgetent

This fixes the configure phase in NetBSD, which doesn't have a separate
tinfo library in base.

Issue #1118

merge: fix cppcheck warnings

 * reduce scope of variables
 * printf format
 * make logic clearer
 * misc fixes
 * match prototypes to their functions

cppcheck: match prototypes to their functions

cppcheck: misc

cppcheck: make logic clearer

cppcheck: printf format

split out progress functions

split out window functions

split out terminal setting

merge: rename functions for consistency

 * body functions
 * header functions
 * enter_state functions
 * alias functions
 * menu functions
 * rfc822 functions

merge: light tidying

 * test ints against 0
 * test int functions against 0
 * tidy comments / docs
 * tidy mutt_str_strdup
 * tidy uses of mutt_str_getenv

merge: refactor config parsing on startup

 * nntp config
 * mailcap config
 * tmpdir config
 * editor, visual config
 * separate get from set Charset

refactor: separate get from set Charset

refactor: editor, visual config

Precedence: environment, config, code

refactor: tmpdir config

Precedence: environment, config, code

refactor: mailcap config

Precedence: environment, config file, code

refactor: nntp config

Precedence: command line, environment, config file, system file

rename rfc822 functions

rename menu functions

rename alias functions

rename enter_state functions

rename header functions

rename body functions

Enhance neomutt -v with NCURSES_VERSION_PATCH

tidy uses of mutt_str_getenv

tidy mutt_str_strdup

refactor: startup hostname

Delay the code that finds the FQDN until after the config and command
line have been processed.

This means that some users, with mis-configured network lookups, can work
around a long timeout during startup.

refactor: move the envlist code to the library

The envlist code keeps a private copy of the environment.

tidy comments / docs

test int functions against 0

cppcheck: reduce scope of variables

test ints against 0

build: fix bmake's lack of variable expansion in "include" lines

slang: don't endwin() if slang wasn't started

merge: trivial improvements

 * docs: update README
 * sort switch cases
 * sort option definitions
 * sort colour fields
 * remove unused variables
 * initialise variables
 * iwyu: fix #include usage
 * docs: add missing comment
 * refactor spanning ifs
 * check for null pointer

check for null pointer

refactor spanning ifs

Refactor an `if` that spans an entire function, to return early.

docs: add missing comment

iwyu: fix #include usage

build: fix dependencies for test dir

initialise variables

scan-build warns that these variables may not get set elsewhere

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

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