fix pointer checks in rfc2047_decode()

fix leak in mutt_attach_mail_sender()

fix leak in imap_parse_path()

fix pointer checks in generate_body()

nntp: check pointers

fix Slist usage

Some config variables were missing NULL checks for Slist types.

config: expand flags typedef

The recent addition, CS_DUMP_SHOW_DEPRECATED, didn't fit into the flags'
typedef.

merge: optimise #include usage

 * fix cppcheck warnings
 * compose: fix minor dependency problem
 * build: check for <syscall.h> too
 * fix error checking in parse_keymap()
 * fix checks in menu_search()
 * fix checks in progress_choose_increment()
 * fix prototypes in md5.h
 * iwyu: optimise #include usage
 * iwyu: pragma keep some #includes
 * iwyu: pragma exports

iwyu: pragma exports

Define some header files as 'facades' -- wrappers for libraries.

iwyu: pragma keep some #includes

After pre-processing, some macros disappear, which means that iwyu
thinks that some headers aren't needed.

iwyu: optimise #include usage

fix prototypes in md5.h

fix checks in progress_choose_increment()

fix checks in menu_search()

fix error checking in parse_keymap()

build: check for <syscall.h> too

compose: fix minor dependency problem

Define the struct before it's referenced in a macro.

fix cppcheck warnings

merge: fix lgtm warnings

 * tidy mutt_draw_tree()
 * tidy mutt_yesorno()
 * reduce timestamp limits
 * rename global Lua

rename global Lua

Globals should have longer names

reduce timestamp limits

The two tests were only applicable on 32-bit architectures.

tidy mutt_yesorno()

tidy mutt_draw_tree()

docs: update credits

build: fix contrib install

config: add missing DT_COMMAND flags

clang-format

fix: crash in index

Before displaying a message, we check that `Context->mailbox` isn't
`NULL`.  However, events in `mutt_display_message()` can cause the
`Context` to be closed.

Fixes: #1854

travis: upgrade to Ubuntu 18.04 (Bionic)

fix: mutt_any_key_to_continue()

Originally, this function relied on `fflush(stdin)` to clear the input
buffer, but that isn't portable.  This version changes the terminal
settings, temporarily, so that it can read single characters without
blocking, or line-buffering.

merge: upstream buffer refactoring

 * Convert mutt_display_message() to buffer pool
 * Convert mutt_save_message() to use buffer pool
 * Convert recvattach save_attachment functions to use buffer pool
 * Convert query_pipe_attachment to use buffer pool
 * Convert print_attachment_list to use buffer pool

Convert print_attachment_list to use buffer pool

Co-authored-by: Richard Russon <rich@flatcap.org>

Convert query_pipe_attachment to use buffer pool

Co-authored-by: Richard Russon <rich@flatcap.org>

Convert recvattach save_attachment functions to use buffer pool

Convert utility functions prepend_curdir() and mutt_check_overwrite()
to use struct Buffer, as they are only used by those two functions.

Co-authored-by: Richard Russon <rich@flatcap.org>

Convert mutt_save_message() to use buffer pool

Change imap_copy_messages() dest parameter to const char *.

mutt_default_save() will be converted to use a buffer next, removing
the dptr fixups.

Co-authored-by: Richard Russon <rich@flatcap.org>

Convert mutt_display_message() to buffer pool

Remove an unused mime-type string generation at the top of the
function too.  The code using the mime type was removed a long time
ago in commit 246198ae.

Co-authored-by: Richard Russon <rich@flatcap.org>

merge: trivial fixes

 * rename TunnelSockData fd members
 * compose: fix nntp field clearing
 * boolify: print_enriched_string()
 * boolify: mutt_simple_format()
 * menu: use Help and Message windows directly

menu: use Help and Message windows directly

The Help and Message windows are global.
There's no need to keep a private copy of their pointers.

boolify: mutt_simple_format()

boolify: print_enriched_string()

compose: fix nntp field clearing

Using clrtoeol() would overwrite windows to the right.
mutt_window_clrtoeol() is limited to the current window.

rename TunnelSockData fd members

For consistency with other code

Fix 293664 by restoring the rounding on the correct branch

Implement mutt_date_epoch()

Closes: #1824
- Use mutt_date_epoch() instead of time()
- Use mutt_date_epoch_ms in progress
- Undo type conversion, avoid double call to mutt_date_epoch
- Give a magic number a name

config: don't dump deprecated items

trans: remove two auto-generated files

Update autosetup

update translations

sync translations

Update zh_CN translation

Update zh_CN translation

zh_CN.po: save with Poedit

de.po: proper translation for term collapse

The intention of the term "collapse" is something like opening and
closing an item. The initial translation which was added two decades
ago means something like "breaks by weakness", which does not fit in
the context of mutt.

Use a term which means literally "opening and closing an item, like a
book".

Signed-off-by: Olaf Hering <olaf@aepfle.de>

Update lt_LT translations

Translated the new autocrypt feature that I've never actually seen in
the UI (because I'm running the latest stable neomutt release because
stability is important to me).  So I'm sure the translation is excellent
and completely error-free.

Update lt_LT translations

Fix a fuzzy one.

There remain many missing and fuzzy translations about what looks to be
a new neomutt feature called 'autocrypt'.  I'll get to these later,
hopefully.

docs: fix formatting of example

fix duplicated translation string

merge: Sort Makefile lists

 * AUTO.DEF: Alphabetize lists
 * CONTRIB: MAKEFILE: Alphabetize lists
 * DOC: MAKEFILE: Alphabetize lists
 * ROOT: MAKEFILE: Alphabetize lists
 * PO: MAKEFILE: Alphabetize lists
 * TEST: MAKEFILE: Alphabetize lists
 * AUTO.DEF: Alphabetize tcl if list

Closes: #1835

AUTO.DEF: Alphabetize tcl if list

This patch alphabetizes and wraps the tcl lists given to if
statements for issue #1835.

Signed-off-by: Mark Stenglein <mark@stengle.in>

TEST: MAKEFILE: Alphabetize lists

This patch alphabetizes the lists in test/Makefile.autosetup for
issue #1835

Lists are kept unwrapped for readability.

Signed-off-by: Mark Stenglein <mark@stengle.in>

PO: MAKEFILE: Alphabetize lists

This patch alphabetizes and wraps the lists present in
po/makefile.autosetup for issue #1835

Signed-off-by: Mark Stenglein <mark@stengle.in>

ROOT: MAKEFILE: Alphabetize lists

This patch alphabetizes and wraps all of the lists in the
root Makefile.autoconfig for issue #1835

Signed-off-by: Mark Stenglein <mark@stengle.in>

DOC: MAKEFILE: Alphabetize lists

This patch alphebatizes and wraps the lists present in
doc/makefile.autosetup for issue #1835

Signed-off-by: Mark Stenglein <mark@stengle.in>

CONTRIB: MAKEFILE: Alphabetize lists

This patch alphabetizes and wraps the lists in this file for issue #1835

Signed-off-by: Mark Stenglein <mark@stengle.in>

AUTO.DEF: Alphabetize lists

This patch alphabetizes the lists present in this file for issue #1835

Signed-off-by: Mark Stenglein <mark@stengle.in>

doxygen: more defines

rename variable alph to hex

For clarity

fix spelling, whitespace

Tidy the progress API (#1822)

* Start tidying the progress API

Issue #1180

* Zero increment is not used

* Reduce buffer size, 24 chars are enough for any size

* Tidy and clarify mutt_progress_init

* Let's keep the original bytes value in pos

* Restore rounding pos to inc

* Add mutt_progress_done, start using it

* Revert "Add mutt_progress_done, start using it"

This reverts commit 92073d9145b33e8ea9d37ecea41a1e6da4cb1058.

docs: fix man page generation

Several small mistakes were causing the man pages to be generated
incorrectly.

Fix mutt_autocrypthdr_new return type

merge: lots of tidying/renaming

 * tidy code
 * rename rfc1524.c to mailcap.c
 * rename mailcap structs/functions
 * unify new/free() functions 1
 * unify new/free() functions 2
 * unify new/free() functions 3
 * myvarlist_free

myvarlist_free

unify new/free() functions 3

'Free' functions:
- Name ends in `_free()`
- Take a `**ptr`
- Creates a native pointer
- Sets pointer to `NULL`

Functions changed:
- compress_info_free()
- crypt_key_free()
- key_free()
- lookup_free()
- maildir_entry_free()
- maildir_free()
- menu_free()
- mhs_sequences_free()
- mix_type2_list_free()
- mutt_actx_entries_free()
- mutt_attachmatch_free()
- mutt_colors_free()
- mutt_color_free()
- mutt_color_list_free()
- mutt_keys_free()
- mutt_opts_free()
- myvar_free()
- parameter_free()
- pgp_key_free()
- pgp_uid_free()
- recipient_set_free()
- smime_key_free()

'New' functions:
- Name ends in `_new()`
- Must be a 'new' function if there's a 'free' function

Functions changed:
- lookup_new()
- maildir_entry_new()
- myvar_new()

unify new/free() functions 2

'Free' functions:
- Name ends in `_free()`
- Take a `**ptr`
- Creates a native pointer
- Sets pointer to `NULL`

Functions changed:
- color_line_free()
- mutt_actx_free()
- mutt_attachmatch_free()
- mutt_autocrypthdr_free()
- mutt_autocrypt_db_account_free()
- mutt_autocrypt_db_gossip_history_free()
- mutt_autocrypt_db_peer_free()
- mutt_autocrypt_db_peer_history_free()

'New' functions:
- Name ends in `_new()`
- Must be a 'new' function if there's a 'free' function

Functions changed:
- class_color_new()
- color_line_new()
- hash_new()
- mutt_actx_new()
- mutt_attachmatch_new()
- mutt_autocrypthdr_new()
- parameter_new()
- pgp_keyinfo_new()

unify new/free() functions 1

- Name ends in `_free()`
- Take a `**ptr`
- Creates a native pointer
- Sets pointer to `NULL`

Functions changed:
- cs_free()
- mailcap_entry_free()
- mutt_addr_free()
- mutt_alias_free()
- mutt_alias_new()
- mutt_body_free()
- mutt_enter_state_free()
- mutt_env_free()
- mutt_seqset_iterator_free()
- nntp_edata_free()
- query_free()
- remailer_free()
- remailer_new()

rename mailcap structs/functions

rename rfc1524.c to mailcap.c

tidy code

- rename variables: b to buf
- fix Buffer check
- remove redundant conditionals

merge: refactor Buffer handling

 * Tidy some Buffer API, keep most buffers on the stack
 * Keep spam member of struct Envelope by value
 * Kill mutt_buffer_from
 * Move buffer_new / buffer_free to static functions in pool.c
 * Add mutt_buffer_make to create a Buffer on the stack
 * Use mutt_buffer_is_empty instead of mutt_buffer_len where it makes sense

Use mutt_buffer_is_empty instead of mutt_buffer_len where it makes sense

Add mutt_buffer_make to create a Buffer on the stack

Move buffer_new / buffer_free to static functions in pool.c

Kill mutt_buffer_from

Keep spam member of struct Envelope by value

Tidy some Buffer API, keep most buffers on the stack

boolify 'optional' variables

Co-authored-by: Pietro Cerutti <gahr@gahr.ch>

rename Pattern members

doxy: tidy list comments

rename lists for consistency

- Move `entries` to end of the struct
- Separate declaration from use
- Rename `ColorLineHead`      to `ColorLineList`
- Rename `LogList`            to `LogLineList`
- Rename `MyVarHead`          to `MyVarList`
- Rename `ObserverHead`       to `ObserverList`
- Rename `PatternHead`        to `PatternList`
- Rename `RegexListNode`      to `RegexNode`
- Rename `ReplaceListNode`    to `Replace`
- Rename `TagHead`            to `TagList`
- Rename `TagNode`            to `Tag`
- Rename `UrlQueryStringHead` to `UrlQueryList`
- Rename `UrlQueryString`     to `UrlQuery`

unify time handling

- Add mutt_date_epoch_ms API, convert gettimeofdays to use the new API
- Add mutt_date_sleep_ms as a general sleep API

Remove useless abstraction

merge: light tidying

 * clang-format
 * pager: rename PagerRedrawData.index to menu
 * drop: unused sha1 code
 * rename two duplicate `struct Entry`
 * rename two duplicate `struct Syntax`

rename two duplicate `struct Syntax`

rename two duplicate `struct Entry`

drop: unused sha1 code

pager: rename PagerRedrawData.index to menu

clang-format

Make sure Progress doesn't reference an out-of-scope buffer

Cleanup mutt_progress_init's callers

Fixes: #1819

Add: mailbox_path() to tidy Mailbox usage