replace 'ACCOUNT' with 'struct Account'

Fix missing NONULL for mutt.set() in Lua (#578)

fixes #567

Signed-off-by: Guyzmo <guyzmo+github+pub@m0g.net>

build: fix the git_ver.h dependencies

Before, 'git_ver.h' was always considered out-of-date.
That meant that main.c kept being recompiled.

Now, the state of git is checked, but if nothing has changed, nothing
will be rebuilt.

build: only compile hcache on request

If none of the header cache backends are requested, don't go into the
hcache dir.

coverity: fix defects

Most of these trivial problems were discovered when NNTP got enabled in
the default build.

- dereference after null check
- resource leak
- uninit pointer
- uninit var
- unused value

merge: upstream fixes (mutt/default)

 * Convert all exec calls to use mutt_envlist(), remove setenv function.
 * Note that mbox-hooks are dependent on $move.
 * Refresh header color when updating label. (closes #3935)
 * Remove glibc-specific execvpe() call in sendlib.c. (see #3937)
 * Add color commands for the compose menu headers and security status. (closes #3915).
 * Fix sidebar count updates when closing mailbox. (closes #3938)
 * Don't modify LastFolder/CurrentFolder upon aborting a change folder operation.
 * Change message modifying operations to additively set redraw flags.
 * Improve maildir and mh to report flag changes in mx_check_mailbox() (closes #3865)
 * Add $header_color_partial to allow partial coloring of headers.
 * Rename REDRAW_SIGWINCH to REDRAW_FLOW.
 * Create R_PAGER_FLOW config variable flag.

Create R_PAGER_FLOW config variable flag.

Use this for $header_color_partial, $markers, and $smart_wrap.  When
these options are changed in the pager, this flag will force a
recalculation of lineInfo.

Remove the manual checks in OP_ENTER_COMMAND for $markers and
$smart_wrap, and instead use the same REDRAW_FLOW processing used for
a SigWinch.

Rename REDRAW_SIGWINCH to REDRAW_FLOW.

The next patch will attach this redraw flag to pager settings.  The
former name becomes somewhat confusing with the expanded usage, so
rename it to something more relevant to what is being redrawn.

Add $header_color_partial to allow partial coloring of headers.

When set, a regexp match will color only the matched text in the
header.  When unset (the default), the entire header will have color
applied.

With appropriate regexps, this allows coloring of just the header
field name.  Of course, it can also be used to highlight arbitrary
phrases in the headers too.

Improve maildir and mh to report flag changes in mx_check_mailbox() (closes #3865)

mx_check_mailbox() would update the header flags, but was not
returning MUTT_FLAGS back to the index loop.  That meant a screen
redraw was needed to be notified of externally modified flags.

Change maildir_update_flags() to return 1 if the flags were actually
changed.  Change maildir_check_mailbox() and mh_check_mailbox() to
return MUTT_FLAGS when that happens.

Thanks to jcdenton and mike-burns for the original patch.

Change message modifying operations to additively set redraw flags.

With the ability to set redraw flags with the menu stack operations,
some operations internally modify the current menu redraw flag.
For instance, _mutt_set_flag() can now set REDRAW_SIDEBAR.

Change the ops that modify messages to use 'redraw |= REDRAW_X'
instead of overwriting the flag value.

Don't modify LastFolder/CurrentFolder upon aborting a change folder operation.

Set LastFolder and CurrentFolder after mx_close_mailbox() has
successfully completed.  Otherwise, if the close is aborted, they will
have incorrect values.

Fix sidebar count updates when closing mailbox. (closes #3938)

The context unread and flagged counts were being updated too early in
mx_close_mailbox().  Cancelling at any of the following prompts would
leave them in an incorrect state.  Additionally, $move could increase
the delete count (for flagged messages), and $delete, if answered no,
could turn off message deletion.

Move all the sidebar buffy stat updating to the bottom of the
function, after all the prompts and processing.

Add color commands for the compose menu headers and security status. (closes #3915).

Add "color compose header" to color the From/To/Subject/etc fields in
the compose menu.

Add "color compose security_encrypt/sign/both/none" to color the
security status of the message.

Remove glibc-specific execvpe() call in sendlib.c. (see #3937)

Changeset fa1192803257 converted all exec calls to use mutt_envlist().
Unfortunately, the call in sendlib.c, execvpe(), is a glibc extension.

Convert back to execvp() for now, to fix the build on MacOS.

Refresh header color when updating label. (closes #3935)

color index with a '~y' pattern were not being updated after
adding/removing labels.

Note that mbox-hooks are dependent on $move.

Add a note to the "Using Multiple Spool Mailboxes" section.

Convert all exec calls to use mutt_envlist(), remove setenv function.

The documentation implies that all children processes will be affected
by the setenv command, so convert all the exec calls to use
mutt_envlist().

The setenv("GPG_TTY") call is no longer needed so remove it.

With that removed, there are no other setenv calls in mutt, so remove
the autoconf check and replacement function.

factor out debug code

docs: remove some upstream references

Minor code tidying

- remove unnecessary comments
- add missing copyright email
- notmuch: remove orphan function name
- remove unused AM_CONDITIONAL
- drop unnecessary bool definition
- fix F_SENSITIVE constant

merge: drop PATCHES, tidy 'mutt -v' output

 * drop PATCHES
 * version: lowercase, tidy
 * docs: drop refs to patches
 * highlight changes

highlight changes

docs: drop refs to patches

version: lowercase, tidy

drop PATCHES

merge: always build features which have no build deps

 * configure: drop --enable-compressed
 * configure: drop --enable-imap
 * configure: need_md5 is now always built-in
 * configure: drop need_imap
 * configure: drop need_socket
 * configure: drop --enable-pop
 * configure: drop --enable-nntp
 * drop: --enable-smtp
 * Makefile.am: reflow lists of files
 * remove preprocessor else clauses

remove preprocessor else clauses

Remove the #else clauses from the preprocessor now that these symbols
are always defined:
    USE_COMPRESSED
    USE_IMAP
    USE_NNTP
    USE_POP
    USE_SIDEBAR
    USE_SMTP
    USE_SOCKET

Makefile.am: reflow lists of files

These lists were edited but not reflowed to keep the commit diff as
small as possible and readable.

drop: --enable-smtp

Make SMTP feature built-in.

configure: drop --enable-nntp

Make NNTP feature built-in.

configure: drop --enable-pop

Make pop feature built-in.

configure: drop need_socket

Socket files are now always integrated in the build.

configure: drop need_imap

Imap being built-in, body cache is too.

configure: need_md5 is now always built-in

md5 was dependent on other features to be built or not. Now that imap is
built-in, we will always need md5 as built-in as well.

configure: drop --enable-imap

Make imap feature built-in.

configure: drop --enable-compressed

Make compression feature built-in.

build: default to silent make rules

build: add separate hcache dir

build: make dotlock optional

add %z format strings to index_format

The %Z format string consists of three columns.
Split those up into individual columns to allow more fine grained
control over which flags should be displayed.
%zs message status flags
%zc message crypto flags
%zt message tag flags

There's a small difference compared to %Z which is that the delete flags
have been moved from the crypto column to the status column. %Z will not
be removed for backwards compatibility reasons.

Signed-off-by: Stefan Assmann <sassmann@kpanic.de>

Add debug_level/debug_file options

This change allows to change debug_level and debug_file on the fly.

It adds the -l mutt arguments to pass the logging file on startup.

debug options passed as mutt arguments always override the configuration
file.

Closes #445

merge: Warn on bindkey aliasing

 * Warn on bindkey aliasing
 * Create source warnings for binding aliases
 * Detect missing function mapping
 * fix bind error messages

fix bind error messages

Some messages were displayed that shouldn't have been (when bind
succeeded).  Some errors weren't getting reported (they were overwritten
by other messages).

Detect missing function mapping

Move the missing function detection code into the `try_bind` function.
Improve the error message to include the menu name.

Create source warnings for binding aliases

This is somewhat hack-y, but it makes it possible to have the binding
alias trigger source warnings with file name and location.

Warn on bindkey aliasing

On binding a key sequence that alias an existing mapping, warn the user.

merge: configure.ac improvements

 * remove check on unix.h
 * remove check on stdarg.h
 * remove check on inline and const
 * remove check on wide chars
 * remove check on fgetpos
 * remove check on memmove
 * remove check on strerror
 * remove check on srand48
 * remove check on long long int
 * signal: remove test on SYS_SIGLIST_DECLARED
 * simplify sig_atomic_t check
 * remove check for <dirent.h> d_ino
 * drop LANGINFO checks
 * remove stuff defined in POSIX:2001
 * use basic getopt from unistd.h
 * drop HAVE_UNISTD_H
 * unused prototypes - unsupported systems
 * sys/types.h
 * sys/select.h
 * inttypes.h includes stdint.h
 * Drop obsolete iconv check
 * Drop another obsolete iconv check
 * tidy md5

tidy md5

No code changes.

This code is imported from the GNU Library.
It contained a lot of definitions and compatability code that we don't
need.  Also, tidy some comment blocks.

@gahr
Leverage on AC_C_BIGENDIAN instead of glibc's endian.h

Drop another obsolete iconv check

Don't check for a bug in 18 year-old packages

Drop obsolete iconv check

Don't check for a bug in 18 year-old packages

inttypes.h includes stdint.h

<inttypes.h> is part of posix:2001
It includes <stdint.h>

sys/select.h

<sys/select.h> is part of posix:2001

sys/types.h

<sys/types.h> is part of posix:2001

unused prototypes - unsupported systems

Drop compatibility for unsupported platforms

drop HAVE_UNISTD_H

<unistd.h> is part of posix:2001

use basic getopt from unistd.h

getopt() is defined in <unistd.h>
<getopt.h> would only be needed if we used getopt_long()

remove stuff defined in POSIX:2001

Headers:
    locale.h
    sys/time.h
    sys/resource.h
    sys/select.h

Header checks:
    HAVE_FTRUNCATE
    HAVE_INTTYPES_H
    HAVE_ISWBLANK
    HAVE_LOCALE_H
    HAVE_SETRLIMIT
    HAVE_SYS_RESOURCE_H
    HAVE_SYS_SELECT_H
    HAVE_SYS_TIME_H

Functions:
    getopt
    getsid
    setegid
    setenv
    setrlimit
    strcasecmp
    strdup
    strftime
    strncasecmp
    strtok_r
    fseeko

Types:
    pid_t
    ssize_t
    socklen_t

drop LANGINFO checks

posix:2001 defines nl_langinfo in <langinfo.h>

All the constants we use are also part of posix:2001
CODESET, D_T_FMT, YESEXPR, NOEXPR

remove check for <dirent.h> d_ino

posix:2001 declares 'd_ino' in <dirent.h>

simplify sig_atomic_t check

posix:2001 declares 'sig_atomic_t' in signal.h
We only need to check if it's declared with the 'volatile' symbol.
Streamline the autoconf checking.

signal: remove test on SYS_SIGLIST_DECLARED

AC_DECL_SYS_SIGLIST check was removed in:

  7ce41d94e: update configure.ac with autoupdate

So testing SYS_SIGLIST_DECLARED doesn't make sense any more, it has been
replaced by HAVE_DECL_SYS_SIGLIST.

remove check on long long int

This is C99 standard, and nobody relies on the LONGLONG symbol anyway

remove check on srand48

Code doesn't refer to srand48

remove check on strerror

Part of C99

remove check on memmove

Part of C99

remove check on fgetpos

Part of C99

remove check on wide chars

These types (wchar_t, wint_t, mbstate_t) and functions (isw* and tow*)
are part of C99 standard.

remove check on inline and const

AC_C_CONST is marked obsolescent and inline is already checked by
AC_PROG_CC_C99, so it's redundant.

remove check on stdarg.h

As stated in Autotools Manual, AC_HEADER_STDC already checks for
stdarg.h presence, so double checking is useless.

remove check on unix.h

This header is not used anymore, this check can be dropped.

build: fix dist, distclean targets

'make dist' didn't install the 'lua' and 'hcache-bench' dirs.
'make distclean' deleted 'po/mutt.pot' and 'doc/neomutt-syntax.vim'

Fixes #566

build: change gpgme requirement back to 1.1.0

RHEL6 is still using GPGME 1.1.8, which doesn't have the function
`gpgme_op_export_keys`.  Revert the GPGME requirement to 1.1.0 and put
back the #ifdef's around the missing function.

build: remove check_sec.sh

This script looked for mistakes when using mutt functions such as
safe_free().  It was crude.  Coverity can do a better job without us
having to litter the code with /* __FREE_CHECKED__ */ comments.

configure: make it more consistent

Most of the diff is reworking how notmuch is built. It used an extra
AM_CONDITIONAL but was the only component to do so, so change that to
make it consistent with the rest of the configure.

Lua picked the same thing but BUILD_LUA is actually never used, so it
could be dropped too.

Rest of the commit cleans up some pollution between MUTTLIBS and LIBS.

remove some obsolete HAVE_ tests

Delete tests against some defines because there is nothing in configure
to generate those defines:

     HAVE_ALLOCA_H
     HAVE_GNUTLS_OPENSSL_H
     HAVE_PGP
     HAVE_REGCOMP
     HAVE_SMIME

They are like `#if 0`, only harder to spot!

clang-format: Insert spaces inside braces

Adding the option 'Cpp11BracedListStyle: false' means that struct data
in braces has some whitespace padding.

const char *missing[3] = {"brown", "lightgray", "default"};

const char *missing[3] = { "brown", "lightgray", "default" };

clang-format: adjust layout of structs

Some structs are packed by clang-format, leaving the closing brace
n the same line as the data:

char *known[] = {"color-xterm", "cygwin", "eterm",  "kterm", "nxterm",
                 "putty",       "rxvt",   "screen", "xterm", NULL};

By inserting a comma after the last data item (allowed by C99),
clang-format produces a more pleasant-looking layout:

char *known[] = {
    "color-xterm", "cygwin", "eterm",  "kterm", "nxterm",
    "putty",       "rxvt",   "screen", "xterm", NULL,
};

translations: remove unused (f)orget option

This undocumented option interfered with the order of the translations.

merge: cleanup of lots of docs

doc: remove feature muttrc files

are duplicates of manual muttrc sections

merge README.notmuch into manual

remove unneded scripts

* the functionality of the mutt_xtitle script already covered by
  https://www.neomutt.org/guide/reference#ts-status-format (according to
  @flatcap)

[ci skip]

remove README.SECURITY

content is already covered inside
https://www.neomutt.org/guide/mimesupport.html#secure-mailcap

[ci skip]

remove BEWARE and devel-notes.txt

* For the inclusion of the BEWARE/devel-notes.txt file on the website,
  please see: neomutt/neomutt.github.io#33/

[ci skip]

update Makefiles

delete TODO files

see https://github.com/neomutt/neomutt/milestone/9

remove legacy files

* README (infos for mutt 1.5) doesn't contain useful infos
* NEWS (infos about mutt 1.2) is too legacy, neomutt started with mutt 1.5

don't generate vim-neomutt syntax file

doc: remove LaTeX/pdf manual generation

remove --enable-sidebar and make it persistent

It was decided to keep the USE_SIDEBAR tests in the codebase to make it
easier to find code related to that feature, but that requires
USE_SIDEBAR to be defined, or it breaks compilation because some symbol
definitions depend on that config option.

github: improve PR template and CONTRIBUTING.md

PR template:

* made structural questions bold to contrast it from the actual
  PR description.

* add note that acceptance criteria can be removed when the PR is
  submitted. Its only meant as a reminder.

* fixed manual link.

CONTRIBUTING.md:

* wrap text at 80 characters.

* add note to mention specific methods or files if the commit only
  changes things there.

[ci skip]

NeoMutt Release 2017-04-28

Fix km_error_key() infinite loop and unget buffer pollution.

'bind pager \Ch help' produces an infinite loop when an unbound key is
pressed in the pager.  The reason is because km_error_key() tries to
verify that the key sequence is really bound to the OP_HELP operation.
It does this by using km_expand_key(), tokenize_unget_string() on the
resulting buffer, then checking if the next km_dokey() returns OP_HELP.

The problem is that km_expand_key() does not always produce a string
that is properly reparsed by tokenize_unget_string().  Control-h
sequences are expanded to ^H.  tokenize_unget_string() recognizes this
as two characters '^' and 'H'.  km_error_key() checks the OP returned,
which is OP_PAGER_TOP for the '^'.  This is not OP_HELP, so it prints
a generic error and returns.  This leaves the 'H' in the input buffer!
Since 'H' (by default) is unbound in the pager, it retriggers
km_error_key(), resulting in an infinite loop.

The same issues can occur without control sequences:
  bind generic ? noop
  bind generic dq help
In the index, hitting an unbound key will end up leaving 'q' in the unget
buffer, because 'd' is bound in the index menu and will be read by km_dokey().

A simple approach to fix this would be to just use the same code as in
mutt_make_help(), which has no double-check.  This would be no worse
than the help menu, but can generate an inaccurate error message (e.g
if '?' were bound to noop)

This patch instead uses OP_END_COND as a barrier in the unget buffer.
It directly inserts the keys in the OP_HELP keymap, instead of using
km_expand_key() + tokenize_unget_string().  After calling km_dokey()
it flushes the unget buffer to the OP_END_COND barrier.

Thanks to Walter Alejandro Iglesias for reporting the bug.

Fix error message when opening a mailbox with no read permission. (closes #3934)

ctx->mx_ops ends up being NULL for both the case that ctx->magic is 0
and -1.  This meant the mutt_perror() error message was never being
printed, because the check for ctx->mx_ops == NULL was taking place
first.

Move the "ctx->magic == -1" check first, so mutt will print out an
appropriate perror message in that case.

docs: fix remaining TLS typos

Fix typo in README.neomutt (#548)

Fix and simplify handling of GPGME in configure.ac (#545)

* Fix
When GPGME is not installed, the m4 macro AM_PATH_GPGME is not defined,
which causes autoconf to fail. The fix adds an `ifdef` around the block
to skip expanding the macro, if it's not defined.

* Enhancement
The enhancement is to assume that we have at leat GPGME 1.2.0 (released
in 2009). This allows us to simplify the configure.ac code block by
avoiding checks for functionalities we know to be there.

Fixes #542

NeoMutt Release 2017-04-21

sync translation to upstream changes