Add `<{edit,edit-or-view,view}-raw-message>` functions

Add mutt-functions `<edit-raw-message>`, `<edit-or-view-raw-message>`
and `<view-raw-message>`.

`<view-raw-message>` is a binding for the C-function
`mutt_view_message()`, which opens the raw message as a read-only file
in an external editor.

`<edit-or-view-raw-message>` is a dispatcher function: If the mailbox
is writable then this is the same as `<edit-raw-message>`, otherwise it
is the same as `<view-raw-message>`.

`<edit-raw-message>` is a synonym for `<edit>` whose name coherent with
the naming scheme of the other two functions introduced.

Change default key binding of `e` from `edit` to
`edit-or-view-raw-message`. This allows viewing the raw message in an
external editor also in read-only mailboxes.

editmsg.c: Implement mutt_view_message() function

Add mutt_view_message() function, which opens the selected message or
the tagged messages as a read-only file in an external editor. This
function is similar to mutt_edit_message() but does not allow modifying
the message.

A usage scenario might be to view the message source code in an editor.

Note: This patch only adds C functions. It does not add any mutt-functions.

doc: Document new mutt functions in manual

Document the new mutt functions `edit-or-view-raw-message` and
`view-raw-message`.

Amend description of `edit` function to mention the two new functions.

neomutt-syntax.vim: Add highlight rules for new functions

Add syntax rules for the new functions `edit-or-view-raw-message` and
`view-raw-message`.

Rename OP_EDIT_MESSAGE to OP_EDIT_RAW_MESSAGE

This brings the constant name in line with the description of its
corresponding mutt-function and paves the way for a coherent name scheme
in the upcoming 'edit-raw-message' family of mutt-functions.

merge: small coverity fixes

 * fix typos
 * fix leak in mutt_select_file()
 * fix leak in mutt_compose_attachment()
 * highlight preproc-like variable
 * add pointer checks to gpgme
 * check retval of REGCOMP()
 * check retval of strtok()
 * check retval of mutt_save_attachment()
 * check retval of mutt_account_get{user,pass,login}()
 * check retval of mutt_system()

check retval of mutt_system()

`mutt_system()` returns int, so check accordingly.

Also, make sure we log if any calls fail.

check retval of mutt_account_get{user,pass,login}()

Functions `mutt_account_getuser()`, `mutt_account_getpass()` and
`mutt_account_getlogin()` all return int, so check accordingly.

Also, make sure we log if any calls fail.

check retval of mutt_save_attachment()

check retval of strtok()

check retval of REGCOMP()

`REGCOMP()` wraps `regcomp()` which returns int.

add pointer checks to gpgme

highlight preproc-like variable

`allow_skip` is only set in one place in a `#ifdef`d block.
This effectively makes it a preprocessor condition.
By changing its name to capitals we let Coverity know how we're using
the variable.

fix leak in mutt_compose_attachment()

Move the `mutt_free_body()` before the two `goto bailout`s.

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.