prepare_sidebar

pgp_use_gpg_agent

pgpring_string_matches_hint

perform_or

perform_and

mutt_ts_capability

mutt_pop_list

mutt_needs_mailcap

mutt_match_spam_list

mutt_matches_list

mutt_limit_current_thread

mutt_group_match

mutt_comp_can_read

mutt_comp_can_append

mutt_addr_is_user

mh_valid_message

mbox_has_new

match_reference

maildir_add_to_context

link_threads

is_display_corrupting_utf8

is_day_name

is_context_available

imap_has_flag

hostname_match

get_initials

feature_enabled

crypt_is_numerical_keyid

create_hcache_dir

count_body_parts_check

compare_flags

cmd_queue_full

check_idn

check_for_mailing_list

check_certificate_expiration

check_certificate_cache

check_boundary

address_uses_unicode

addresses_use_unicode

addrcmp

add_quote_suffix

Fix memleak when attaching files.

merge: fix coverity defects

 * coverity: strfcpy
 * coverity: add variable - function arg could be NULL/invalid
 * coverity: add variable - failed function leads to invalid variable
 * coverity: add variable - Context could become NULL
 * coverity: add variable - alloc/strdup could return NULL
 * coverity: add variable - route through code leads to invalid variable
 * coverity: remove variable test
 * coverity: test functions
 * coverity: tidy switches
 * coverity: unused variables
 * coverity: refactor only
 * coverity: check for buffer underruns
 * coverity: fix leaks
 * coverity: minor fixes

coverity: minor fixes

Fix a few minor bugs.

coverity: fix leaks

Fix resource leaks.

coverity: check for buffer underruns

Prevent buffer underruns caused by functions that can return -1.

coverity: refactor only

Refactor code for clarity.

coverity: unused variables

Remove unused variables.

coverity: tidy switches

Simplify a few switch cases:
- Factor out trivial fall throughs
- Document any other fall throughs

coverity: test functions

Test the return value of some functions that could fail.

coverity: remove variable test

Remove a few simple tests where the condition is guaranteed to be true.

coverity: add variable - route through code leads to invalid variable

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.