buffer: tidy checks for empty Buffers

Fix memory leak in compose write-fcc function

The outer multipart was not being freed after writing.

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

Convert compress setup_paths() to use buffer

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

Convert compose menu fname variable to buffer

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

Convert buffer callers of mutt_get_field() to use buffer version

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

Create mutt_buffer_get_field()

This will reduce the buffer->dptr fiddling all over the place into one
spot until mutt_enter_string() can be converted.

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

Fix mutt_save_message() "enter_fname()" to use buffer version

Commit be632f11 converted this function to use the buffer pool, but
accidentally used the non-buffer prompt when there was already a
buffer-converted version.

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

Convert compose check_attachments() to use buffer pool

Modify error message so that the filename comes last, to prevent a
long path from hiding the error message or prompt.

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

Convert mutt_complete() to use the buffer pool

Add helper functions mutt_buffer_substrcpy() and
mutt_buffer_concatn_path().

Remove mutt_concatn_path() because mutt_complete() was the only
caller.

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

Convert bcache to use buffer pools

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

Fix memory leak when attaching messages

Setting actx->idx[]->content->parts to NULL causes a memory leak,
because message attachments store a BODY there that needs to be freed.

I looked through the various use cases of ci_send_message() and
actually believe content->parts will be NULL for all other cases
except message attachments.  From the comment added to
delete_attachment():

  Other ci_send_message() message constructors are careful to free
  any body->parts, removing depth:
   - mutt_prepare_template() used by postponed, resent, and draft files
   - mutt_copy_body() used by the recvattach menu and $forward_attachments.

  I believe it is safe to completely remove the "content->parts =
  NULL" statement.  But for safety, am doing so only for the case
  it must be avoided: message attachments.

Co-authored-by: Austin Ray <austin@austinray.io>

Properly revert on an autocrypt postpone failure

Co-authored-by: Austin Ray <austin@austinray.io>

merge: fix new mail behaviour

Fix a number of regressions to do with new mail.

 * fix: mbox_check_stats() retvals
 * fix mbox's handling of new mail
 * fix <change-folder> when $change_folder_next is set
 * fix sidebar colour for 'new' mailboxes
 * change check_stats() to return number of new emails

change check_stats() to return number of new emails

fix sidebar colour for 'new' mailboxes

This depends on the new-ness of a Mailbox, not the presence of new
emails.

fix <change-folder> when $change_folder_next is set

fix mbox's handling of new mail

`Mailbox.has_new` is tied to the timestamp of an mbox, not its count of
new mail.  Visiting a Mailbox will clear the flag, touch(1) the file.

Whether the new emails get changed to old emails depends on $mark_old.

fix: mbox_check_stats() retvals

Change mbox_check_stats() to clearly show when a Mailbox has new mail.
Return:
 * -1 Error
 *  0 Success, no new mail
 *  1 Success, some new mail

A bug was introduced when mbox_check_stats() was created that caused
'mbox' Mailboxes to always appear as if they had new mail.

drop unused variable

'suffix' hasn't been used since the iconv code was stripped out.

initialise gpgme pointers

These typedef'd types are all pointers, so initialise them to NULL.

doxy: tidy wrapper comments

window: fix clear_screen

The compose screen wasn't being cleared properly by just `clearok()`.

build: WIFEXITED is defined in sys/wait.h

merge: refactor use of curses

Tidy:
 * eliminate curses from Progress Bar
 * move colour definitions to color.h
 * tidy naming of color functions
 * move KeyEvent out of mutt_curses.h

Refactor:
 * SET_COLOR() to mutt_curses_set_color()
 * NORMAL_COLOR() to mutt_curses_set_color()
 * ATTR_SET() to mutt_curses_set_attr()
 * curs_set() to mutt_curses_set_cursor()
 * beep() to mutt_beep()
 * addch() to mutt_window_addch()
 * addnstr() to mutt_window_addnstr()
 * addstr() to mutt_window_addstr()
 * clrtobot() to mutt_window_clrtobot()
 * move() to mutt_window_move_abs()
 * printw() to mutt_window_printf()
 * mutt_window_mvaddstr()
 * clearok() to mutt_window_clear_screen()
 * mutt_window_getxy() to mutt_window_get_coords()

refactor: mutt_window_getxy() to mutt_window_get_coords()

Encapsulate a curses function to reduce dependencies.

refactor: clearok() to mutt_window_clear_screen()

Encapsulate a curses function to reduce dependencies.

refactor: mutt_window_mvaddstr()

Encapsulate a curses function to reduce dependencies.

refactor: printw() to mutt_window_printf()

Encapsulate a curses function to reduce dependencies.

refactor: move() to mutt_window_move_abs()

Encapsulate a curses function to reduce dependencies.
This function works with absolute screen coordinates rather than coords
based on a MuttWindow.  This function should be eliminated soon.

refactor: clrtobot() to mutt_window_clrtobot()

Encapsulate a curses function to reduce dependencies.

refactor: addstr() to mutt_window_addstr()

Encapsulate a curses function to reduce dependencies.

refactor: addnstr() to mutt_window_addnstr()

Encapsulate a curses function to reduce dependencies.

refactor: addch() to mutt_window_addch()

Encapsulate a curses function to reduce dependencies.

refactor: beep() to mutt_beep()

Encapsulate a curses function to reduce dependencies.

refactor: curs_set() to mutt_curses_set_cursor()

Encapsulate a curses function to reduce dependencies.

refactor: ATTR_SET() to mutt_curses_set_attr()

Encapsulate a curses function to reduce dependencies.

refactor: NORMAL_COLOR() to mutt_curses_set_color()

Encapsulate a curses function to reduce dependencies.

refactor: SET_COLOR() to mutt_curses_set_color()

Encapsulate a curses function to reduce dependencies.

refactor: move KeyEvent out of mutt_curses.h

KeyEvent doesn't belong in mutt_curses.h.

refactor: tidy naming of color functions

Tidy colour functions for consistency.

refactor: move colour definitions to color.h

The colour defines and structures don't depend on curses.

refactor: eliminate curses from Progress Bar

Change the Progress Bar to use the MuttMessageWindow rather than writing
to absolute screen coordinates.

iwyu: backtrace

clang-format

Unclobber mutt_parse headers

Recurse into subparts when counting attachments

Fixes #1782

fix empty buffer

merge: fix coverity defects

 * config: expand flags typedef
 * fix Slist usage
 * nntp: check pointers
 * fix pointer checks in generate_body()
 * fix leak in imap_parse_path()
 * fix leak in mutt_attach_mail_sender()
 * fix pointer checks in rfc2047_decode()
 * check the buffer in enriched_putwc()
 * fix var type in text_enriched_handler()
 * check retval of mutt_str_atoui()
 * check the buffer in enriched_puts()
 * ensure that the buffer always exists in mutt_buffer_alloc()

ensure that the buffer always exists in mutt_buffer_alloc()

check the buffer in enriched_puts()

check retval of mutt_str_atoui()

fix var type in text_enriched_handler()

The fgetwc() return value was being truncated into a wchar_t.
This meant the error condition couldn't be detected.

check the buffer in enriched_putwc()

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