fix leak in mutt_select_file()

fix typos

Make sure objects are not compiled before git_ver.h is generated

autosetup: expand --everything in `neomutt -v`

doxygen: document parameter functions

doxygen: document envelope functions

doxygen: document address functions

docs: add a sentence to quasi-delete docs

Cleanup "SSL is unavailable" error in mutt_conn_find

- Remove the sleep
- Be more informative about the failed connection

Issue #934

Sanitize mutt_copy_message* APIs

rename public functions with a leading underscore

rename static functions with a leading underscore

Correct spelling mistakes.

doc: display_filter escape sequence

Issue #897

fix: drop capability aliases

`CapabilityAliases` was a list of alternative names (just one) for IMAP
capability strings.  This isn't necessary.  We can put the strings in
`Capabilities` and fix the enumeration.

```c
enum ImapCaps
{
  ...
  X_GM_EXT1,
  X_GM_ALT1 = X_GM_EXT1,
  ...
};
```

bug: imap gmail search

The order of strings in Capabilities should match the enums in ImapCaps.

Do not turn CRLF into LF when dealing with transfer-encoding=base64 (#920)

Fixes #722

Check that OpenSSL md5 supports -r before using it (#915)

* Check that OpenSSL md5 supports -r before using it

Fixes #914

* Remove unused variable assignment

Fixes #914

merge: trivial code fixes/changes

 * move test to the right place
 * Remove unnecessary semicolon
 * build: fix shadow variable warning
 * refactor malloc/calloc
 * ignore temporary build files
 * fix copyright messages
 * fix plain unsigned
 * unused prototypes, whitespace
 * Standardise spelling s/RfC/RFC/
 * alias drop self

alias drop self

struct Alias began:

```c
struct Alias
{
  struct Alias *self; /* XXX - ugly hack */
  ...
```

There didn't seem to be any reason for this pointer.
The two places that allocate a new Alias, immediately set the self
pointer to the object.  No users of `self` test it first.

Standardise spelling s/RfC/RFC/

unused prototypes, whitespace

fix plain unsigned

Be specific.  Replace `unsigned` with `unsigned int`.

fix copyright messages

ignore temporary build files

refactor malloc/calloc

Replace a malloc() and repeated memset()s with a single calloc().

build: fix shadow variable warning

Remove unnecessary semicolon

move test to the right place

Respect --with-ssl path (#917)

Fixes #916

autosetup: fix check for missing sendmail

If sendmail wasn't on the path, then configure.autosetup defined
    SENDMAIL=false

Define NCURSES_WIDECHAR to require wide-char support from ncurses (#908)

See https://lists.gnu.org/archive/html/qemu-devel/2017-06/msg00634.html
for a discussion. This might be a solution for #589.

 Fix wrong extern declaration of globals in conn/conn_globals.h (#919)

* Revert "Move definition of conn/globals.h back into globals.h (#912)"

This reverts commit fe03e2b126ca899f6fdd8276125bef3c66b6aef4.

* Fix wrong extern definition of globals in conn/conn_globals.h

Issue #918

Move definition of conn/globals.h back into globals.h (#912)

Fixed #907

Make sure Url is initialized even when u->scheme == U_UNKNOWN (#911)

Fixed #910

Enhance ifdef feature to support my_ vars (#906)

* Enhance ifdef feature to support my_ vars

Issue #905

* Use myvar_get, do not reinvent the wheel

Issue #905

Fix build, apply clang-format

fix build for RHEL6

RHEL6 doesn't have the function sasl_client_done().

Add comment blocks to ~60 functions

build: Create a Connection Library (libconn)

There are no changes to the code, just rearrangements of functions.

merge: Document 70 IMAP functions

There are no code changes in this set.

 * rearrange functions
 * rename static functions
 * fix some types
 * comment blocks

comment blocks

fix some types

rename static functions

rearrange functions

No code changes

Static functions first
General public functions
Mailbox functions

NeoMutt 2017-10-27

update translations

clang-format

update mailmap

docs: update credits

update changelog

version 2017-10-27

Wrap dirname(3) inside a mutt_dirname() function

* Wrap dirname(3) inside a mutt_dirname() function

This new function, opposite to the original dirname(3), does not modify
its argument and can be passed a const char * argument. This means that
callers mustn't copy their strings before calling dirname anymore.

autosetup: check for sys_siglist[]

url: make notmuch query string parser generic (#812)

* Notmuch: remove custom UriTag linked list

Use STAILQ helper for UriTag linked list.

* url: make notmuch query string parser generic

Enhance the url.c to handle query strings.

This permits to remove the notmuch custom url parser.

* url: always parse query string

This removes url_parse_with_qs methods.

* fix some problems

- double free in nm_normalize_uri()
- set path to NULL in url_parse()
  If we're setting all the other members, we should set the path too
- tidy some doxygen docs

* notmuch: remove useless starting / from db_filename

build: --enable-everything includes lua (#868)

Fixes #855

Do not use CPPFLAGS / CFLAGS together with CC_FOR_BUILD (#882)

* Do not use CPPFLAGS / CFLAGS together with CC_FOR_BUILD
* Restore -std=c99 when building makedoc
* Make sure host-independent CFLAGS are passed to CC_FOR_BUILD
* Let CC_FOR_BUILD carry CFLAGS_FOR_BUILD, there's no reason to split them

Fix smart wrap in pager without breaking header

This commit addresses the issue of smart wrap not working if a line
started with whitespace.

This behavior was a result commit f5e472b where smart wrap was disabled
on lines beginning with a space or tab to prevent headers from having
weird wrapping.

This commit addresses this issue by only applying smart wrapping if the
pager is displaying a message body. This allows smart_wrap to work
inside of a message and preserves what commit f5e472b addressed.

Additionally, the buf[cnt] checks have been simplified with the use of
the ISSPACE() macro.

While fixing the smart wrap, it became apparent that the HEADER portion
of the pager was not properly updating when the wrap size was modified.
To address this, the wrap command was changed from R_PAGER to
R_PAGER_FLOW in init.h. This redraws the pager after wrap is modified.

call the folder-hook before saving to $record

fix actions on tagged messages (#823)

Many actions aren't performed on all tagged messages if the messages are
within a collapsed thread.

lib: move List and Queue into library

Update autosetup to latest master branch

This includes the fix for a more fine-grained interaction between
$prefix, /etc, and /var.
See https://github.com/msteveb/autosetup/issues/31.

Issue 888: Fix imap mailbox flag logging

remove the blank line in the keywords logging, per @flatcap

use Buffer and related functions for mailbox flag print

Review comments from @flatcap on IRC

free() -> FREE()

revert free() -> FREE()

Correct use of FREE() per @flatcap

tabs->spaces, per @flatcap

clang-format

One log line for flags, per @gahr

Fixes for endianness detection (#893)

* Move endian check after system extensions

GNU/Linux does not seem to define BYTE_ORDER in vanilla c99 mode.

Issue #883

* Make sure WORDS_BIGENDIAN is defined if big-endian

Issue #883

* Fix typo: s/Endianess/Endianness/

Issue #883

autosetup: delete makedoc on 'make clean' (#891)

Now that makedoc is always built, alter the 'make clean' target to
delete it.

docs: compressed and nntp features are now always built

initialise variables

These variables were passed uninitialised to a function which increments
them.

Discovered by Coverity after the 'if-assign' commits.

refactor out O_NOFOLLOW

O_NOFOLLOW is part of POSIX:2008 which isn't required for NeoMutt.
This refactoring streamlines the code.

use mutt_array_size()

minor code fixes

* Fix typos
* Fix function prototype
* Remove spurious semicolons
* Remove duplicate #include

Update autosetup to latest master branch

doxygen: fix a few warnings

Move mutt_rename_file to lib/file.[hc]

Use host compiler to build the documentation

Issue #870

Use bundled wcscasecmp if an implementation is not found in libc

Issue #879

Use safe_calloc to initialize memory, simplify size_t overflow check

add extra checks to mutt_pattern_exec()

bool: convert function parameters in nntp.h

* nntp_active_fetch()
* nntp_select_server()

Caller affected
* main()
* mutt_compose_menu()
* mutt_index_menu()
* _mutt_select_file()
* nntp_open_mailbox()
* nntp_post()
* nntp_select_server()

notmuch: virtual-mailboxes should accept a limit

* Fix issue #849

nm_uri_from_query() checks if there is a custom limit and includes it
in the URI. This keeps the limit intact after nm_normalize_uri()
terminates. Otherwise, the limit is stored within nm_normalize_uri()'s
temporary NmCtxData struct and lost upon termination.

Additionally, nm_normalize_uri() properly initializes tmp_ctxdata, which
sets the default db_limit value. This ensures db_limit is either
NmDbLimit or the custom limit. My earlier attempt to resolve #849 did
not initialize db_limit and suffered from random db_limit values.

The url_parse_query() check was removed from nm_normalize_uri() as
new_ctxdata() contains the same block of code. Error message numbering
has been updated the reflect the change.

* Remove redundant variable

Refactor per suggestions in code review to remove redundant variable and
update method signature.

The removal of the redundant variable prompt the need to change
tmp_ctxdata from NmCtxData to *NmCtxData. Calls to its fields have been
updated to reflect its new status.

* wipe the context before use

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

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

    if ((rc = mutt_socket_readln(buf, bufsize, conn)) < 0)

To:

    rc = mutt_socket_readln(buf, bufsize, conn);
    if (rc < 0)

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

English and other cleanups

imap/notmuch tags: Add some documentation

Let autosetup define PWD, do not unnecessarily try to create hcache dir (#847)

Issue #845

tags: Show fake header for all backends

Closes #864

Generate neomuttrc even if configured with --disable-doc

Issue #862

Add polling for the IDLE command. (closes #3957)

Add $imap_poll_timeout poll for IDLE, since this is also a command
that will freeze after waking if $imap_idle is set.

Allow to specify --docdir at configure time

Issue #857

Fix neomutt.pot location, remove from git

Issue #853

Fix update-po

Issue #853

Bool: change functions in mx.h

* maildir_parse_message()
* maildir_parse_stream()
* mx_tags_is_supported()
* mx_update_tables()

Caller affected
* append_message()
* append_message()
* maildir_update_tables()
* mx_sync_mailbox()
* mx_update_tables()

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.