rename lib to mutt

merge: upstream fixes (mutt/default)

 * Note which ssl config vars are GnuTLS or OpenSSL only
 * Add message count to $move quadoption prompt
 * Add %R (number of read messages) for $status_format
 * Updated French translation
 * Add $change_folder_next option to control mailbox suggestion order
 * Fix $smart_wrap to not be disabled by whitespace-prefixed lines
 * Remove useless else branch in the $smart_wrap code

Remove useless else branch in the $smart_wrap code. (see #3857)

Thanks to Vincent Lefèvre for noticing the nested else was redundant,
since buf_ptr is already set to "buf + cnt" after the format_line()
call.

This allows us to merge the inner and outer if statement, resulting in
simpler code.

Fix $smart_wrap to not be disabled by whitespace-prefixed lines. (closes #3857)

changeset:737102af74eb fixed a folded header display issue with $smart_wrap
by disabling $smart_wrap for lines beginning with whitespace.

Unfortunately, this turns off smart wrapping in the body of an email
too, even when the line has other whitespace breaks in it.

An earlier commit, changeset:125076e0fdfa added an infinite loop fix
when MUTT_PAGER_NSKIP is set, by disabling the smart_wrap if the space
backtracking went to the beginning of the line.  That is, a line
beginning with 1+ whitespace followed by a single long word.

Extend this second commit by always disabling the smart_wrap in that
case, not just when MUTT_PAGER_NSKIP is set.  This also solves the
folded header issue without other side effects.

Add $change_folder_next option to control mailbox suggestion order.

This patch is out of the Gentoo package maintained by Fabian Groffen.
From the link below, it appears the original author was Simon Burge.

I made some minor changes to get it to apply correctly, and cleaned up
the documentation a bit.

Original patch description:
This patch brings back the original behaviour of change-folder, which
some people find more useful.  It suggests the next folder in your
mailboxes list that has new mail measured from the current folder,
instead of the first folder from your mailboxes list in any case.  This
prevents starvation of folders defined at the back of your list ;)
See discussion: http://thread.gmane.org/gmane.mail.mutt.devel/12457

Updated French translation.

Add %R (number of read messages) for $status_format.

Add message count to $move quadoption prompt.

Display the number of messages that will be moved in the quadoption
prompt, which might prove helpful for some people.

Thank you to Daan van Rossum for suggesting the improvement.

Note which ssl config vars are GnuTLS or OpenSSL only. (closes #3781)

It was not obvious that the ssl_ca_certificates_file was only for
GnuTLS, or that the ssl_usesystemcerts was only for OpenSSL.

merge: library moves/improvements

 * build: rename headers.c to header.c
 * build: move *_free functions
 * doxygen: add custom layout
 * lib: tidy headers
 * lib: move NLS magic to message.c
 * lib: move function to file.c
 * lib: move functions to string.c
 * lib: move function to list.c
 * lib: move functions to mbyte.c

merge: add <edit-or-view-raw-message>

 * Rename OP_EDIT_MESSAGE to OP_EDIT_RAW_MESSAGE
 * neomutt-syntax.vim: Add highlight rules for new functions
 * doc: Document new mutt functions in manual
 * editmsg.c: Implement mutt_view_message() function
 * Add `<{edit,edit-or-view,view}-raw-message>` functions

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

lib: move functions to mbyte.c

lib: move function to list.c

lib: move functions to string.c

lib: move function to file.c

lib: move NLS magic to message.c

lib: tidy headers

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()