move charset functions to libmutt

github: make removal of irrelevant sections easier

When users remove the irrelevant things for their issue, they often
leave the indentation of the nested list. That causes github to render
those lines as code instead of markdown list items.

Fixup smart modify-labels-then-hide for !tag case (#975)

The code used to automatically handle setting the quasi delete
state to whether the message is still visible after performing
the change.

This was broken for the !tag case in

commit 7d151fd0919f6cc7ba9cdc1e7de90697c66dce43
Author: Mehdi Abaakouk <sileht@sileht.net>
Date:   Tue Feb 21 10:50:35 2017 +0100

    Share label editing between imap/notmuch

Which replaced the !still_visible with simply true.

modify gpg.rc to accommodate GPG 2.1 changes

remove debug/logging from makefile/configure

drop #ifdef DEBUG

Don't clear the macro buffer during startup

* Move the folder hook after the folder has been opened
* Prevent a SIGWINCH during startup from clearing the macro buffer

Fixes #227

build: restore externs

The MacOS build failed, with linker errors for libconn's global
variables.

By initialising them, we hope to force the linker to use them.

Fix reference to neomuttrc

merge: refactor and document email functions

 * move functions to mutt_address.c
 * rename rfc822.c to address.c
 * move rfc822_new_address to address.c
 * standardise the address functions
 * standardise the envelope functions
 * standardise the parameter functions
 * move address functions
 * move envelope function
 * move parameter functions
 * rename mutt_tags to tags
 * add comments
 * rename rfc822 constants
 * minor tidying/boolification
 * move statics first

move statics first

minor tidying/boolification

rename rfc822 constants

add comments

rename mutt_tags to tags

move parameter functions

move envelope function

move address functions

standardise the parameter functions

standardise the envelope functions

standardise the address functions

move rfc822_new_address to address.c

rename rfc822.c to address.c

move functions to mutt_address.c

Don't do I/O in a signal handler

Remove unused parameters

* curs_main.c main_change_folder(flags)
* imap/imap.c imap_buffy_check(force)
* init.c mutt_nm_tag_complete(pos)
* ncrypt/crypt_gpgme.c verify_sender(protocol)
* recvcmd.c attach_forward_bodies(flags)
* mutt/file.c mutt_file_lock(path)
* mutt/file.c mutt_file_unlock(path)
* recvcmd.c mutt_attach_bounce(hdr)
* recvcmd.c mutt_attach_resend(hdr)
* recvcmd.c attach_forward_msgs(hdr)
* recvcmd.c attach_include_reply(flags)
* send.c mutt_make_misc_reply_headers(hdr,ctx)
* sendlib.c encode_8bit(istext)

Add the missing debug macro

The vital macro got lost from the earlier commit.

Fix stdout + stderr redirection in hcachever.sh

merge: more additions to libmutt

 * move signal functions to libmutt
 * standardise the naming of the signal functions
 * move exit syslist to libmutt
 * factor out ICONV_CONST from headers

factor out ICONV_CONST from headers

move exit syslist to libmutt

standardise the naming of the signal functions

move signal functions to libmutt

debug: Strip function names from debug messages

debug: replace mutt_debug with a macro

The macro passes the file and line number to mutt_debug_real().

merge: standardise the string formatting functions

There are lots of callback functions used by `mutt_expando_format()`.
These commits make sure they all have similar names and have similarly
named parameters and variables.

 * rename formatting functions
 * rename format parameters/variables
 * update format function comments
 * rearrange switch statements in format functions
 * light tidy of format functions

light tidy of format functions

rearrange switch statements in format functions

update format function comments

rename format parameters/variables

rename formatting functions

merge: rename some variables

The majority of the code uses the variable name 'buf' for a char buffer.
Most return-code variables are called 'rc'.

This mass rename makes the code more consistent.

 * buff -> buf
 * ret -> rc
 * rv -> rc

rename variables: rv -> rc

rename variables: ret -> rc

rename variables: buff -> buf

merge: trivial code improvements

 * drop unused code in #if 0
 * doxygen: fix comments
 * fix shadow variable
 * drop duplicate prototype
 * remove unnecessary alloc checks
 * add logging around getaddrinfo()

add logging around getaddrinfo()

If the user's networking isn't set up correctly, this call can
potentially take several seconds to timeout.

remove unnecessary alloc checks

drop duplicate prototype

fix shadow variable

doxygen: fix comments

drop unused code in #if 0

merge: code style improvements

 * standardise infinite loops
 * fix justify constants
 * insert braces
 * split ifs
 * fallthrough

Fix ansi escape sequences with both reset and color parameters

The initial 0 "reset/normal" was setting attr = ANSI_OFF, which was
subsequently being or'ed with ANSI_COLOR.

The color resolution was then checking for the ANSI_OFF bit and
turning all coloring off, even though ANSI_COLOR was also set.

Change the color resolution to check for "attr == ANSI_OFF" instead, so
the color isn't reset unless the reset occurs last in the sequence.

(closes #3689)

fallthrough

tidy: clang-format

split ifs

insert braces

fix justify constants

standardise infinite loops

fix out-of-tree builds

Remove vim syntax file from the main repo

The syntax file is now in an extension at https://github.com/neomutt/neomutt.vim

Migrate code to use mutt_file_chmod{,_add,_rm} functions

Migrate the old code to use the new functions mutt_file_chmod(),
mutt_file_chmod_add(), mutt_file_chmod_rm() instead of plain chmod().

Add chmod() wrappers to library

Add functions for changing/adding/removing permissions of files.

mutt_file_chmod          - set permissions
mutt_file_chmod_add      - add permissions
mutt_file_chmod_add_stat - add permissions
mutt_file_chmod_rm       - remove permissions
mutt_file_chmod_rm_stat  - remove permissions

The *_stat functions accept an additional optional stat structure
parameter, which must point to the stat structure of the file.
Using the *_stat functions can save a call to stat() as the stat
structure does not need to be queried by the mutt_file_chmod_* function.

build: fix update-po target

merge: Standardise the names of the muttlib functions

 * rename lib to mutt
 * rename base64 functions
 * rename buffer functions
 * rename date functions
 * rename file functions
 * rename hash functions
 * rename list functions
 * rename mapping functions
 * rename mbyte functions
 * rename md5 functions
 * rename memory functions
 * rename sha1 functions
 * rename string functions

rename string functions

rename sha1 functions

rename memory functions

rename md5 functions

rename mbyte functions

rename mapping functions

rename list functions

rename hash functions

rename file functions

rename date functions

rename buffer functions

rename base64 functions

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.