Reduce variable scope - mutt/base64.c

Reduce variable scope - mh.c

Reduce variable scope - menu.c

Reduce variable scope - mbox.c

Reduce variable scope - main.c

Reduce variable scope - init.c

Fallback to C.UTF-8 if en_US.UTF-8 cannot be set

* Fallback to C.UTF-8 if en_US.UTF-8 cannot be set

Issue #1088

notmuch_database_open_verbose is in API 4.2.0 + refactor conditional

Issue #1076

notmuch: stop if db open fails

* If the database open fails, stop
* Use the verbose version of open so we can catch the error message

issue #1076

merge: refactor to reduce variable scope

These changes should not affect the behaviour of the code.

 * alias.c
 * attach.c
 * browser.c
 * buffy.c
 * color.c
 * commands.c
 * compose.c
 * conn/getdomain.c
 * conn/socket.c
 * conn/ssl.c
 * conn/ssl_gnutls.c
 * conn/tunnel.c
 * copy.c
 * curs_lib.c
 * curs_main.c
 * doc/makedoc.c
 * filter.c
 * from.c
 * handler.c
 * hdrline.c
 * header.c
 * help.c
 * hook.c
 * imap/auth_cram.c
 * imap/command.c
 * imap/imap.c
 * imap/util.c
 * init.c
 * keymap.c

Reduce variable scope - keymap.c

Reduce variable scope - init.c

Reduce variable scope - imap/util.c

Reduce variable scope - imap/imap.c

Reduce variable scope - imap/command.c

Reduce variable scope - imap/auth_cram.c

Reduce variable scope - hook.c

Reduce variable scope - help.c

Reduce variable scope - header.c

Reduce variable scope - hdrline.c

Reduce variable scope - handler.c

Reduce variable scope - from.c

Reduce variable scope - filter.c

Reduce variable scope - doc/makedoc.c

Reduce variable scope - curs_main.c

Reduce variable scope - curs_lib.c

Reduce variable scope - copy.c

Reduce variable scope - conn/tunnel.c

Reduce variable scope - conn/ssl_gnutls.c

Reduce variable scope - conn/ssl.c

Reduce variable scope - conn/socket.c

Reduce variable scope - conn/getdomain.c

Reduce variable scope - compose.c

Reduce variable scope - commands.c

Reduce variable scope - color.c

Reduce variable scope - buffy.c

Reduce variable scope - browser.c

Reduce variable scope - attach.c

Reduce variable scope - alias.c

Fix usage of mutt_progress_init after b310689

docs: Clarify precedence of settings in config files

Move address to libmutt and merge it with mutt_address

Update to latest autosetup

Issue #1071

fix: add missing library externs

NeoMutt 2018-02-23

update translations

clang-format

new contributors

changelog

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