coverity: fix two minor defects

differentiate hcaches

fix hcachever.sh script

This script stopped working when all the structs were renamed
(and lots of the moved out of mutt.h)

compile-time output: use two lists

When asking NeoMutt for a list of compile-time features, the current
implementation assumes a mixture of options which can be enabled through
./configure, and some which default to on, without compile-time
exclusion.

Instead, separate this output in NeoMutt.

configure: remove MUTTLIBS

It's not clear to anyone why this variable existed in the first place,
so it was decided that it should be removed.

Most of the time, AC_CHECK_LIB and AC_SEARCH_LIBS will already alter
LIBS to prepend to it the found library, so there is no real reason to
duplicate this mechanism.

Some manual assignments have been kept because either the
action-if-found was set to no-op, or because new found libraries were
appended to MUTTLIBS, so this keeps the diff minimal with the same
"behaviour".

See issue #552

docs: fix pattern typo

Thanks to cybrNaut

merge: upstream fixes (mutt/default)

 * fix signed/unsigned comparison in longest_common_prefix
 * Fix crash when $postponed is on another server.
 * bcache: cast to avoid implicit signed/unsigned comparison in bcache_path
 * drop unused flags argument from imap_access

drop unused flags argument from imap_access

We are not using an actual interface so it is pointless.

bcache: cast to avoid implicit signed/unsigned comparison in bcache_path

Fix crash when $postponed is on another server.

imap_mxcmp() translates NULL to "INBOX".  When $postponed points to a
URL with an empty or "INBOX" path, this will end up matching against a
NULL idata->mailbox in imap_status().  This resulted in a crash
because idata->ctx is also NULL.

Thanks to Olaf Hering for the detailed bug report and suggested fix.

fix signed/unsigned comparison in longest_common_prefix

doxygen: add config file

doxygen: tidy existing comments

name unnamed enums

rename some structs/enums/unions/constants

struct/enum/union
    Capability_Aliases CapabilityAliases
    Capability_Alias   CapabilityAlias
    CryptKeyinfo       CryptKeyInfo
    div_type           DivType
    DnArrayS           DnArray
    FgetConvS          FgetConv
    format_flag        FormatFlag
    group_state_t      GroupState
    hash_key           HashKey
    hcache_ops_t       HcacheOps
    history_class_t    HistoryClass
    imap_auth_res_t    ImapAuthRes
    IMAP_COMMAND_TYPE  ImapCommandType
    key_cap_t          KeyCap
    mdb_txn_mode       MdbTxnMode
    NmCtxdata          NmCtxData
    NmHdrdata          NmHdrData
    NmHdrtag           NmHdrTag
    packet_tags        PacketTags
    pattern_exec_flag  PatternExecFlag
    pgp_ring_t         PgpRing
    pop_auth_res_t     PopAuthRes
    sb_src             SidebarSrc
    Sysexits           SysExits
    url_scheme         UrlScheme
    url_scheme_t       UrlScheme
    validate           Validate
    XDGType            XdgType

Constants
    kXDGConfigDirs     XDG_CONFIG_DIRS
    kXDGConfigHome     XDG_CONFIG_HOME
    ADDR               GS_ADDR
    NONE               GS_NONE
    RX                 GS_RX
    txn_read           TXN_READ
    txn_uninitialized  TXN_UNINITIALIZED
    txn_write          TXN_WRITE

Variables
    firstMessage       first_message
    hideQuoted         hide_quoted
    lastCached         last_cached
    lastLine           last_line
    lastLoaded         last_loaded
    lastMessage        last_message
    lineInfo           line_info
    maxLine            max_line
    newMailCount       new_mail_count
    QuoteList          quote_list
    SearchBack         search_back
    SearchCompiled     search_compiled
    searchDir          search_dir
    SearchFlag         search_flag
    SearchRE           search_re
    WrapMargin         wrap_margin

fix: stop allowing duplicate mailboxes

update ChangeLog.md

NeoMutt 2017-07-07 (1.8.3)

merge: sync to mutt/default

update translations

tidy whitespace

clang-format

update mailmap

docs: update credits

version: 2017-07-07

merge: translations

 * Update Chinese (Simplified) translation
 * Updated Danish translation.
 * Updated Japanese translation.

Updated Japanese translation.

Updated Danish translation.

Update Chinese (Simplified) translation

merge: upstream fixes (mutt/default)

 * Add more description for the %S and %Z $index_format characters.
 * Add L10N comments to the GNUTLS certificate prompt.
 * Add ~<() and ~>() immediate parent/children patterns. (closes #3144)

Add ~<() and ~>() immediate parent/children patterns. (closes #3144)

Thanks to Jeremie Le Hen for the original patch, and for his
persistence in getting this feature merged.

Add L10N comments to the GNUTLS certificate prompt.

The prompts are shared with OpenSSL, and so the prompts end up being
separate from the action key strings.

Add more description for the %S and %Z $index_format characters.

flags: update the hdr message last

this allow the color of the message of a colapsed thread to reflect
changes that occured in the subthread, and commited by this depth
first search.

Revert "don't close the pager after a timeout"

This reverts commit 5acd76ee270428302ec5e623a0cf743c17c4c05b.

Support Gmail's X-GM-RAW server-side search

Let Gmail search filters be used from mutt with =/

Closes #644

fix gpgme S/MIME non-detached signature handling

- gpgsm ends the session after the decrypt error, so restart gpgsm when
  retrying with "maybe_signed".
- evolution sends encrypted + signed S/MIME mails with non-detached
  signatures

merge: minor build improvements

 * move smime_keys into contrib
 * drop instdoc script
 * remove unnecessary steps
 * tidy makefile

Add ~<() and ~>() immediate parent/children patterns. (closes #3144)

Thanks to Jeremie Le Hen for the original patch, and for his
persistence in getting this feature merged.

build: tidy makefile

This commit makes no changes to the build.
It simply tidies up the whitespace and some lists.

build: remove unnecessary steps

Three man pages were processed with sed, but it didn't alter the files.

build: drop instdoc script

This script performed some string substitutions in the files it was
installing.  This was already done by the Makefile.

build: move smime_keys into contrib

menu: fix the thread tree color

The background tree color is really meaningless as it mixes with the
background color of indexes subject in unpleasant ways. We prefer to
create a new color based on the tree foreground color only.

Remove stale certificate prompt translations.

With the OpenSSL changes, unmaintained translation prompts will not function properly.

Clear out the translation strings for those using different action keys.

This changeset does not touch the translation files that are actively
maintained.  They will be properly updated before the 1.9.0 release.

Update po files.

Add L10N comments to the GNUTLS certificate prompt.

The prompts are shared with OpenSSL, and so the prompts end up being
separate from the action key strings.

Add more description for the %S and %Z $index_format characters.

browser tweak

See issue #609

Uses CurrentFolder to populate LastDir with IMAP

 * Needs to rollback to !folder instead of !*LastDir
 * Closes issue #640

fix perms

don't close the pager after a timeout

Setting a timeout hook could cause the pager to automatically close
itself when viewing help or an attachment.

    timeout-hook 'exec sync mailbox'

The timeout was causing a fake keypress event which cause the pager to
quit.

Fixes #591

colour emails with a '+' in them

The default highlighting for email addresses didn't recognise '+'
characters as being valid.

Fixes #496

sidebar: stabilise sort order

When sorting by a numeric type, fallback to the path if the numbers are
the same.

Fixes #486

doc: limitations of new-mail %f expando

docs: escape <>'s in nested conditions

Add config vars for forwarded message attribution intro/trailer.

Add $forward_attribution_intro and $forward_attribution_trailer to
allow the default strings to be overridden.

Fixes #522

dupe-string, don't destroy original

fix the padding expando '%>'

Debian BTS 866366

The non-greedy '%>' has been broken since the introduction of the
nested-if patch.

The second command should create a line of dots ending in a ']'

    set index_format="%?M?collapsed%|.]&expanded?
    set index_format="%?M?collapsed%>.]&expanded?

merge: add some colour schemes

 * added neonwolf colorscheme
 * added black 256 solarized colorscheme
 * added zenburn colorscheme
 * added vombatidae colorscheme
 * removed note to use slang for neonwolf colorscheme
 * build: add colour schemes

build: add colour schemes

Closes #340

removed note to use slang for neonwolf colorscheme

Thanks to @h3xx for the hint.

added vombatidae colorscheme

Thanks to @octol

added zenburn colorscheme

Thanks to Renato Cunha

Source:
https://bitbucket.org/trovao/configs/src/c3413bafa5ba70da2329225a9009078b078ccee1/mutt-zenburn/dotmuttrc

added black 256 solarized colorscheme

Thanks to @altercation

added neonwolf colorscheme

Thanks to @h3xx for publishing it.

merge: build fixes for Solaris

 * Handle native Solaris GSSAPI.
 * Fix compilation error on Solaris.
 * Include term.h for tigetstr except for slang build.
 * Look for wide curses in libcurses itself.
 * Remove unneeded forward declarations.

Add config vars for forwarded message attribution intro/trailer.

Add $forward_attribution_intro and $forward_attribution_trailer to
allow the default strings to be overridden.

Remove unneeded forward declarations.

Look for wide curses in libcurses itself.

NetBSD's curses library provides wide character support.

Include term.h for tigetstr except for slang build.

Fixes
curs_main.c:529:11: warning: implicit declaration of function ‘tigetstr’ [-Wimplicit-function-declaration]

Including it before header.h causes compilation errors because
of "int lines;" in header.h and
in term.h.

Fix compilation error on Solaris.

  resize.c: In function 'mutt_resize_screen':
  resize.c:38:18: error: storage size of 'w' isn't known
     struct winsize w;
                    ^
  resize.c:47:19: error: 'TIOCGWINSZ' undeclared (first use in this function)
       if (ioctl(fd, TIOCGWINSZ, &w) != -1)
                     ^

Handle native Solaris GSSAPI.

Signed-off-by: Thomas Klausner <wiz@NetBSD.org>

Do not set old flag if mark_old is false

If `mark_old` is false, unread mails will not be marked old after
they've been viewed. But they will be marked old if found in the `/cur`
subdirectory of the Maildir. This is inconsistent behaviour, since a
user with disabled `mark_old` will expect no old flags at all.

merge: fix some leaks

 * fix some resource leaks
 * fix leaks in lua get/set options
 * fix dodgy buffers

fix dodgy buffers

fix leaks in lua get/set options

fix some resource leaks

- CID 169545
- CID 169546
- CID 171381
- CID 171382

Block SIGWINCH during connect(). (closes #3941)

FreeBSD's connect() does not respect SA_RESTART, so a SIGWINCH will
end up interrupting the connect.

If this code is needed in other places, it should be moved into
signal.c.  For this one place, inlining the sigprocmask() seemed okay.

fix whitespace errors

tidy the version strings

sync to upstream's "encrypt-to-self"

NeoMutt's variables:
    pgp_encrypt_self
    smime_encrypt_self

have been superceded by Mutt's:
    pgp_self_encrypt
    pgp_self_encrypt_as
    smime_self_encrypt
    smime_self_encrypt_as

Note: NeoMutt's variables will be deprecated at a later date.

More sophisticated check for BDB version + support for DB6 (non default)

This commit enhances configure.ac to actually parse (as in CPP) and
extract the values of DB_VERSION_MAJOR and DB_VERSION_MINOR from the
Berkeley DB include files. The previous approach to just use grep failed
on some system where the includes are just wrappers around the real bdb
headers.

Also, add support for the current db6 version (6.2), although db5 is
still the first one looked for.

Issue #613

fix: initialise debug_file

Make sure that `debug_file` is initialised early enough.
Also, call `mutt_expand_path()` to enable ~/ shortcuts.

Related to issue #654

contrib: add Mutt logos

Issue #655

Fix typo in smime_self_encrypt_as documentation. (closes #3953)

merge stable

Minor documentation fixes.

The muttrc man page doesn't contain the "Format Strings" section in
the manual, so add a pointer to the explanation of conditionals in
$status_format.

revert part of 40f81d1 Add debug_level/debug_file options

This patch moves the set_default/restore_default couple after the defaults have
been set by the init function; this was correctly placed in 1.7.2 and somehow it
got moved to the wrong place, which means that tmpdir (and many other variable)
are not correctly set with their new default value determined from environment
variables (or other logic), so when they get resetted with a 'reset all' call in
.muttrc, they will never have the correct default to fall back to and they will
use the wrong one in init.h

This patch fixes:
- http://bugs.debian.org/865822
- http://bugs.debian.org/865842

Issue #654

score address pattern do not match personal name

Fixes #646
Debian BTS 383769

open attachments in read-only mode

Fixes #647
Debian BTS 528233
Mutt trac #3261

Add Cc, In-Reply-To, and References to default mailto_allow

These headers are quite useful for replying to mailing lists
via mailto links, as they don't break threading.

Debian has been shipping these in the /etc/Muttrc file since
September 2016 without much issues.

Fixes #115

merge: trivial code changes

 * build: don't delete non-existent files
 * remove unused option
 * docs: fix typos
 * update mailmap

update mailmap

docs: fix typos

Fixes #648

remove unused option

build: don't delete non-existent files

merge: upstream fixes (mutt/default)

 * Change the compose menu fields to be dynamically padded.
 * Rename 'sign as' to 'Sign as'; makes compose menu more consistent.
 * Make GPGME key selection behavior the same as classic-PGP. (see #3950)
 * Force hard redraw after $sendmail instead of calling mutt_endwin. (closes #3952) (see #3948)
 * Enable all header fields in the compose menu to be translated.
 * Auto-pad translation for the GPGME key selection "verify key" headers.
 * Improve the L10N comment about Sign as.

Improve the L10N comment about Sign as.

Make it clearer what the indentation behavior is, and add suggestions
about the width.

Thanks to TAKAHASHI Tamotsu for the original patch suggesting changes
to the L10N comments.

Auto-pad translation for the GPGME key selection "verify key" headers.

Remove the ridiculous need for the translators to pad the translation
strings themselves.