Problems with muttrc.5:

Ambiguous or invalid backslash.  This doesn't cause groff a problem.
but it confuses doclifter and may confuse older troff implementations.

Unknown or invalid macro.  That is, one that does not fit in the
macro set that the man page seems to be using.  This is a serious
error; it often means part of your text is being lost or rendered
incorrectly.

Use local definitions of .EX/.EE or .DS/.DE to avoid low-level troff
requests in the page body.  There are plans to add these to groff man;
in the interim, this patch adds a compatible definition to your page.

Problems with mutt.1:

Ambiguous or invalid backslash.  This doesn't cause groff a problem.
but it confuses doclifter and may confuse older troff implementations.

Spelling error or typo.

include <stdbool.h> for definition 'true'

closes #3642

Fix segfault when viewing text attachments in compose menu.  (closes #3644)

The segfault was introduced in changeset b9f9e3147eb4.  Since decoding
and charset conversion aren't needed for attachments when composing a
message, this patch reverts to just using mutt_save_attachment() to view
"raw data" for text attachments in the compose/send case.

This patch is based on Michael Elkins' patch at
http://dev.mutt.org/trac/attachment/ticket/3644/view_attach_compose_segfault
with just a missing return value check added.

Ensure that BUFFER variables are initialized to prevent later attempts
to traverse an uninitialized pointer.

Consistently use mutt_buffer_init function

Despite the existence of the mutt_buffer_init function, most
initializations were instead done with memset.  Use the specific
function instead to make it easier if later changes are made to how the
initialization should be done.

fix more comment typos

fix typos in comments

Don't exit pager if quit=ask is answered "no".

This patch addresses ticket 2788.

Currently if you quit ('Q') from inside the pager, with quit=ask-yes
or quit=ask-no, and you respond "no", mutt leaves you back in the index
menu.

This patch changes it so that mutt does not exit the pager.

backout c1371176ea45

Backed out changeset 1142ed8974fa

use gethostname() to determine the system host name

use getaddrinfo() to look up canonical DNS name, and fall back to hinting from /etc/resolv.conf

see #3298

fix various compiler warnings; most were due to unchecked return values from system calls.

use mkdtemp() to create temporary directory rather than mktemp() followed by mkdir()

closes #3637

use of sscanf() had undefined behavior, replace with simple parsing instead

see #3636

increase string size by one byte to accomodate the terminating null character in fscanf() since the maximum field width does not include it

closes #3636

hat tip: Will Fiveash

fix potential buffer overflow in off-by-one bounds checking error

closes #3635

explicitly call mutt_refresh() prior to endwin() to force a screen redraw.

[4903e80356d3] removed the call, but some environments (screen+xterm under
SuSE11/12) don't properly flush without it prior to the endwin() call

do not use mutt_message() to display prompt in mutt_yesorno().

properly calculate screen width of prompt string by using mutt_strwidth() rather than strlen()

use safe_asprintf() return the formatted yes/no string since specifying COLS+1 is wrong when dealing with multibyte characters

closes #3412
closes #3352

add support for $mail_check_recent to mh mailbox driver

closes #3629

do not explicitly reset the terminal attributes and move the cursor in mutt_endwin().  endwin() itself should handle this.

closes #3627

remove PKG_CHECK_MODULE() check and always use AC_CHECK_HEADERS()

only check for <idn/idna.h> when <idna.h> isn't found

closes #3626

use pkg-config to look for libidn if available.

checking <idn/idna.h> rather than including -I/usr/include/idn for Solaris 11.

closes #3624

backout [392e945dfba7]

closes #3293
see #3234

when falling back to viewing an attachment as text in the internal pager, perform charset conversion

this change is meant to replace [392e945dfba7]

see #3234
see #3293

use a separate flag for mutt_hcache_store() to signal that the gettimeofday() value should be substituted, so that a 0 uidvalidity can be used by the imap driver.

closes #3501

change semantics of mutt_is_autoview() to return 1 if there is a matching mailcap entry and either $implicit_autoview is set or the MIME type matches the user's auto_view list.

closes #3496
closes #3426

detect pipermail-style obscured return path in mbox From lines

see #1200

allow use of sync-mailbox when the current limit has no visible messages

closes #3538

avoid debug warning about trailing whitespace in a base64 encoded attachment

closes #3558

When "make install" fails to chgrp or chmod the mutt_dotlock binary, add the reason why we are doing this so the user knows what to look for.

Fix regression introduced by [10cc06eff52b] where the CLEARLINE() removal was not replaced by a move() to position the cursor at the correct location to draw the status bar.

closes #3619

Remove unused variable `matched'.

Fix memory leak when multiple x-label or supersedes header fields exist in the header.

In order to keep the current behavior where the value of the last header field present is the one used, free any previous value.

Backed out changeset bf84f24e512d

display +/-NCURSES_WIDECHAR when displaying ncurses version to show whether we are linked against -lncurses or -lncursesw

Drop unneeded variable HEADER_CACHE that causes clang warnings.

The variable was unused and clang warned that it was not emitted to the .o file.

fix xml validation error in manual.xml in auto-generated content due to missing paragraph mark in the $editor docstring

Fix accidental var redeclation with --enable-exact-address

closes #3615

merge

reset to color normal before clearing to end of line when printing a message/error string

closes #3617

update regexp to include const keyword.  The auto generated documentation of functions was failing due to a change in the source code format.

Ensure that the in-reply-to header field was actually present before deciding it was removed.

closes #3378

update ChangeLog

add __FREE_CHECKED__

document the TLS changes since 1.5.21

make the SETCOLOR() macro use bkgdset() on systems which have it, and attrset() when we don't.

closes #3005

remove m4_translit because it generates a warning with newer versions of autoconf when used in AC_INIT.  Instead, use tr -d to output the version string without a trailing newline.

restore ability for version.sh to cope with a repository directory with a space in it.

Use the directory where the version.sh script is located as the
directory where it is run.  Since this will be run in a separate shell
anyway, there's no need to worry about changing current directory.

This eliminates the need for the caller to specify where the source
directory is located, and doesn't require that the code take special
care to use the appropriate directory.  The fallback method of using
`cat VERSION` was broken in this respect, it would not work when run
from a different directory.
---
 configure.ac |    2 +-
 version.sh   |    7 ++++---
 2 files changed, 5 insertions(+), 4 deletions(-)

use [] around body of AC_CHECK_HEADER to avoid problems with older versions of
autoconf

avoid using m4_esyscmd_s() since it is not available in older versions of autoconf.  use m4_esyscmd() and m4_translit() to whack the newline instead.

see #3613

Uses SSLv23_client_method() in mutt_ssl_starttls() because TLSv1_2_client_method() does not honor the SSL_OP_NO_TLS* flags.

closes #3612

If neither --with-idn nor --without-idn are given, use -I/usr/include/idn when checking for the libidn header files, if present (needed for Solaris 11).

Don't bother checking for the idna_* functions if we didn't find -lidn

Fix VPATH-based generation of non-hg-mq based patchlists.

Note: Makefile.am runs patchlist.sh with $(srcdir)/PATCHES provided on stdin.

Fix VPATH build outside Mercurial repositories.

add version.sh and check_sec.sh to EXTRA_DIST so they are including in the "make dist" output.

fix forgotten RETSIGTYPE change id [18a724e9eba0]

remove deprecated check for signal() return type

update configure.ac with autoupdate

change AM_CONFIG_HEADER to AC_CONFIG_HEADERS and move it after the AM_INIT_AUTOMAKE.  config.h was not being generated from the dist package.

modernize intialization of configure.ac

Fix 'make distcheck': missing txt2c* and po/mutt.pot cleanup

remove redundant #include statements

remove extern decl for Charset_is_utf8 since menu.c includes mbyte.h which has the same decl.

backout out changeset 36b0cc717ecc

allow for CRLF line endings when comparing for the OpenPGP block identifiers

adapted from http://svnweb.freebsd.org/ports/head/mail/mutt/files/patch-pgp?revision=301308&view=markup

support nbsp in the pager.

imported from http://svnweb.freebsd.org/ports/head/mail/mutt/files/patch-nbsp?revision=301308&view=markup

detect newer BDB versions up to 5.2

imported from http://svnweb.freebsd.org/ports/head/mail/mutt/files/patch-bdb?revision=301308&view=markup

http://thread.gmane.org/gmane.mail.mutt.devel/15710

Interix doesn't have btowc, but it does have mbtowc.  Linux manpage also
discourages the use of btowc.

updated cs.po

use WACS_* macros for printing wide versions of the line drawing characters when linking against ncursesw.

This patch is based on http://prefix.gentooexperimental.org:8000/mutt-patches/file/fea851a2808f/solaris-ncurses-chars.patch

make configure define _POSIX_C_SOURCE, _XOPEN_SOURCE and _XOPEN_SOURCE_EXTENDED to comply with POSIX.1-2008 and XSI conforming applications.

update ru.po

remove duplicate const keyword

disable tls v1.1/1.2 in mutt_ssl_starttls() based on $ssl_use_tlsv1_*

see #3571

add support for TLSv1.1/1.2 when using GNUTLS.

see #3571

add variables for disabling TLSv1.1/1.2 when compiled against OpenSSL 1.0.1

see #3571

rename iswsp() to is_email_wsp()

add new inline funtion skip_email_wsp() to be used in lieu of SKIPWS() when parsing ASCII protocols rather than user input.

change use of SKIPWS() to skip_email_wsp() in places where it is likely to be a problem.

fix problem where mutt_substrdup() was called with the start
pointer > end pointer, as a result of using SKIPWS().  This
occurred because the header field had an empty body, so the
trailing newline was skipped over.  The fix is to just skip over
ascii space and htab.

closes #3609

if the peer's x.509 cert is not valid, we need to allow the user to inspect it even if the digital signatures are valid

closes #3506

updated fr.po

fix segfault in pop driver.  when rereading the message header, the ENVELOPE* struct is freed, but ctx->subj_hash retains a pointer to env->real_subj, so the hash table entry must be deleted prior to calling mutt_free_envelope()

http://pkgs.fedoraproject.org/cgit/mutt.git/tree/mutt-1.5.21-pophash.patch?h=f18

Fix NULL pointer dereference in notation name

closes #3574

use TLSv1.2 if available in the system OpenSSL when using starttls

use SSL_get_version() to display the actual SSL protocol used rather than the fixed string SSL.

Correct typos

update documentation for $pgp_getkeys_command to note that %r is replaced with the email address, not the public key ID

closes #1208

use new function iswsp() in place of isspace() when testing for whitespace.

closes #3608
closes #2956

make imap_private.h include message.h for IMAP_HEADER_DATA definition

cast &ctx->data to IMAP_HEADER_DATA** to fix compiler warning

imap_free_header_data should check for *data being NULL, rather than "data"

Make imap_free_header_data type-safe

Also push conditional into function for safety and consistency with
other free functions.

Fix VPATH build by running txt2c.sh from ${srcdir} rather than ".".

invoke gpgme_check_version() as required to initialize the library.

move locale code to initialization function.

closes #3300

additional documentation for $editor.  closes #3602

list-reply documentation should mention its behavior wrt the List-Post header field.

closes #3599

don't perform rfc3676 space stuffing when recalling a postponed message where it has already been done.

closes #3600

make ascii_strcasecmp properly handle unequal length strings with the same prefix

closes #3601

Add compiler and configure info to mutt -v output (closes #3537)

Makefile(.am) updated to produce conststrings.c, which contains
C strings representing:

* the compiler's own version information;
* the CFLAGS value from the Make environment
* the ./configure options

main.c is updated to print them when running 'mutt -v'.

txt2c.sh is added to produce conststrings.c. txt2c.sh uses a compiled
txt2c binary if possible, for complete fidelity to the source strings
in whatever encoding they may use.  If txt2c is not available (could
not be compiled, or was not compiled natively) it falls back on a shell
function to approximate the output using sed and tr.

mutt_buffer_init: just memset the buffer (closes #3588)

This was meant to be part of the previous commit.

Split mutt_buffer_new out of mutt_buffer_init.

Currently, no callers were providing a non-NULL buffer to
mutt_buffer_init, and splitting it will allow more sane semantics for
buffer allocation, initialization, and destruction in a later patch.