Fix type mismatch in mutt_progress_init()

Fixes the following GCC warning:

    curs_lib.c: In function ‘mutt_progress_init’:
    curs_lib.c:471:65: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 4 has type ‘size_t {aka unsigned int}’ [-Wformat=]
           snprintf(progress->sizestr, sizeof(progress->sizestr), "%ld", progress->size);
                                                                     ^

Fix type mismatch in mutt_pretty_size()

Fixes the following GCC warnings:

    muttlib.c: In function ‘mutt_pretty_size’:
    config.h:55:19: warning: format ‘%lld’ expects argument of type ‘long long int’, but argument 4 has type ‘size_t {aka unsigned int}’ [-Wformat=]
     #define OFF_T_FMT "%" PRId64
                       ^
    muttlib.c:614:22: note: in expansion of macro ‘OFF_T_FMT’
         snprintf(s, len, OFF_T_FMT "K", (n + 51) / 1024);
                          ^~~~~~~~~
    config.h:55:19: warning: format ‘%lld’ expects argument of type ‘long long int’, but argument 4 has type ‘size_t {aka unsigned int}’ [-Wformat=]
     #define OFF_T_FMT "%" PRId64
                       ^
    muttlib.c:621:22: note: in expansion of macro ‘OFF_T_FMT’
         snprintf(s, len, OFF_T_FMT "M", (n + 52428) / 1048576);
                          ^~~~~~~~~

Add spaces after "References:" and "In-Reply-To:"

Context might be NULL - avoid dereferencing it

Issue #999

Review of "Bring consistency among sizes"

Issue #928

Bring consistency among sizes

There are instances in the code that refer to lengths or sizes of strings and
buffers and the like, but they are not typed in a consistent manner. The purpose
of this commit is to provide some consistency to these scenarios by changing
their type from unsigned int, int, unsigned long, etc. to size_t.

In general, these situations were identified by globally searching the code
for patterns along the lines of:
(unsigned|unsigned int|int|long) (len|blen|olen|length|l|size)
and changing the type to size_t on a case-by-case basis.

The following situations were explicitly ignored:

- autosetup/jimsh0.c, because it appears to be third-party code

- inputs to function calls relating to ssl and sasl, because they don't take
size_t as inputs and recasting in the function call didn't seem worth the
increased messiness

- for loop indices, because in some cases unsigned indices can breed bugs

This commit takes care of most obvious cases. There may certainly be subtler
ones that were missed in this pass.

Resolves: #928

fix variable names

Revert "Don't clear the macro buffer during startup"

Fixes #997

This reverts commit baf6ebbfc939278efb1895bed47e553ea7de19b1.

Fix SMTP RET parameter which was substituted by rc (#996)

Fixes #995

Configure number of directory components displayed in sidebar

In cases of a complex maildir directory structures, one may want to hide the
highest `N` directories from display in the sidebar to keep things looking
clean and legible.

This commit introduces a new config parameter, `sidebar_component_depth`, which
takes a positive integer as its value.

For example, suppose I have the following directory structure:
```
Inbox
dir1/dir2
dir1/dir2/dir3/sushi
dir1/dir2/dir3/neomutt/developer
```

By setting `sidebar_component_depth=2` in one's rc file, the sidebar will
display these maildirs in the following way:
```
Inbox
dir2
dir3/sushi
dir3/neomutt/developer
```

As seen in this example, the two highest directories have been truncated from
each path except in the case when there are `<= N` subdirectories, in which
case the bottom-most directory is displayed.

This parameter has a default value of 0, which disables the feature.

A previously-existing option, `sidebar_short_path`, provides similar behavior,
but truncates all except the bottom-most directories. When that option is
enabled, `sidebar_component_depth` is ignored.

Issue #813

Kill autotools (#994)

man: "mutt" --> "neomutt"

* man: "mutt" --> "neomutt"
* man: fix debug file and bug tracking system
* man: fix file reference
* Makefile uses lower-case files.
* man: update bugs section, update references

NeoMutt 2017-12-08

update translations

clang-format

update mailmap

update credits

update changelog

version 2017-12-08

Update autosetup to latest master

Update autosetup to commit b5a0e85d87a46d931ff1e4f8e5de8cd5307f93ba
- this makes it accept --enable-silent-rules and --disable-silent-rules
amongst other things, as requested in
https://github.com/msteveb/autosetup/issues/33

Install stuff to $(PACKAGE) in $(libexecdir), not $(libdir)

This does not really make a huge difference, probably, but it
is infinitely more correct.

Define prefix in Makefile.autosetup

Debian tools and probably other tools too pass arguments like
--mandir=\${prefix}/share/man
to configure. The prefix survives the output into the Makefile,
therefore, we need to define prefix in the Makefile too.

build: moved the check for idn before the check for notmuch

(original version from @gahr see [1])

[1] https://github.com/neomutt/neomutt/pull/771#issuecomment-342147768

Fix segfault when pipe'ing a deleted message

Fixes #796

Support reading FQDN from mailname files

Currently, we support reading from /etc/mailname and /etc/mail/mailname.
Adding support for more files is trivial. Reading these files is only
done if NeoMutt was configured without the --with-domain option.

Issue #974

restore folder settings after folder-hook

Issue #866 (4aaf1ef7) fixed a problem where the folder-hook wasn't being
called for the 'fcc' (sent mail folder).

If we run the folder-hook for the fcc folder, we must run the
folder-hook for the current folder afterwards.

Fixes #977

merge: minor code improvements

 * fix camelCase variable names
 * rename variables starting with a underscore
 * minor code fixes
 * return early where possible

return early where possible

minor code fixes

rename variables starting with a underscore

fix camelCase variable names

Local variable names should be lower case (with optional underscores).

add sleep after SMTP error

Fixes #982

merge: add charset functions to libmutt

 * move charset functions to libmutt
 * standardise the naming of the charset functions
 * add doxygen comments

add doxygen comments

standardise the naming of the charset functions

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