version 2018-02-23

fix: missing ! when checking regex pointer

merge: trivial improvements

 * doxygen fix comments
 * use exit
 * constants
 * disable the folder-hook for $record
 * unused variables
 * fix redundant code

fix redundant code

unused variables

These variables set then either
- overwritten before being used
- not used before going out of scope

disable the folder-hook for $record

It's possible to use $record to save an email to a different account.
The right thing to do is to call the `folder-hook` for the save folder,
save the email, then call the `folder-hook` for the 'current' folder.

Calling the hook this way causes problems for lots of people who have
commands in their hooks that are called when they didn't expect.

The hooks are left in the code wrapped in: `#ifdef RECORD_FOLDER_HOOK`

constants

use exit

doxygen fix comments

merge: upstream fixes

 * Remove useless assignments in add_to_list functions.
 * Add history-search function, bound to ctrl-r.
 * Avoid a potential integer overflow if a Content-Length value is huge.
 * Updated French translation.
 * update translations

update translations

Updated French translation.

Avoid a potential integer overflow if a Content-Length value is huge.

Add history-search function, bound to ctrl-r.

Create a very basic "search history" functionality in the line editor.
It uses the current input, and searches backward through history.  If
there is one match, it immediately uses that otherwise it pops up a
simple menu of matches.

Remove useless assignments in add_to_list functions.

The code was probably pulled from some of the header processing
functions, which were in a loop.  In those versions, "last" pointed to
a collection of new entries appended, but in these two functions,
updating last just obfuscates the code.

Allow choosing a particular version of BerkeleyDB via --with-bdb-version (#1064)

Fix #1005

fix regexes using '\!' (not) (#1062)

The flag, in struct Option, allowing a regex to use a '!' (not) is part
of the type, not the flags field.

Fix decoding of parameter in RFC2231 format (#1061)

Issue #1060

Ncurses always has resizeterm, remove unnecessary ifdefs

Issue #1057

Split HAVE_RESIZE out of HAVE_RESIZETERM

Update autosetup, use negated and nested conditionals in @if substitutions

fix crash when decoding parameters (#1056)

Fix out-of-bounds read in mutt_str_lws_len

Issue #1054

Alternative way to check for DocBook DTD

merge: regex fixes

 * fix partial initialisation of Regex config
 * carefully check all regex pointers
 * Fix free_opt for regex

Fix free_opt for regex

carefully check all regex pointers

fix partial initialisation of Regex config

configure: Check for Docbook DTDs

* configure: Check for Docbook DTDs

Run xsltproc with a minimal Docbook file.
If this succeeds, then the docs will probably build correctly.

Co-authored-by: Pietro Cerutti <gahr@gahr.ch>

Tidy parameter API, switch to TAILQ

Move parameter.[ch] to mutt/

trim some hcache descriptions

boolify some variables

fix docs for mutt_md5_toascii

fix regex completion

reply_regexp -> reply_regex

quote_regexp -> quote_regex

browser: <change-dir> mutt_realpath() <goto-parent>

For "regular" filesystems (not IMAP or Notmuch folder hierarchies),
<change-dir> and the browser GUI now resolve paths via a new mutt library
function `mutt_realpath`. This means no more path buildup such as
"a/b/../../c". Additionally, Symlinks are resolved automatically. This is
a circumstance of realpath(). Note that a future PR is planned to add a
non-symlinking option.

The new <goto-parent> function has a default shortcut key of 'p' for parent.
Since this function uses the pre-existing `mutt_get_parent_path()`,
<goto-parent> may also work on IMAP and Notmuch folder hierarchies
(tested and working on IMAP)

Note: The paths passed to `mutt_get_parent_path()` formerly would not
parse correctly a path which included a final trailing slash (eg 'abc/')
So, I added it a check to that function to remove it before hitting
the parser.

Note2: The browser for IMAP was janky before this commit, but the browser
is not the primary mailbox navigation tool, afterall. At least 'p'
<goto-parent> adds to its functionality. I expect a similar evaluation for
Notmuch folder browsing, although I have never used Notmuch. Any
contributions to make a better IMAP/Notmuch folder browsing experience
are welcome. I have reported some of the fallbacks in Github PR #1037
comments (look for IMAP headlines).

Fix off-by-1 bug in strnfcpy, add test cases

Co-authored-by: Michael Bazzinotti <mbazzinotti@gmail.com>

fix rfc2047 field decoding

fix parsing of urls containing '?'

The URL schemes come in two patterns
- smtp://user:pass@host
- notmuch:///path?query

The 'host' types don't take a query string.
'notmuch' and 'mailto' don't use a 'user:pass' string.

This fix will still be confused by a notmuch URL that has a '?' in the
path component, but no query string.  (but I don't think the code ever
coped with that).  e.g.

- notmuch:///dir?name/

Fixes #1043

Let mutt_str_strnfcpy return the string length

Let mutt_str_strfcpy return the string length, add tests

merge: history refactoring

 * reduce scope of variables
 * rename functions
 * code tidying
 * doxygen: add comments

doxygen: add comments

code tidying

- Check ints against 0
- Rename parameters
- Bracket logic
- Reduce scope of variables

rename functions

reduce scope of variables

merge: trivial improvements

 * coverity
 * name unicode chars
 * Fix spelling
 * doxygen tidying
 * Add a blurb about $new_mail_command in the mailboxes section.

Add a blurb about $new_mail_command in the mailboxes section.

doxygen tidying

Fix spelling

name unicode chars

coverity

merge: md5 improvements

 * Remove unused functions, unexpose implementation functions
 * Implement and use mutt_md5_toascii
 * Initial tests
 * Use mutt_md5_toascii some more
 * Add and use an API to process a NULL-terminated string
 * More API polishing
 * Rename test cases to better reflect what they're doing

MD5 - Rename test cases to better reflect what they're doing

MD5 - More API polishing

MD5 - Add and use an API to process a NULL-terminated string

MD5 - Use mutt_md5_toascii some more

MD5 - Initial tests

MD5 - Implement and use mutt_md5_toascii

MD5 - Remove unused functions, unexpose implementation functions

Kill some magic numbers, use mutt_array_size

Make Weekdays, Months, and TimeZones statics, expose mutt_date_make_tls

Kill mutt_rfc2047_encode_32 macro

Fix reply_regexp which wasn't formatted correctly.

And add Swedish "Sv:" prefix to reply_regexp.

Unit test framework (#1031)

Minimal unit-test framework to test C library APIs

merge: RFC2047 improvements

 * Handle RFC2047 words split in the middle of a multibyte character
 * move rfc2047 functions to library
 * Use our base64 API instead of duplicating the decoding algorithm
 * Move mutt_rfc2047_choose_charset to mutt_ch_choose
 * light tidy

light tidy

* fix typos
* rename parameters
* add ()s to logic

Move mutt_rfc2047_choose_charset to mutt_ch_choose

Issue #1015

Use our base64 API instead of duplicating the decoding algorithm

Also, fix a memory leak while at it.

Issue #1015

move rfc2047 functions to library

Handle RFC2047 words split in the middle of a multibyte character

fix for pgpewrap in default gpg.rc (#1023)

Fixes an bug introduced by changes intorduced in commit
465fad6d21ac3b9d2ae5d12bae7705f5f04323e7 to contrib/gpg.rc that breaks signed
and encrypted mail. The issues is just a typo related to way that pgpewrap
works. This should fix issue #998.

merge: light refactoring

 * move unsigned atoi functions
 * comparison ordering
 * braces
 * move functions to library

move functions to library

braces

comparison ordering

move unsigned atoi functions

Fix a couple memory leaks in pattern.c

BUFFER.data was not being freed in a couple cases.

The pattern and simple search were not being freed if imap_search()
failed.

merge: fix coverity defects

 * drop unreachable condition
 * set new regex
 * negative check
 * check size_t value
 * unused result
 * ptr checks
 * negative index
 * tidy logic
 * fix logic
 * unchecked returns
 * forward null checks

forward null checks

unchecked returns

fix logic

tidy logic

negative index

ptr checks

unused result

add new var for calc

check size_t value

negative check

set new regex

drop unreachable condition

Simplify ALL_FILES definition, make it easier to add/remove files

merge: trivial improvements

 * rename parameters bufsz -> buflen
 * tidy mapping functions
 * move mutt_pretty_size to libmutt
 * iwyu: remove unnecessary headers
 * rename charset functions
 * mutt_str_getenv()
 * use mutt_str_getenv()

use mutt_str_getenv()

mutt_str_getenv()

rename charset functions

To avoid a clash with the configset functions to come.

iwyu: remove unnecessary headers

move mutt_pretty_size to libmutt

tidy mapping functions