Revert "don't close the pager after a timeout"

This reverts commit 5acd76ee270428302ec5e623a0cf743c17c4c05b.

Support Gmail's X-GM-RAW server-side search

Let Gmail search filters be used from mutt with =/

Closes #644

fix gpgme S/MIME non-detached signature handling

- gpgsm ends the session after the decrypt error, so restart gpgsm when
  retrying with "maybe_signed".
- evolution sends encrypted + signed S/MIME mails with non-detached
  signatures

merge: minor build improvements

 * move smime_keys into contrib
 * drop instdoc script
 * remove unnecessary steps
 * tidy makefile

build: tidy makefile

This commit makes no changes to the build.
It simply tidies up the whitespace and some lists.

build: remove unnecessary steps

Three man pages were processed with sed, but it didn't alter the files.

build: drop instdoc script

This script performed some string substitutions in the files it was
installing.  This was already done by the Makefile.

build: move smime_keys into contrib

menu: fix the thread tree color

The background tree color is really meaningless as it mixes with the
background color of indexes subject in unpleasant ways. We prefer to
create a new color based on the tree foreground color only.

browser tweak

See issue #609

Uses CurrentFolder to populate LastDir with IMAP

 * Needs to rollback to !folder instead of !*LastDir
 * Closes issue #640

fix perms

don't close the pager after a timeout

Setting a timeout hook could cause the pager to automatically close
itself when viewing help or an attachment.

    timeout-hook 'exec sync mailbox'

The timeout was causing a fake keypress event which cause the pager to
quit.

Fixes #591

colour emails with a '+' in them

The default highlighting for email addresses didn't recognise '+'
characters as being valid.

Fixes #496

sidebar: stabilise sort order

When sorting by a numeric type, fallback to the path if the numbers are
the same.

Fixes #486

doc: limitations of new-mail %f expando

docs: escape <>'s in nested conditions

Add config vars for forwarded message attribution intro/trailer.

Add $forward_attribution_intro and $forward_attribution_trailer to
allow the default strings to be overridden.

Fixes #522

dupe-string, don't destroy original

fix the padding expando '%>'

Debian BTS 866366

The non-greedy '%>' has been broken since the introduction of the
nested-if patch.

The second command should create a line of dots ending in a ']'

    set index_format="%?M?collapsed%|.]&expanded?
    set index_format="%?M?collapsed%>.]&expanded?

merge: add some colour schemes

 * added neonwolf colorscheme
 * added black 256 solarized colorscheme
 * added zenburn colorscheme
 * added vombatidae colorscheme
 * removed note to use slang for neonwolf colorscheme
 * build: add colour schemes

build: add colour schemes

Closes #340

removed note to use slang for neonwolf colorscheme

Thanks to @h3xx for the hint.

added vombatidae colorscheme

Thanks to @octol

added zenburn colorscheme

Thanks to Renato Cunha

Source:
https://bitbucket.org/trovao/configs/src/c3413bafa5ba70da2329225a9009078b078ccee1/mutt-zenburn/dotmuttrc

added black 256 solarized colorscheme

Thanks to @altercation

added neonwolf colorscheme

Thanks to @h3xx for publishing it.

merge: build fixes for Solaris

 * Handle native Solaris GSSAPI.
 * Fix compilation error on Solaris.
 * Include term.h for tigetstr except for slang build.
 * Look for wide curses in libcurses itself.
 * Remove unneeded forward declarations.

Remove unneeded forward declarations.

Look for wide curses in libcurses itself.

NetBSD's curses library provides wide character support.

Include term.h for tigetstr except for slang build.

Fixes
curs_main.c:529:11: warning: implicit declaration of function ‘tigetstr’ [-Wimplicit-function-declaration]

Including it before header.h causes compilation errors because
of "int lines;" in header.h and
in term.h.

Fix compilation error on Solaris.

  resize.c: In function 'mutt_resize_screen':
  resize.c:38:18: error: storage size of 'w' isn't known
     struct winsize w;
                    ^
  resize.c:47:19: error: 'TIOCGWINSZ' undeclared (first use in this function)
       if (ioctl(fd, TIOCGWINSZ, &w) != -1)
                     ^

Handle native Solaris GSSAPI.

Signed-off-by: Thomas Klausner <wiz@NetBSD.org>

Do not set old flag if mark_old is false

If `mark_old` is false, unread mails will not be marked old after
they've been viewed. But they will be marked old if found in the `/cur`
subdirectory of the Maildir. This is inconsistent behaviour, since a
user with disabled `mark_old` will expect no old flags at all.

merge: fix some leaks

 * fix some resource leaks
 * fix leaks in lua get/set options
 * fix dodgy buffers

fix dodgy buffers

fix leaks in lua get/set options

fix some resource leaks

- CID 169545
- CID 169546
- CID 171381
- CID 171382

Block SIGWINCH during connect(). (closes #3941)

FreeBSD's connect() does not respect SA_RESTART, so a SIGWINCH will
end up interrupting the connect.

If this code is needed in other places, it should be moved into
signal.c.  For this one place, inlining the sigprocmask() seemed okay.

fix whitespace errors

tidy the version strings

sync to upstream's "encrypt-to-self"

NeoMutt's variables:
    pgp_encrypt_self
    smime_encrypt_self

have been superceded by Mutt's:
    pgp_self_encrypt
    pgp_self_encrypt_as
    smime_self_encrypt
    smime_self_encrypt_as

Note: NeoMutt's variables will be deprecated at a later date.

More sophisticated check for BDB version + support for DB6 (non default)

This commit enhances configure.ac to actually parse (as in CPP) and
extract the values of DB_VERSION_MAJOR and DB_VERSION_MINOR from the
Berkeley DB include files. The previous approach to just use grep failed
on some system where the includes are just wrappers around the real bdb
headers.

Also, add support for the current db6 version (6.2), although db5 is
still the first one looked for.

Issue #613

fix: initialise debug_file

Make sure that `debug_file` is initialised early enough.
Also, call `mutt_expand_path()` to enable ~/ shortcuts.

Related to issue #654

contrib: add Mutt logos

Issue #655

revert part of 40f81d1 Add debug_level/debug_file options

This patch moves the set_default/restore_default couple after the defaults have
been set by the init function; this was correctly placed in 1.7.2 and somehow it
got moved to the wrong place, which means that tmpdir (and many other variable)
are not correctly set with their new default value determined from environment
variables (or other logic), so when they get resetted with a 'reset all' call in
.muttrc, they will never have the correct default to fall back to and they will
use the wrong one in init.h

This patch fixes:
- http://bugs.debian.org/865822
- http://bugs.debian.org/865842

Issue #654

score address pattern do not match personal name

Fixes #646
Debian BTS 383769

open attachments in read-only mode

Fixes #647
Debian BTS 528233
Mutt trac #3261

Add Cc, In-Reply-To, and References to default mailto_allow

These headers are quite useful for replying to mailing lists
via mailto links, as they don't break threading.

Debian has been shipping these in the /etc/Muttrc file since
September 2016 without much issues.

Fixes #115

merge: trivial code changes

 * build: don't delete non-existent files
 * remove unused option
 * docs: fix typos
 * update mailmap

update mailmap

docs: fix typos

Fixes #648

remove unused option

build: don't delete non-existent files

merge: upstream fixes (mutt/default)

 * Change the compose menu fields to be dynamically padded.
 * Rename 'sign as' to 'Sign as'; makes compose menu more consistent.
 * Make GPGME key selection behavior the same as classic-PGP. (see #3950)
 * Force hard redraw after $sendmail instead of calling mutt_endwin. (closes #3952) (see #3948)
 * Enable all header fields in the compose menu to be translated.
 * Auto-pad translation for the GPGME key selection "verify key" headers.
 * Improve the L10N comment about Sign as.

Improve the L10N comment about Sign as.

Make it clearer what the indentation behavior is, and add suggestions
about the width.

Thanks to TAKAHASHI Tamotsu for the original patch suggesting changes
to the L10N comments.

Auto-pad translation for the GPGME key selection "verify key" headers.

Remove the ridiculous need for the translators to pad the translation
strings themselves.

Enable all header fields in the compose menu to be translated.

While it is suggested they need not be, for some locales it might be
preferable to be able to localize the fields.

For instance, fr_FR may want to change the punctuation to have a space
in front of the colon.  ja_JP may be more comfortable with a native
character set description of the field when using it every day.

Force hard redraw after $sendmail instead of calling mutt_endwin. (closes #3952) (see #3948)

Adding a mutt_endwin() seemed like a clean solution to allowing
ncurses pinentry for $sendmail, but it leaves other users watching a
blank screen.  This change is extremely likely to generate a large
number of complaints and bug reports.  So instead, force a hard
refresh afterwards.

Make GPGME key selection behavior the same as classic-PGP. (see #3950)

Classic-PGP key selection auto-selected a match if there was a single
strong, valid, address-match result, even if there were other weak
matches.

GPGME was prompting in the same situation, if there were other weak
matches.

Change GPGME to match the classic behavior, as this is more useful and
matches the manual description.

Rename 'sign as' to 'Sign as'; makes compose menu more consistent.

Change the compose menu fields to be dynamically padded.

Pad based on the maximum width of the fields.  Note that this is a bit
of a mess, because some of the fields are translated while others are
not.

Update documentation for modify-labels-then-hide

This is a follow up to #602, documenting the new
behavior.

make modify-labels-then-hide hide or unhide

If the message is no longer in the current query after modifying
the label it will be hidden as before, but if it is in the current
query (again), it will be unhidden.

Related to #601

Add reorganised collapse_flagged patch

Decode CRLF line endings to LF when copying headers

This fixes display issues with files that consist of CRLF
line endings rather than just LF line endings, for example,
emails exported directly from the gmail API.

This only kicks in when CH_DECODE is set which might be a bit
strange, because that does RFC2047 header decoding, but it seems
a safe place to place this line end decoding without potentially
affecting anything else.

Include pattern for broken threads

References #616

From Frederic Duval:
http://cedricduval.free.fr/mutt/patches/download/patch-1.5.1.cd.pattern_broken.1

Improve search for mime.types

- Make the search order sane
- Print an error if mime.types is not found

Search order is:
    /etc/mime.types
    SYSCONFDIR/mime.types
    PKGDATADIR/mime.types
    ~/.mime.types

The last matching file wins.

fix: maildir creation

- added missing recursive directory creation after user prompt
- checked for trailing '/' when saving mbox

merge: start tidying Buffy handling

 * tidy some buffy code
 * split mutt_parse_mailboxes into mutt_parse_unmailboxes
 * drop VirtIncoming

drop VirtIncoming

Change the notmuch code to use Incoming rather than its own VirtIncoming.
These changes have no visible effect to the user.

This removes a lot of duplicate code and simplifies the logic in the
sidebar.

split mutt_parse_mailboxes into mutt_parse_unmailboxes

Split mutt_parse_mailboxes into two functions: one to add, one to remove.
There's not much overlap in the two functions and separating them
simplifies the logic.

Where unmailboxes used to use two loops, there's now only one.

tidy some buffy code

As a prelude to some notmuch changes, tidy up some buffy-handling
functions.  There are no functional changes.

- rename some variables
- reduce the scope of variables
- add some bools

CODE_OF_CONDUCT: remove jekyll code

[ci skip]

docs: add code of conduct

[ci skip]

merge: trivial code changes

 * reduce scope of for loop variables
 * replace variables with bools
 * rename static function
 * strip out once-used macro
 * while (1) -> while (true)

fix mailmap

while (1) -> while (true)

strip out once-used macro

rename static function

replace variables with bools

Each variable is limited to a single function.
Each was only set to 0 or 1.

Fucntions:
address_header_decode
alternative_handler
attach_forward_bodies
check_host
ci_send_message
clean_references
copy_clearsigned
crypt_getkeybyaddr
crypt_select_key
crypt_write_signed
disable_coredumps
display_line
do_uncolor
draw_sidebar
eat_message_range
eat_range
envelope_defaults
expand_aliases_r
get_token
handle_docline
imap_auth_sasl
imap_browse
imap_conn_find
imap_fast_trash
imap_fetch_message
imap_make_msg_set
imap_pretty_mailbox
imap_read_headers
imap_read_literal
init_common
init_menu
maildir_check_mailbox
main
mbox_check_mailbox
menu_redraw_index
mh_rewrite_message
mix_check_message
mix_make_chain
mutt_account_hook
mutt_alias_menu
mutt_attach_reply
mutt_body_handler
mutt_buffy_check
mutt_builtin_editor
mutt_check_simple
mutt_compose_attachment
mutt_copy_hdr
mutt_count_body_parts
mutt_display_message
mutt_draw_statusline
mutt_edit_attachment
mutt_edit_headers
mutt_expand_fmt
mutt_FormatString
mutt_index_menu
mutt_is_application_smime
mutt_multi_choice
mutt_pager
mutt_parse_color
mutt_parse_hook
mutt_parse_mono
mutt_parse_multipart
mutt_print_attachment
mutt_progress_update
mutt_remove_duplicates
mutt_sasl_start
mutt_signed_handler
mutt_view_attachment
mutt_write_fcc
mutt_write_rfc822_header
mutt_yesorno
nm_check_mailbox
nm_nonctx_get_count
nm_sync_mailbox
nntp_capabilities
nntp_hcache_update
nntp_newsrc_gen_entries
nntp_open_connection
parse_parameters
parse_pub_line
parse_regex
pgpring_dump_keyblock
pgp_application_pgp_handler
pgp_copy_clearsigned
pgp_decrypt_mime
pgp_find_keys
pgp_getkeybyaddr
pgp_getkeybystr
pgp_gpgme_application_handler
pgp_gpgme_decrypt_mime
pgp_read_packet
pgp_select_key
pgp_sign_message
pgp_traditional_encryptsign
pop_fetch_headers
print_smime_keyinfo
qp_decode_line
query_menu
reopen_mailbox
resolve_color
resolve_types
rfc1524_expand_filename
rfc2047_decode
rfc2231_decode_parameters
select_msg
show_one_sig_status
shrink_histfile
smime_getkeys
smime_select_key
ssl_init
tls_init
utf8_to_utf7
_mutt_expand_path
_mutt_parse_color
_mutt_traverse_thread

reduce scope of for loop variables

Each change is limited to a single function:
    attach_forward_bodies
    attach_forward_msgs
    attach_reply_envelope_defaults
    check_all_msg
    find_parent
    mutt_attach_bounce
    mutt_attach_reply
    mutt_attach_resend
    mutt_builtin_editor
    mutt_gen_attach_list

translation: update Chinese (Simplified)

init.c: allow sourcing of multiple files

Credits: Cedric Duval,
http://cedricduval.free.fr/mutt/patches/download/patch-1.5.4.cd.source_multiple.2

Issue #616

docs: fix ifdef examples

Fix trailing whitespaces

merge: drop obsolete options / docs

update gitignore

Update .gitignore to match all the latest build changes.

Completely remove dotlock

The last-ditch attempt to lock a file, dating back to the times when NFS
and SMB file-locking wasn't reliable.

drop UPDATING files

Drop:
    UPDATING    (from upstream)
    UPDATING.gz

The information in here is also in our ChangeLog.

One ChangeLog to bind them all

Drop files:
    ChangeLog      (from upstream)
    ChangeLog.nntp

Rename:
    ChangeLog.neomutt -> ChangeLog.md

drop configure options --enable-exact-address

Unused option.

drop configure option --with-exec-shell

Using /bin/sh is safe.

drop configure option --enable-nfs-fix

No longer needed.

drop configure option --disable-warnings

Always enable the warnings.

remove stamp-h.in

This file is generated by configure and shouldn't be committed to the
repo.

remove references to synonyms

Undocumented           | Real variable
-----------------------|-------------------------
edit_hdrs              | edit_headers
envelope_from          | use_envelope_from
forw_decode            | forward_decode
forw_decrypt           | forward_decrypt
forw_format            | forward_format
forw_quote             | forward_quote
hdr_format             | index_format
indent_str             | indent_string
mime_fwd               | mime_forward
msg_format             | message_format
pgp_autoencrypt        | crypt_autoencrypt
pgp_autosign           | crypt_autosign
pgp_auto_traditional   | pgp_replyinline
pgp_create_traditional | pgp_autoinline
pgp_replyencrypt       | crypt_replyencrypt
pgp_replysign          | crypt_replysign
pgp_replysignencrypted | crypt_replysignencrypted
pgp_verify_sig         | crypt_verify_sig
post_indent_str        | post_indent_string
print_cmd              | print_command
smime_sign_as          | smime_default_key
xterm_icon             | ts_icon_format
xterm_set_titles       | ts_enabled
xterm_title            | ts_status_format

Remove refs to deleted configure options

Obsolete configure options:
    --enable-hcache
    --enable-imap
    --enable-mailtool
    --enable-nntp
    --enable-pop
    --enable-smtp
    --with-included-gettext
    --with-regex

cleanup hdr_format_str

Sort options in alphabetical order for better overview.
Also while at it, removed a few superfluous blank lines.

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

doc: fix spaces in the manual

Thanks to Bo YU (@yuzibo) and Floyd Anderson (@floand)

These were introduced when `tidy` was run.