Refactor: Remove unused return type

* mutt_tunnel_socket_setup()

fix: prevent timezone overflow

On 32-bit arches it's easy to over-/under-flow the date.
We replace these with the max/min values allowed in a time_t.

When we encounter one of these dates, pretend it's UTC to prevent
another overflow.

Fixes #819
Fixes #820

split up 'if' statements that assign and test (2)

Split 'if' statements that combine an assignment with a test.
e.g. from:

    if (!(adr = mutt_parse_adrlist(adr, buf)))

To:
    adr = mutt_parse_adrlist(adr, buf);
    if (!adr)

This makes the statements a little easier to read and debug.

split up 'if' statements that assign and test

Split 'if' statements that combine an assignment with a test.
e.g. from:

    if ((pc = imap_get_flags(&(idata->flags), pc)) == NULL)

To
    pc = imap_get_flags(&(idata->flags), pc);
    if (!pc)

This makes the statements a little easier to read and debug.

merge: minor code cleanups

 * move functions to library
 * lib: move MIN/MAX macros
 * simplify null checks
 * kill preproc expansion laziness
 * fix: variable type when using fread
 * reduce scope of variables

reduce scope of variables

C99 allows a 'for' loop to define its variable.
This reduces the variable's scope making its use clearer.

fix: variable type when using fread

kill preproc expansion laziness

Replace macros which were shortcuts for code, e.g.

```
  #define CUR_ENV Context->hdrs[i]->env
  function(CUR_ENV);
  CUR_ENV->from = "user";
  #undef CUR_ENV
```

simplify null checks

Simplify checks:
* `if (x == NULL)` -> `if (!x)`
* `if (x != NULL)` -> `if (x)`

lib: move MIN/MAX macros

Remove a duplicate definition of the MIN/MAX macros.

move functions to library

These functions have no NeoMutt dependencies:

* imap_make_date()
* imap_parse_date()
* imap_wordcasecmp()
* rstrnstr()

Moving the functions to the library will make it easier to test them and
find common behaviour.

NeoMutt 2017-10-13

update translations

clang-format

update mailmap

docs: update credits

update changelog

version: 2017-10-13

Deprecation: changed called function in mutt_sasl.c

* mutt_sasl_done()

bool: convert struct member in mutt_regex.h

* struct Regex
* restore_default()

Allow enabling of fmemopen (#842)

Issue #839

keyword: remove doc menu entry

We remove keyword dead code in 5c600997cd0e028a237fe6847ddffa1f2c340ea6.

This change removes the memu entry in the doc.

Closes #836

Use mutt_sleep(0)

Issue #777

Prefer a helpful error message over a BEEP

Issue #777

Do not fail if deflate is not in libz

Issue #828

Rebrand `.muttdebug` to `.neomutt`

The debug files should be called `.neomuttdegub` for consistency.

fix: crash using uncolor (#833)

Using 'uncolor' on the last 'color' added would cause a crash.

Sort the folders list when browsing an IMAP server (#831)

Issue #803

If set, use -D_FILE_OFFSET_BITS=64 when checking for GPGMe (#829)

* If set, use -D_FILE_OFFSET_BITS=64 when checking for GPGMe

Issue #826

* Let OFF_T_FMT be "%" PRId64 whenever LFS is there

Issue #826

Support EXTRA_CFLAGS and EXTRA_LDFLAGS, kill unused variable

EXTRA_CFLAGS and EXTRA_LDFLAGS can now be passed to make on the command
line. These are meant to be used for those flags that would otherwise
create problems at configure stage, like address sanitizers flags.

While at it, remove the setting of MUTT_VERSION, which is now unused.

docs: fix case of Notmuch

docs: sync to website

Install doc into `neomutt`

The documentation is installed into the `mutt` subdirectory by default.
This directory should be renamed to `neomutt`.

NeoMutt 2017-10-06

update translations

clang-format

update mailmap

docs: update credits

update changelog

version: 2017-10-06

TAILQify struct Connection

STAILQify struct ColorLine

fix: add more range-checking on dates/times

fix: overflow in mutt_mktime()

mutt_mktime() can overflow when used on 32-bit arches.
This was due to a bogus comparision.

Now, extreme future dates will be interpreted as 2038-01-19.

Fixes #794

keywords: Remove dead code/doc of keywords patch

Keywords patch support have been removed by 92e6dc1d89f7187eeea1412e51598600925e23f5.

This change removes associated config and docs.

Closes #769

travis: restore config

Autosetup-based configuration

Pull #745

merge: keywords

 * Make the notmuch tags generic
 * tags: Add index_format %J
 * Add IMAP keywords support
 * Share label editing between imap/notmuch
 * Tidying the code
 * tags: always pass correct buffer length
 * imap: fix memory leak
 * notmuch: remove unused methods declaration
 * tags: remove needs of HEADER_TAGS macro
 * tags: make private method private
 * tags: rename HeaderTag to TagList
 * tags: remove useless ifdef
 * tags: store transformed tags only if needed
 * imap: Don't use keywords terminology, but flags
 * tags: rename OP_MAIN_MODIFY_LABEL/TAGS
 * tags: rename modify-labels/modify-tags
 * tags: rename hdr_tags to driver_tags
 * tags: move tags_editor and tags_commit to mx
 * tags: don't depends on header.c
 * tags: Don't duplicate tags allocation
 * tags: fix notmuch editor return code
 * tags: rename DRIVER_LABEL to DRIVER_TAGS
 * always pass char to mutt_str_append_item

always pass char to mutt_str_append_item

tags: rename DRIVER_LABEL to DRIVER_TAGS

tags: fix notmuch editor return code

tags: Don't duplicate tags allocation

The HeaderTags and TagList structure can be replaced
by a simple linked list.

tags: don't depends on header.c

tags: move tags_editor and tags_commit to mx

This methods handles the run of dedicated methods on the mailbox, and
not the tags list manipulation.

It makes more sense to have it in mx.c like other methods with the same
purpose.

tags: rename hdr_tags to driver_tags

To avoid confusion about hdr_tags and tags in X-Label headers.

This renames all hdr_tags_* in drvier_tags_*.

tags: rename modify-labels/modify-tags

tags: rename OP_MAIN_MODIFY_LABEL/TAGS

imap: Don't use keywords terminology, but flags

tags: store transformed tags only if needed

tags: remove useless ifdef

OP_MAIN_MODIFY_LABELS and OP_MAIN_MODIFY_LABELS_THEN_HIDE no more
an notmuch thing. So make them always compiled.

tags: rename HeaderTag to TagList

tags: make private method private

tags: remove needs of HEADER_TAGS macro

notmuch: remove unused methods declaration

imap: fix memory leak

In imap_commit_message_tags() cmd is not freed in case of error.

tags: always pass correct buffer length

In notmuch editor we was passing sizeof(buf) with buf a char* with an
unknown allocated size.

In imap, we have assumed/hardcoded LONGSTRING. That's true, but we
should not do that.

edit_msg_tags() now takes the buflen as parameter so the root caller
can pass the size of the buffer.

Tidying the code

Share label editing between imap/notmuch

Add IMAP keywords support

This change introduces IMAP keyword support.

* index_format gets %g, %G, %J and %Gx to display them
* pattern gets g to search/match imap keywords
* colors.c gets index_tag/index_tags to color them
* OPS gets modify-labels to edit them
* tag-transforms/tag-formats/hidden_tags can be display them in more sexy manner

Most of the code is shared with the notmuch backend.

Upstream: https://dev.mutt.org/trac/ticket/3210
Closes neomutt/neomutt#225

tags: Add index_format %J

This change adds %J to index_format, to be able to show message tags
only when they are different from the previous one in a thread.

Make the notmuch tags generic

This change make the notmuch tags generic.

This is in preparation of reusing it for imap keywords feature.

merge: upstream fixes (mutt/default)

 * Fix uses of context->changed as a counter.
 * Make cmd_parse_fetch() more precise about setting reopen/check flags.
 * Enable $reply_self for group-reply, even with $metoo unset.

Enable $reply_self for group-reply, even with $metoo unset.

For a (r)eply, the $metoo variable does not have any effect because
there is a single recipient.

However with (g)roup reply, the To was initially set to the sender,
but subsequently removed my $nometoo.  The empty To line is then
replaced with the Cc line, resulting in a (logically correct but)
surprising outcome.

Change the behavior such that $nometoo will not strip a single To
recipient when $reply_self is set.

Make cmd_parse_fetch() more precise about setting reopen/check flags.

Previously any FETCH with FLAGS would result in either
  idata->reopen |= IMAP_EXPUNGE_PENDING;
  -or-
  idata->check_status = IMAP_FLAGS_PENDING;
being set.

This is unnecessary in the case of responses to FLAGS.SILENT updates
sent by mutt (which seem to commonly happen now-a-days).

Change imap_set_flags() to compare the old server flags against the
new ones, and report when _those_ updates would/did result in a local
header flag change.  Only set one of the reopen/check_status flags in
the event of an actual change (or potential change if a local change
has been made to the header.)

Fix uses of context->changed as a counter.

The first was in mx_update_tables(), but only when "not committing".
This is used by mh/maildir during an "occult" update, and in imap when
expunging the mailbox.  It meant to simply turn on changed when a
single changed header is seen.

The second use was in imap_sync_message_for_copy().  Previously this
was used for a server side copy/save, but is now also used for
fast-trash copying.  Remove the code that was trying to decrement the
counter: this function is not capable of properly setting a status
bit.

pgpewrap: free opts on exit

opts is allocated with malloc, but never free'd in both exit scenario's
leading to a memory leak of opts.

pgpewrap: remove duplicate if statement

The condition argc <= 1 is equal to argc < 2, remove the duplicate argc
<2 statement.

imap: Fix mx.mbox leak in imap_get_parent_path

valgrind reports a memory leak in imap_get_parent_path due to no
free'ing mx.mbox. As described in imap_parse_path, the callee should
free mx.mbox.

fix: crash using notmuch expando with imap

The NotMuch code didn't check for a NotMuch mailbox when dereferencing
Header->data.

Thanks to @xunam.

Fixes #805

bool: include missing stdbool.h header

bool: convert function parameters in mutt_notmuch.h

* nm_longrun_init()

Affected static functions in mutt_notmuch.c

* append_message()
* append_replies()
* append_thread()
* do_database_open()
* get_db()
* get_query()
* get_query_string()
* is_longrun()
* read_mesgs_query()
* read_threads_query()

Padded and sorted prototypes in init.h

spacing: Align function prototypes in init.h

fix: digest_collapse was broken (#801)

A recent upstream change stopped $digest_collapse from working.
The outer mime container wasn't being checked for type 'multipart/digest'.

A digest nested within another mime container should also be collapsed.

Fixes #789

Fix some problems with autoconf (#793)

* build: restore tabs in configure.ac

No functional changes

* build: fix out-of-tree build

* Built items in /doc weren't being installed.
* Force install to /usr/lib, not /usr/lib64

* build: stop yes/no include dirs

A bug in the curses detection often lead to things like:
* -Ino/include  -Lno/lib
* -Iyes/include -Lyes/lib

clang-format

STAILQify struct CryptModules

bool: convert function parameters in mutt.h

* mutt_matches_ignore()

bool: convert extern variable in mbyte.h

* Charset_is_utf8
* mutt_hcache_restore()
* mutt_set_charset()

Charset_is_utf8 affects `convert` in mutt_set_charset() and mutt_hcache_restore()
-> affects several static functions in hcache/hcache.c

* dump_address()
* dump_body()
* dump_buffer()
* dump_char()
* dump_char_size()
* dump_envelope()
* dump_parameter()
* dump_stailq()

* restore_address()
* restore_body()
* restore_buffer()
* restore_char()
* restore_envelope()
* restore_parameter()
* restore_stailq()

merge: rename mutt to neomutt

 * rename man pages
 * move helper programs to lib dir
 * move mime.types to doc dir
 * contrib: change mutt references to neomutt
 * build: don't install .dist files

build: don't install .dist files

contrib: change mutt references to neomutt

move mime.types to doc dir

Many distros have a package providing /etc/mime.types

We put our copy in:
    doc/neomutt/mime.types

move helper programs to lib dir

Programs:
    lib/neomutt/pgpewrap
    lib/neomutt/pgpring
    lib/neomutt/smime_keys

rename man pages

Man:
    man1/neomutt.1
    man1/pgpewrap_neomutt.1
    man1/pgpring_neomutt.1
    man1/smime_keys_neomutt.1
    man5/mbox_neomutt.5
    man5/mmdf_neomutt.5
    man5/neomuttrc.5

drop unused param in mutt_bcache_put

The 'tmp' param was always true.
Factor it out.

Thanks to @gahr for the idea/patch.

bool: convert function parameters in bcache.h

* mutt_bcache_put()

build: expand buffers to avoid gcc7 warnings

bool: convert function parameters in history.h

* mutt_history_add()
* mutt_history_at_scratch()

bool: convert function parameters in buffy.h

* buffy_check()
* buffy_maildir_check()
* buffy_mbox_check()
* mh_buffy()