Encode URIs created with nm_uri_from_query

Uses url_pct_encode to do the encoding. Note that url_pct_encode now
also encodes the ampersand, which shouldn't cause problems given that
it's only used for usernames and passwords otherwise.

Addresses issue #603

nm_description_to_path: fix a mailbox type

In nm_description_to_path a mailbox type is supposed to be MUTT_NOTMUCH,
since the function queries the path of a *notmuch* mailbox using
a description string.

Fixes #683.

Kill bundled wchar_t support, rely on the system's

Issue #690

Do not rely on backends setting error codes on success

From the API documentation of KyotoCabinet, TokyoCabinet and QDBM, I am
not convinced that the error code routines consistently return 0 whenever
the previous operation succeeded. Also, I am not convinced that they
return 1 whenever the previous operation failed.
It is important that we do not return 0 on failure, so let's try to get
the error code and fallback to -1 if we couldn't get anything more
specific.

Issue #696

Review hcache implementation return codes

Especially, in this batch I have reviewed the return codes for fetch and
delete operations. KyotoCabinet, QDBM and TokyoCabinet were indeed
incorrectly implemented, as these backends generally return true on
success and false on failure.

I still have to investigate how to get a proper return code from QDBM.

Issue #696

Doxify the Header cache

The initial framework was written by @flatcap. I have merely filled in a
few blanks.

Issue #696

build: fix make dist

merge: upstream fixes (mutt/default)

 * Fix menu color calls to occur before positioning the cursor. (see #3956)
 * Change imap_cmd_start() to return OK when the cmd_queue is finished. (closes #3956)
 * When guessing an attachment type, don't allow text/plain if there is a null character. (see #2933)
 * Add timeout parameter to mutt_socket_poll.
 * Add $imap_poll_timeout to allow mailbox polling to time out.
 * Convert raw_socket_poll() to use gettimeofday().

Convert raw_socket_poll() to use gettimeofday().

As long as gettimeofday() is supported, it's better to be consistent
within mutt and ensure greater portability.

Change the raw_socket_poll() wait timer to count milliseconds, like
the mutt_progess_update() code.

Thanks to Vincent Lefèvre for his, as always, invaluable advice.

Add $imap_poll_timeout to allow mailbox polling to time out.

Enable the polling flag for the NOOP in imap_check_mailbox(), the
STATUS command in imap_buffy_check(), and the LOGOUT command.

This is not intended to handle all blocking-IO related issues.
However, the periodic NOOP and STATUS are the most frequent places for
mutt to freeze up, especially after a laptop is sleep/woken.

Since these are quick operations with little data, this is a good place
to check if the connection is still working before hanging on a read.

Add timeout parameter to mutt_socket_poll.

This will be used in the next commit to add a timeout when polling for
new mail.

When guessing an attachment type, don't allow text/plain if there is a null character. (see #2933)

Type text/plain should not contain any null characters.  Slightly
improve the type guesser by forcing an attachment with any null
characters to be application/octet-stream.

Note the type guesser could use much more improvement, but this is an
easy and obvious fix.

Change imap_cmd_start() to return OK when the cmd_queue is finished. (closes #3956)

Some response handlers can end up recursively calling
imap_cmd_start(), processing all the command completions.  If the
outer call was an imap_exec(), this would result in the loop never
being terminated (or just blocking reading a server that has already
finished all the commands).

Change the callers that are simply using it to read a response,
without having called cmd_start(), to check for IMAP_CMD_OK instead.
Currently this is just the open connection function.

Fix menu color calls to occur before positioning the cursor. (see #3956)

It is possible for menu->color() to end up fetching an imap message,
and therefore generating a status message.

Because of this, we need to make those calls before we position the cursor.

merge: trivial code fixes

 * doxygen: fix comments
 * contrib: drop old iconv config
 * tidy: squeeze blank lines
 * tidy: remove duplicate prototypes
 * tidy: remove unnecessary casts
 * tidy: left-align goto labels
 * tidy: sort out typedef usage
 * tidy: rearrange code to avoid forward def
 * tidy: use DT_MASK to get type
 * tidy: sort out ++ -- comma operators
 * tidy: drop rarely-used macro
 * tidy: drop deprecated types
 * tidy: drop unnecessary INITVAL usage
 * rename: global boolean variables (insert underscores)
 * rename: global quad variables (insert underscores)
 * rename: tidy global variable naming
 * drop old testing code

drop old testing code

rename: tidy global variable naming

Rename the global variables to match the MixedCase coding style, e.g.
    Spoolfile -> SpoolFile

rename: global quad variables (insert underscores)

Rename constants for better legibility, such as:
    OPT_PGPENCRYPTSELF -> OPT_PGP_ENCRYPT_SELF

rename: global boolean variables (insert underscores)

Rename constants for better legibility, such as:
    OPTNOCURSES -> OPT_NO_CURSES

tidy: drop unnecessary INITVAL usage

Variables with a static/global scope are automatically initialised to 0/NULL.
About 10 variables were initialised to non-zero values, but they were
reset with values from init.h before they were used.

tidy: drop deprecated types

These have been unnecessary since 2012

tidy: drop rarely-used macro

tidy: sort out ++ -- comma operators

Where possible:
- Change pre- to post-decrement
- Change pre- to post-increment
- Drop commas (separate statements)

tidy: use DT_MASK to get type

tidy: rearrange code to avoid forward def

tidy: sort out typedef usage

tidy: left-align goto labels

tidy: remove unnecessary casts

tidy: remove duplicate prototypes

tidy: squeeze blank lines

contrib: drop old iconv config

doxygen: fix comments

merge: create library of shared code

 * split out state functions
 * split out parameter functions
 * create library
 * change code to use lib/lib.h
 * move the ascii functions to the library
 * move the base64 functions to the library
 * move the date functions to the library
 * move the exit functions to the library
 * move the md5 functions to the library
 * move the sha1 functions to the library
 * move the memory functions to the library
 * move the message functions to the library
 * move the debug functions to the library
 * move the string functions to the library
 * move the file functions to the library
 * move the buffer functions to the library
 * move the hash functions to the library
 * drop lib.c
 * doxygen: update config
 * rename the library to libmutt

rename the library to libmutt

- strip lib_ prefix from files
- rename liblib to libmutt

The only renaming complication was `lib_string.h`.
It's been renamed to `string2.h` so that `#include <string.h>` still works.

doxygen: update config

drop lib.c

move the hash functions to the library

move the buffer functions to the library

move the file functions to the library

move the string functions to the library

move the debug functions to the library

move the message functions to the library

move the memory functions to the library

move the sha1 functions to the library

move the md5 functions to the library

move the exit functions to the library

move the date functions to the library

move the base64 functions to the library

move the ascii functions to the library

change code to use lib/lib.h

create library

split out parameter functions

split out state functions

doxygen: tidy comments

@return x, y -> multiple @retval
@return -> @retval
more /* */ comments to /**< */
merge .h and .c function descriptions
Prefix defines with #

bind editor <delete> to delete-char

Look for ncurses headers in more dedicated locations first (#679)

This solves a problem on FreeBSD, where the base system is shipped with
ncurses 5.9 (/usr/include/ncurses.h). When building against 3rd party
ncurses 6.0 from ports (/usr/local/include/ncurses/ncurses.h), the
configure script picks up the 5.9 headers and the 6.0 library. This
might cause problems, since 6.0 is not ABI compatible with 5.0.

No need to strdup the result of tigetstr (#682)

doxygen: doc some more functions

Fix the printing of ncurses version in -v output

Fix the configure check for S-Lang (#676)

Issue: #667

docs: document some functions

merge: rename functions/constants for consistency

 * rename ESC_char
 * rename mutt_format_string and mutt_FormatString
 * rename SHA1_*
 * remove build constant SHA1HANDSOFF
 * #define constants in upper case

#define constants in upper case

remove build constant SHA1HANDSOFF

rename SHA1_*

rename mutt_format_string and mutt_FormatString

`mutt_format_string` and `mutt_FormatString` were much to similarly
named to be useful.  The first only does formatting.  The second uses
expandos to allow custom expansion.

To avoid future confusion, I've created new names that are distinct:
    mutt_format_string mutt_simple_format
    mutt_FormatString  mutt_expando_format

rename ESC_char

merge: improve doxygen comments

 * name all enum types
 * fix all comments
 * fix colours and links
 * add file templates
 * add struct/enum/union templates
 * convert existing comments

doxygen: convert existing comments

Lots of functions were already commented.
This commit re-formats the comments to be doxygen-compatible.

Despite the size of the commit, no code has changed.

doxygen: add struct/enum/union templates

doxygen: add file templates

doxygen: fix colours and links

doxygen: fix all comments

name all enum types

only match real mailboxes when looking for new mail

nntp: sync to upstream

NeoMutt 2017-07-14 (1.8.3)

merge: sync to mutt/default

update translations

Update German translation

clang-format

docs: update ChangeLog

version: 2017-07-14

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

drop unused flags argument from imap_access

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

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

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