coverity: add variable - alloc/strdup could return NULL

coverity: add variable - Context could become NULL

coverity: add variable - failed function leads to invalid variable

coverity: add variable - function arg could be NULL/invalid

coverity: strfcpy

Replace strncpy with strfcpy to avoid unterminated strings.

merge: upstream fixes (mutt/default)

 * Remove redraw flag setting after mutt_endwin().
 * Change km_dokey() to pass SigWinch on for the MENU_EDITOR. (see #3877)
 * Separate out the compose menu redrawing. (see #3877)
 * Separate out the index menu redrawing. (see #3877)
 * Prepare for pager redraw separation. (see #3877)
 * Separate out the pager menu redrawing. (see #3877)
 * Don't create query menu until after initial prompt. (see #3877)
 * Silence imap progress messages for pipe-message. (see #3929)
 * Ensure mutt stays in endwin during calls to pipe_msg() (closes #3929)

Ensure mutt stays in endwin during calls to pipe_msg() (closes #3929)

The previous commit solved the problem reported in #3929: progressbar
output while downloading a message via IMAP was refreshing the mutt
ncurses session after launching the pipe program.

To ensure another place in the code doesn't inadvertantly generate
output, wrap OPTKEEPQUIET around the calls to
pipe_msg()/mutt_wait_filter() too.

Silence imap progress messages for pipe-message. (see #3929)

_mutt_pipe_message() calls endwin(), and then calls pipe_msg().  If an
imap message body hasn't already been downloaded, this can end up
calling imap_fetch_message().

The progress messages in imap_fetch_message() were restoring curses,
just after extract_url was running.  This was leading to a condition
where mutt curses didn't think the screen had changed after
extract_url exited.

There was already a check for isendwin() inside imap_fetch_message(),
but it wasn't wrapped around the progressbar creation/usage.  Add a
check for those places too.

Don't create query menu until after initial prompt. (see #3877)

A resize in the prompt will trigger a redraw, but the data won't be
loaded yet, displaying a blank screen instead of the previous menu.
Once the query is done, the data is loaded, but the menu->redraw state
has been changed by the resize.

We could manually flag a redraw, but it makes more sense visually
logically to just create the menu after the query and results are
loaded.

Separate out the pager menu redrawing. (see #3877)

The pager relies on REDRAW_SIGWINCH, so add that to _mutt_get_field().

Prepare for pager redraw separation. (see #3877)

Move some of the code inside SigWinch handling into the REDRAW part of
the code.

SigInt is handled by mutt_getch(), so remove the redundant code from
inside the pager.

Separate out the index menu redrawing. (see #3877)

Separate out the compose menu redrawing. (see #3877)

Add a custom_menu_redraw to the menu and change menu_redraw() to call
that instead if set.

Change km_dokey() to pass SigWinch on for the MENU_EDITOR. (see #3877)

Change _mutt_enter_string() to pass the SigWinch through for
_mutt_get_field() or mutt_enter_string() to handle.

Add a call to mutt_current_menu() in _mutt_get_field() to properly
redisplay the screen in that case.

Remove redraw flag setting after mutt_endwin().

There is no need to set a redraw flag in this case.  The screen will
automatically be properly redrawn on the next refresh().

Remove, lest it get propagated.

fix make dist distcheck

bool-ify structs

Convert all struct members that are used as booleans, to "bool".

fix slang bools

slang defines its own bool types which interfere with <stdbool.h>
So, undefine theirs and redefine ours.

merge: bool-ify functions

 * Convert two function to return bool
 * More boolification
 * Boolify is_parent
 * Boolify eat_* functions
 * Boolify is_ascii
 * Boolify pgp_cansign and pgp_canencrypt
 * Boolify mx_is_mh and mx_is_maildir
 * Boolify remaining mx_is_* functions
 * More boolification
 * Boolify mutt_nm_tag_complete and mutt_nm_query_complete

Boolify mutt_nm_tag_complete and mutt_nm_query_complete

Issue: #492

More boolification

Issue: #492

Boolify remaining mx_is_* functions

Issue: #492

Boolify mx_is_mh and mx_is_maildir

Issue: #492

Boolify pgp_cansign and pgp_canencrypt

Issue: #492

Boolify is_ascii

Issue: #492

Boolify eat_* functions

Issue: #492

Boolify is_parent

Issue: #492

More boolification

Issue: #492

Convert two function to return bool

Issue: #492

merge: assorted small fixes

 * build: fix warnings
 * add full neomutt version to log startup
 * drop duplicate versions of the sidebar contrib
 * drop comment from sidebar docs
 * build: remove unused macros
 * build: use SEEK_SET, SEEK_CUR, SEEK_END
 * bug: add missing break
 * build: remove constant code
 * bug: don't pass large object by value
 * fix: use correct buffer size
 * fix: bug in uncolor for notmuch tag
 * build: workaround slang warnings
 * shadow variables
 * 0 -> NULL

0 -> NULL

shadow variables

build: workaround slang warnings

The prototypes for these four functions use (char*),
whereas the ncurses versions use (const char*)
  addnstr
  addstr
  mvaddnstr
  mvaddstr

We redefine the helper macros to hide the compiler warnings.

fix: bug in uncolor for notmuch tag

A bug caused by the order of COLOR constants prevented "uncolor" working on
MT_COLOR_INDEX_TAG.

fix: use correct buffer size

One path of pop_auth_sasl() uses an incorrect buffer size parameter.

bug: don't pass large object by value

imap_delete_mailbox was passed an IMAP_MBOX object by value.

The object is >500 bytes so it would be better as a pointer.
Passing by value also led to a potential double-free.

build: remove constant code

Refactor the DOMAIN code to put it in one block.

The old code would define a constant in a #ifdef, which it then tested later.

bug: add missing break

Add a missing break to a switch.
OP_EDITOR_QUOTE_CHAR leak through into OP_EDITOR_TRANSPOSE_CHARS.

build: use SEEK_SET, SEEK_CUR, SEEK_END

Change all the uses of fseek() and fseeko() to use defined constants:
    SEEK_SET, SEEK_CUR, SEEK_END

build: remove unused macros

Remove some unused macros and substitue in one that is used just once.

drop comment from sidebar docs

The comment is no longer applicable.

drop duplicate versions of the sidebar contrib

These:
    contrib/sample.muttrc-sidebar
    contrib/sample.vimrc-sidebar
are dupes of:
    doc/muttrc.sidebar
    doc/vimrc.sidebar

add full neomutt version to log startup

Changes:
    NeoMutt/20170306 (1.8.0) debugging at level 2
to
    Neomutt 20170306-98-bf1d78 (1.8.0) debugging at level 2

build: fix warnings

educate github's linguist

GitHub creates language stats about NeoMutt using "linguist".
Unfortunately, it thinks that mutt.h and init.h are C++.

merge: upstream fixes (mutt/default)

 * Fix mutt_refresh() pausing during macro events.
 * Add a menu stack to track current and past menus.
 * Change CurrentMenu to be controlled by the menu stack.
 * Set refresh when popping the menu stack.
 * Remove redraw parameter from crypt send_menus.
 * Don't full redraw the index when handling a command from the pager.
 * Filter other directional markers that corrupt the screen.
 * Remove the OPTFORCEREDRAW options.
 * Remove SidebarNeedsRedraw.
 * Change reflow_windows() to set full redraw.
 * Create R_MENU redraw option.
 * Remove refresh parameter from mutt_enter_fname().

Remove refresh parameter from mutt_enter_fname().

Also remove it from mutt_save_message(), which used it to pass through
to mutt_enter_fname().  The callers of this already had redraw logic,
to which REDRAW_STATUS merely needed to be added.

Create R_MENU redraw option.

Previously, the R_INDEX option meant both the index as well as all
other menus.  The removal of the OPTFORCEREDRAWINDEX option caused problems
with redrawing other menus for options such as arrow_cursor.

One solution would be change R_INDEX back to meaning "everything"
except pager, but there are only a handful of options that affect
other menus.

Instead, create R_MENU to indicate options that affect either all
menus or one of the other menus beside the index and pager.

Change reflow_windows() to set full redraw.

A full redraw should always be set in this case.  This also enables us
to remove redraw flags for some options.

Remove SidebarNeedsRedraw.

The menu stack can be used to flag a redraw of the sidebar window.

Remove the OPTFORCEREDRAW options.

Use the menu stack to flag redraws for the index and pager.

Filter other directional markers that corrupt the screen.

Don't full redraw the index when handling a command from the pager.

This causes a noticable flicker when moving through messages from the pager.

Remove redraw parameter from crypt send_menus.

The parameter was used to notify the caller if the sign (a)s menu was
invoked, which displayed the key selection menu.  This is no longer
necessary with the menu stack pop operation.

Set refresh when popping the menu stack.

This removes the need for the OPTNEEDREDRAW option and MAYBE_REDRAW
macro previously used to communicate back the need to refresh after
exiting a menu.

Remove the redraw parameter from ci_bounce_message() and
mix_make_chain() which served the same purpose.

Change CurrentMenu to be controlled by the menu stack.

The pager menu is a bit tricky with respect to the menu->menu and
CurrentMenu, so add a few comments.

Add a menu stack to track current and past menus.

Change the pager to use a MENU, right now just to hold the refresh
state.

Fix mutt_refresh() pausing during macro events.

Changeset a07e8215a0ef split input buffering into two pools.
Unfortunately, the mutt_refresh() was not changed to check the correct
buffer count, resulting in unnecessary refreshes during macros.

The SSL interactive certificate prompts set OPTIGNOREMACROEVENTS and
then put up a confirmation menu.  Perhaps we've just been lucky, but
it seems we should refresh the screen in those cases if we're in the
middle of a macro.  Add a check for this option in mutt_refresh() too.

merge: Tidy the code - Part 3

 * fix: typo in browser_compare_count{_new}
 * build: remove generated files from repo
 * tidy: comments
 * build: look for translations in all files
 * fix: arguments to printf-style functions
 * tidy: license text
 * tidy: unify include-guards
 * build: tidy makefiles
 * tidy: initialise pointers
 * tidy: make strcmp-like functions clearer
 * tidy: unify sizeof usage
 * build: remove forward declarations
 * tidy: remove ()s from return

tidy: remove ()s from return

The 'return' operator doesn't require its argument to the ()'d.
This has lead to some very inconsistent code.

This commit removes the ()s from return where the argument is a single
token, e.g.
    return 42;
    return fn(x);

There are ~480 occurrences.

build: remove forward declarations

tidy: unify sizeof usage

The sizeof operator doesn't always require the argument to be ()'d.
However it wasn't used consistently.  Add ()s to all uses of sizeof.

tidy: make strcmp-like functions clearer

There are ~600 uses of strcmp-like functions:
- ascii_strcasecmp
- ascii_strcmp
- ascii_strncasecmp
- ascii_strncmp
- imap_mxcmp
- imap_wordcasecmp
- mutt_inbox_cmp
- mutt_strcasecmp
- mutt_strcmp
- mutt_strncasecmp
- mutt_strncmp
- strcasecmp
- strcmp
- strncasecmp
- strncmp
- wcscasecmp

Because strcmp returns 0 on a successful match, conditionals can easily
be misread:

    if (strcmp(s, "hello"))
    {
        /* code if string 's' DOES NOT match */
    }

These have been changed to explicitly mention the return value:

    if (strcmp(s, "hello") != 0)
    {
        /* code if string 's' DOES NOT match */
    }

tidy: initialise pointers

Initialise ~1000 pointer declarations to NULL.
This incurs a very tiny cost at runtime.

This command was used to create a vim location-list:
    grep -ni "^[ ^I]\+\<[a-z_0-9]\+ *\* *[a-z_0-9]\+;"

build: tidy makefiles

- sort out subdirs
- remove unnecessary HAVE_CONFIG_H define
- tidy source lists
- drop NLS config reference
- drop unused make targets

tidy: unify include-guards

- add missing include-guards
- unify existing include-guards
- unnecessary conditionals around whole source

tidy: license text

- add missing license text
- tidy existing license text

fix: arguments to printf-style functions

By adding an attribute to printf-style functions, the compiler can spot
errors in the arguments.

Because the attributes aren't portable, they haven't been committed.
    __attribute__((format (printf, 1, 2)))

build: look for translations in all files

List ALL of the source files to the passed to the translations.

tidy: comments

- drop CVS marker: $Id$
- convert // comments
- delete blank line between comment and function

build: remove generated files from repo

Build temporary:
po/stamp-po

Copied in by autoreconf
po/boldquot.sed
po/en@boldquot.header
po/en@quot.header
po/insert-header.sin
po/quot.sed
po/remove-potcdate.sin

fix: typo in browser_compare_count{_new}

A typo meant that sorting by "count" or "count new" could not be done in
reverse.

add mailmap entries

This covers all committers since the start of NeoMutt.

doc: use neomutt mailinglist (#483)

Closes neomutt/neomutt.github.io#30

Website page updated too.

Fix (un)sidebar_whitelist to expand paths.

Thanks to Arturo for reporting the issue.

Closes #485

Fix inbox-first functionality when using mutt_pretty_mailbox (#490)

Fix inbox-first functionality when using mutt_pretty_mailbox

Fixes #489

merge: Tidy the code - Part 2

 * build: force OPS to be rebuild after a reconfigure
 * build: fix minor code warnings / style
 * tidy: be specific about void functions
 * build: expand a few more alloc macros
 * tidy: add argument names to function prototypes
 * build: drop local copy of regex code
 * tidy: indent the docbook manual
 * tidy: rearrange code to avoid forward declarations
 * tidy: limit the scope of some functions
 * build: give the compress functions a unique name
 * tidy: use snake_case for function names
 * build: add missing newlines to mutt_debug

build: add missing newlines to mutt_debug

tidy: use snake_case for function names

e.g.
- getDate          -> get_date
- mutt_menuDestroy -> mutt_menu_destroy

build: give the compress functions a unique name

The compressed mailbox functions are passed as pointers to mutt.
Their names, e.g. "open_mailbox" are OK for compilation, but they are
ambiguous when debugging.

- prefix exported function names with "comp_"

tidy: limit the scope of some functions

Functions that are only used in one file (compilation unit), should be
declared "static".  Also, they shouldn't have the prefix "mutt_".

- make two functions static (only used in one place)
- remove "mutt_" prefix
- add missing "static" declaration

tidy: rearrange code to avoid forward declarations

This change, though huge, is quite simple.

It simply rearranges functions, within files, to remove the need to
forward-declare them.  There were ~90 such declarations.

Forward declarations are rarely *needed* and just add to the cruft that
needs to be maintained.

tidy: indent the docbook manual

Indent the xml of the docbook manual.
Put a blank line before <chapter>, <sect*> or <table>

This doesn't change the manual functionally, but it makes it easier to
navigate.

build: drop local copy of regex code

The regular expression (regex) is now a standard part of the C libraries.
Use that, rather than our 18 year old version.

tidy: add argument names to function prototypes

This isn't very useful:
int function (int, int);

This is clearer
int function (int row, int col);

build: expand a few more alloc macros

Following some early work, expand some more macros to functions:
- pgp_new_keyinfo()
- new_pattern()
- rfc822_new_address()

tidy: be specific about void functions

Some functions have been declared with "()" when they should be "(void)".

build: fix minor code warnings / style

- remove an unnecessary backslash
- put variable in same preprocessor scope
- move prototype to where it's needed

build: force OPS to be rebuild after a reconfigure

A common problem is getting build errors after running ./configure for
the second time, e.g.
    ./configure [FEW  OPTIONS]; make
    ./configure [MANY OPTIONS]; make

This commit forces keymap_defs.h to be rebuilt after a configure.

build: don't look for lua libs unless asked for

fixes: #478

merge: upstream fixes (mutt/default)

 * Add SNI support for OpenSSL. (see #3923)
 * Add SNI support for GnuTLS. (closes #3923)
 * Add shortcuts for IMAP and POP mailboxes in the file browser
 * Change OpenSSL to use SHA-256 for cert comparison. (closes #3924)
 * Fix conststrings type mismatches. (closes #3926)
 * Pass envlist to filter children too.  (closes #3922)
 * Fix mutt_envlist_set() for the case that envlist is null. (see #3922)
 * Fix setenv overwriting to not truncate the envlist. (see #3922)

Fix setenv overwriting to not truncate the envlist. (see #3922)

The refactor in 2b9c40f13e13 exposed a bug I hadn't noticed.  The
match loop performed a FREE() on the slot.  Then, below, it was
checking if (*envp) to see whether it was overwriting or creating a
new slot.  However, FREE() nulls out *envp.  This would end up
truncating the envlist just after the set slot!

Move the free down, using a mutt_str_replace(), when overwriting the
slot.

Fix mutt_envlist_set() for the case that envlist is null. (see #3922)

Pass envlist to filter children too.  (closes #3922)

The new setenv patch neglected to pass the envlist for filters too.

Unfortunately, the filter code was already set up to pass COLUMNS to
children, so it needed to be changed to add this to the envlist
instead.

Factor out mutt_envlist_set() from the parse_setenv() function, which
the filter code can then use to set COLUMNS after forking.

Fix conststrings type mismatches. (closes #3926)

The generation programs for conststrings.c: txt2c.c and txt2c.sh,
specified the resultant types as "unsigned char[]" while version.c
declared them as "const char[]".

txt2.c generates 0xXX hex codes for each individual character, thus
the "unsigned" definition.  With link-time optimization, some versions
of gcc notice the mismatch and emit a warning.

Change the declarations to match the definitions and cast to char[]
when they are used.

Change OpenSSL to use SHA-256 for cert comparison. (closes #3924)

Note the GnuTLS code compares the certs directly to check if they are
in the certfile.

Add shortcuts for IMAP and POP mailboxes in the file browser

Mailbox list may not be properly displayed in a standard 80-column
terminal window if the $folder variable contains a long URL.  In such
a case only left part of each entry name can be visible with the default
value of $folder_format.  What's worse, this visible part may not be
enough to distinguish between the entries.

Thus in this case mutt_pretty_mailbox() will be just as useful as for
local mailboxes.

Add SNI support for GnuTLS. (closes #3923)

Add SNI support for OpenSSL. (see #3923)

The original patch for this is by Phil Pennock at:
https://people.spodhuis.org/phil.pennock/software/mutt-patches/

I have removed the OpenSSL version check and defined(OPENSSL_NO_TLSEXT)
check because:
  * SSL_set_tlsext_host_name() was added in 0.9.8f [11 Oct 2007]
  * OpenSSL 1.1 no longer has the OPENSSL_NO_TLSEXT compilation option
  * https://rt.openssl.org/Ticket/Display.html?id=2788&user=guest&pass=guest
    shows that the no-tlsext compilation option has been broken for some time.
  * Going forward, I'd like to minimize and start removing cruft required
    to support ancient/insecure versions of libraries.

build: drop two more obsolete m4 scripts (#477)