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.

move functions to library

These functions have no NeoMutt dependencies:

* imap_make_date()
* imap_parse_date()
* imap_wordcasecmp()
* rstrnstr()

Moving the functions to the library will make it easier to test them and
find common behaviour.

NeoMutt 2017-10-13

update translations

clang-format

update mailmap

docs: update credits

update changelog

version: 2017-10-13

Deprecation: changed called function in mutt_sasl.c

* mutt_sasl_done()

bool: convert struct member in mutt_regex.h

* struct Regex
* restore_default()

Allow enabling of fmemopen (#842)

Issue #839

keyword: remove doc menu entry

We remove keyword dead code in 5c600997cd0e028a237fe6847ddffa1f2c340ea6.

This change removes the memu entry in the doc.

Closes #836

Use mutt_sleep(0)

Issue #777

Prefer a helpful error message over a BEEP

Issue #777

Do not fail if deflate is not in libz

Issue #828

Rebrand `.muttdebug` to `.neomutt`

The debug files should be called `.neomuttdegub` for consistency.

fix: crash using uncolor (#833)

Using 'uncolor' on the last 'color' added would cause a crash.

Sort the folders list when browsing an IMAP server (#831)

Issue #803

If set, use -D_FILE_OFFSET_BITS=64 when checking for GPGMe (#829)

* If set, use -D_FILE_OFFSET_BITS=64 when checking for GPGMe

Issue #826

* Let OFF_T_FMT be "%" PRId64 whenever LFS is there

Issue #826

Support EXTRA_CFLAGS and EXTRA_LDFLAGS, kill unused variable

EXTRA_CFLAGS and EXTRA_LDFLAGS can now be passed to make on the command
line. These are meant to be used for those flags that would otherwise
create problems at configure stage, like address sanitizers flags.

While at it, remove the setting of MUTT_VERSION, which is now unused.

docs: fix case of Notmuch

docs: sync to website

Install doc into `neomutt`

The documentation is installed into the `mutt` subdirectory by default.
This directory should be renamed to `neomutt`.

NeoMutt 2017-10-06

update translations

clang-format

update mailmap

docs: update credits

update changelog

version: 2017-10-06

TAILQify struct Connection

STAILQify struct ColorLine

fix: add more range-checking on dates/times

fix: overflow in mutt_mktime()

mutt_mktime() can overflow when used on 32-bit arches.
This was due to a bogus comparision.

Now, extreme future dates will be interpreted as 2038-01-19.

Fixes #794

keywords: Remove dead code/doc of keywords patch

Keywords patch support have been removed by 92e6dc1d89f7187eeea1412e51598600925e23f5.

This change removes associated config and docs.

Closes #769

travis: restore config

Autosetup-based configuration

Pull #745

merge: keywords

 * Make the notmuch tags generic
 * tags: Add index_format %J
 * Add IMAP keywords support
 * Share label editing between imap/notmuch
 * Tidying the code
 * tags: always pass correct buffer length
 * imap: fix memory leak
 * notmuch: remove unused methods declaration
 * tags: remove needs of HEADER_TAGS macro
 * tags: make private method private
 * tags: rename HeaderTag to TagList
 * tags: remove useless ifdef
 * tags: store transformed tags only if needed
 * imap: Don't use keywords terminology, but flags
 * tags: rename OP_MAIN_MODIFY_LABEL/TAGS
 * tags: rename modify-labels/modify-tags
 * tags: rename hdr_tags to driver_tags
 * tags: move tags_editor and tags_commit to mx
 * tags: don't depends on header.c
 * tags: Don't duplicate tags allocation
 * tags: fix notmuch editor return code
 * tags: rename DRIVER_LABEL to DRIVER_TAGS
 * always pass char to mutt_str_append_item

always pass char to mutt_str_append_item