2019-09-21 12:27:57 -0700  Kevin McCarthy  <kevin@8t8.us> (6931c1ad)

        * Update UPDATING file for 1.12.2 release.

M	UPDATING

2019-09-09 18:32:59 -0700  Kevin McCarthy  <kevin@8t8.us> (c65c12be)

        * Fix mutt_write_mime_body() application/pgp-encrypted handling.
        
        It would blindly overwrite any attachment with that type, as if it
        were the version label part of a multipart/encrypted message.
        
        However attachments with extenstion .gpg are labeled that type if
        $mime_type_query_command is set to "xdg-mime query filetype".  This
        would cause the attachment to be overwritten too.
        
        Add a check for a missing body->filename, which should only be the
        case for the manually constructed part.

M	sendlib.c

2019-09-01 16:58:46 +0200  Gero Treuner  <gero@70t.de> (472324f8)

        * add fallback for inotify_init1

M	configure.ac
M	monitor.c
M	monitor.h

2019-08-28 14:41:26 -0700  Kevin McCarthy  <kevin@8t8.us> (4c2f7c70)

        * Fix inotify configure test.
        
        AC_CHECK_FUNCS executes "action-if-found" or "action-if-not-found" for
        each function in the list.  If a system has some but not all of the
        functions, both will end up being executed.
        
        This caused a build failure on an older system with exactly that
        situation.
        
        Change to only define USE_INOTIFY if all functions exist.

M	configure.ac

2019-07-25 18:18:08 -0700  Kevin McCarthy  <kevin@8t8.us> (588f8d69)

        * Fix accidental fall-through for <quote-char> if aborted.
        
        If <quote-char> was aborted, it was falling through to the
        <transpose-chars> function.
        
        This problem was introduced 19 years when the <transpose-chars> case
        was moved below <quote-char>.  Previously it fell through to the
        default case which beeped.

M	enter.c

2019-06-28 12:57:24 -0700  Kevin McCarthy  <kevin@8t8.us> (f3ec740d)

        * Don't read or save history if $history_file isn't set.

M	history.c

2019-06-22 14:57:36 -0700  Kevin McCarthy  <kevin@8t8.us> (840b813f)

        * Make sure mailcap test %s is sanitized.
        
        It's not clear to me if %s is allowed as part of a test field.
        However since we are passing the attachment filename, we should
        sanitize it first.

M	rfc1524.c

2019-06-22 12:35:58 -0700  Kevin McCarthy  <kevin@8t8.us> (a99a75b7)

        * Fix compose and edit attachment symlink failure code.
        
        In the case where safe_symlink() fail, mutt prompts to continue, but
        did not properly reset the filename to be operated on.
        
        Fix up mutt_view_attachment() to use the same flow as the others, to
        allow for easier comparison.

M	attach.c

2019-06-22 10:09:02 -0700  Kevin McCarthy  <kevin@8t8.us> (2fd320a8)

        * Fix send-mode printing when expand_filename() returns 1.
        
        It will return this when a nametemplate entry already matched the
        passed in filename.  So this bug required both a print entry and an
        already matching nametemplate entry to trigger.

M	attach.c

2019-06-15 09:12:15 -0700  Kevin McCarthy  <kevin@8t8.us> (38e7c257)

        * automatic post-release commit for mutt-1.12.1

M	ChangeLog
M	VERSION
2019-06-15 09:02:36 -0700  Kevin McCarthy  <kevin@8t8.us> (e299de25)

        * Update UPDATING file for 1.12.1 release.

M	UPDATING

2019-06-11 13:15:22 -0700  Kevin McCarthy  <kevin@8t8.us> (dc662474)

        * Add $fcc_before_send, defaulting unset.
        
        When set, the message will be Fcc'ed the same as sent.  $fcc_clear and
        $fcc_attach will be ignored.  This is because of the difficulty of
        unwinding changes, notably Protected Headers, without potentially
        breaking signatures.

M	doc/manual.xml.head
M	init.h
M	mutt.h
M	send.c

2019-06-13 17:56:38 -0700  Kevin McCarthy  <kevin@8t8.us> (81fb9dd8)

        * Improve $reverse_realname documentation.
        
        Make it clear that, even if set, a missing realname part of the
        matching address will be filled in by $realname.

M	init.h

2019-06-09 11:07:16 -0700  Kevin McCarthy  <kevin@8t8.us> (0727d324)

        * Mention sources for ~p and ~P patterns.
        
        The manual only mentioned alternates.  Add $from and local
        account/hostname to the list.
        
        The man page implied the only source was alternates.  Change to the
        same wording as the manual.
        
        Thanks to @rear1019 for pointing out the misleading man page.

M	doc/manual.xml.head
M	doc/muttrc.man.head

2019-06-07 13:43:36 -0700  Kevin McCarthy  <kevin@8t8.us> (6f5e6bbb)

        * Improve imap_append_message() error message handling.
        
        If the rc is IMAP_CMD_BAD, then either idata->buf is stale or an error
        message has already been printed (in cmd_handle_untagged()).
        
        Use imap_next_word() to skip over the next two words instead of
        directly skipping over SEQLEN, in case the buffer is in a different
        format.  We don't want to jump over the end of string.
        
        Skip the mutt_error() if there is nothing to print.

M	imap/message.c

2019-06-06 16:03:42 -0700  Kevin McCarthy  <kevin@8t8.us> (dd327606)

        * Enable the idata->check_status using bit operations.
        
        Commit e3f66d7e fixed dropped new mail notications, removing the
        unsetting of idata->reopen IMAP_NEWMAIL_PENDING in imap_cmd_finish()
        when an EXPUNGE was processed.
        
        However, imap_cmd_finish() can be called twice by
        imap_check_mailbox().  First as part of the imap_exec(), and manually
        again just below.
        
        Now that the IMAP_NEWMAIL_PENDING still exists, a second call could
        overwrite idata->check_status if both reopen flags were set.
        
        This unfortunately affects update_index(), which behaves differently
        for MUTT_REOPENED.
        
        I need to change the return value of mx_check_mailbox() in master to
        preserve all the bits, so the index can both notify of new mail and
        update_index() properly.
        
        For stable, the best fix is to use bit operators to enable the
        check_status flags in imap_cmd_finish() (and cmd_parse_fetch for
        flags), and keep the imap_check_mailbox() priority of setting its
        return value (it prioritizes IMAP_EXCHANGE_PENDING).

M	imap/command.c

2019-06-06 13:38:03 -0700  Kevin McCarthy  <kevin@8t8.us> (4d95b2cf)

        * Improve robustness of imap_append_message().
        
        First, check the imap_cmd_step() return value instead of looking at
        idata->buf for "OK".  If the connection bombed and imap_cmd_step()
        returned IMAP_CMD_BAD, the value of idata->buf is stale.
        
        If the server returned "+ OK" for the command continuation request
        response, the call to imap_code(idata->buf) would even end up
        returning true, despite that the append failed!  (See #110, although
        at the time of commit I can only hypothesize this is what is
        happening.)
        
        Second, check the status of the writes.  flush_buffer() was not
        passing the rc from mutt_socket_write_n(), which was further making
        the above disaster scenerio possible.

M	imap/message.c

2019-06-05 14:04:55 -0700  Kevin McCarthy  <kevin@8t8.us> (e3f66d7e)

        * Fix dropped new mail notifications when an EXPUNGE_PENDING is set.
        
        Prior to the fetch_headers rework and introduction of
        idata->max_msn (starting around e0376c75), cmd_handle_untagged() was
        looking directly at ctx->msgcount, which isn't fixed up until
        imap_expunge_mailbox().  At that time, more care had to be taken
        inbetween handling the EXPUNGE message and the actual expunge of the
        mailbox because of the discrepency between server state and mailbox
        context state.
        
        idata->max_msn is now decremented during the processing of EXPUNGE and
        VANISHED notices from the server, so reflect "current" state.  So,
        when we receive an EXISTS notice, we no longer need the checks for
        expunge state and can always set the NEWMAIL_PENDING flag.
        
        Additionally, fix imap_cmd_finish() to retain the IMAP_NEWMAIL_PENDING
        flag after handling an expunge.  The expunge does not grab new
        messages so dropping the flag would cause mutt to forget the new mail
        status until another EXISTS command.
        
        Since this is a stable branch fix, I'm leaving the either/or
        processing of expunge versus new mail in imap_cmd_finish().  However,
        I don't see why this has to be done in two calls.  I may rework that
        in master to process an expunge and then the new mail one after the
        other.

M	imap/command.c

2019-05-31 09:37:38 -0700  Kevin McCarthy  <kevin@8t8.us> (22c6df82)

        * Remove FORCE prerequisite on version.h.
        
        This is causing package build issues on OpenBSD because of a
        difference in make behavior.
        
        In GNU make, it always invoke the 'version.h' target, but then checks
        to see if version.h changed before retriggering targets with that
        prerequisite.  On OpenBSD, invoking the version.h target always
        retriggers targets with that prerequisite.
        
        This means that updating the version information now requires a 'make
        clean' or manual removal of version.h.

M	Makefile.am

2019-05-30 09:27:15 -0700  Kevin McCarthy  <kevin@8t8.us> (dd1601a1)

        * Minor documentation correction.

M	doc/manual.xml.head

2019-05-25 14:26:08 -0700  Kevin McCarthy  <kevin@8t8.us> (8ae8394f)

        * Disable state messages for attachments when forwarding.
        
        First, option(OPTVIEWATTACH) is only set for (s->flags & DISPLAY), so that
        check can be ignored.
        
        With that removed, all this is doing is printing
          [-- This is an attachment ...]
        with no context in the middle of non-display rendering for the case
        $honor_disposition is set with a handler.
        
        This makes no sense and I believe is a logic error.  Perhaps this
        never took effect, but with $forward_attachments it now affects
        forwarding in that case.

M	handler.c

2019-05-25 13:12:04 -0700  Kevin McCarthy  <kevin@8t8.us> (b00a06e5)

        * Have $forward_attachments look at attachment disposition.
        
        If $honor_disposition is set, an "attachment" disposition decodable
        attachment won't be included in the body, so we want it to be added as
        an attachment to the forwarded email.

M	send.c

2019-05-25 09:18:22 -0700  Kevin McCarthy  <kevin@8t8.us> (ee9983f0)

        * automatic post-release commit for mutt-1.12.0

M	ChangeLog
M	VERSION
M	po/bg.po
M	po/ca.po
M	po/cs.po
M	po/da.po
M	po/de.po
M	po/el.po
M	po/eo.po
M	po/es.po
M	po/et.po
M	po/eu.po
M	po/fi.po
M	po/fr.po
M	po/ga.po
M	po/gl.po
M	po/hu.po
M	po/id.po
M	po/it.po
M	po/ja.po
M	po/ko.po
M	po/lt.po
M	po/nl.po
M	po/pl.po
M	po/pt_BR.po
M	po/ru.po
M	po/sk.po
M	po/sv.po
M	po/tr.po
M	po/uk.po
M	po/zh_CN.po
M	po/zh_TW.po
2019-05-25 09:08:33 -0700  Kevin McCarthy  <kevin@8t8.us> (7d8db3e2)

        * Set UPDATING file release date for 1.12.0.

M	UPDATING

2019-05-20 12:32:37 -0700  Kevin McCarthy  <kevin@8t8.us> (8f739fbe)

        * Make mutt.texi and mutt.info generation continue on failure.
        
        Add warning messages, noting the appropriate packages to install.
        
        Note: mutt.texi won't normally be built unless configure finds one of
        the possible expected conversion program names.  Unfortunately, the
        canonical name, "docbook2texi", is used by both the docbook2x and
        docbook-utils projects.
        
        So on Arch, the canonical name is what we need, while on Debian and
        Red Hat/Fedora it isn't.  We search for the alternative names first,
        but it's still possible to invoke the wrong program if the needed
        package is not installed.

M	doc/Makefile.am

2019-05-20 06:59:18 -0700  TAKAHASHI Tamotsu  <ttakah@lapis.plala.or.jp> (be42d1a0)

        * Updated Japanese translation.

M	po/ja.po

2019-05-19 14:40:08 -0700  Kevin McCarthy  <kevin@8t8.us> (0e78280a)

        * Adjust docbook2texi program search.
        
        Look for db2x_docbook2texi first, because Fedora/Red Hat has their own
        version of "docbook2texi": db2x_docbook2texi.  I'm not sure what to do
        if they have don't have that installed but have docbook2texi
        installed, since this is a terrible name collision.
        
        Thanks to Moritz Barsnick for pointing out the different binary (and
        package) names on Fedora.

M	configure.ac

2019-05-19 10:14:09 -0700  Kevin McCarthy  <kevin@8t8.us> (7bef99de)

        * Update gpgme and gpg-error automake checks.
        
        Pull updated autoconf files from the GPGME 1.13.0 release and use
        those new macros.
        
        Add a call to AM_PATH_GPG_ERROR() and include $(GPG_ERROR_LIBS) in the
        libraries.
        
        Thanks to Eike Rathke for finding the build problem, and for his
        patch fixing the issue.  I opted for just grabbing the newest autoconf
        files from gpgme instead, but his fixed worked great too.

M	Makefile.am
M	configure.ac
A	m4/gpg-error.m4
M	m4/gpgme.m4

2019-05-15 09:09:25 -0700  Ivan Vilata i Balaguer  <ivan@selidor.net> (cc8a201f)

        * Updated Catalan translation.

M	po/ca.po

2019-05-15 13:05:09 +0200  Vincent Lefevre  <vincent@vinc17.net> (3b6f6b82)

        * Avoid undefined behavior on huge integer in a RFC 2231 header.
        
        The atoi() function was called on the index, which can potentially
        be huge in an invalid message and can yield undefined behavior. The
        mutt_atoi() function is now used for error detection.

M	rfc2231.c

2019-05-14 09:26:38 -0700  Kevin McCarthy  <kevin@8t8.us> (e8d057a9)

        * Add a "backticks in double quotes" example to the manual.

M	doc/manual.xml.head

2019-05-14 09:00:46 -0700  Kevin McCarthy  <kevin@8t8.us> (14bc2fc0)

        * Fix sample muttrc to use better quoting practices.
        
        Use single quotes in the password encryption example.
        
        For password decryption, put the backquotes inside double quotes to
        avoid special characters being re-interpreted.

M	contrib/sample.muttrc-starter

2019-05-13 12:53:58 +0200  Vincent Lefevre  <vincent@vinc17.net> (79563636)

        * fr.po: updated Project-Id-Version for the 1.12 release

M	po/fr.po

2019-05-12 13:59:16 -0700  Petr Pisar  <petr.pisar@atlas.cz> (f3eb92f0)

        * Updated Czech translation.

M	po/cs.po

2019-05-11 06:45:10 -0700  lilydjwg  <lilydjwg@gmail.com> (74447017)

        * Updated Simplified Chinese translation.

M	po/zh_CN.po

2019-05-10 13:18:46 -0700  Morten Bo Johansen  <mbj@mbjnet.dk> (831ea9c5)

        * Updated Danish translation.

M	po/da.po

2019-05-10 11:09:45 -0700  Vsevolod Volkov  <vvv@mutt.org.ua> (7e713495)

        * Updated Russian translation

M	po/ru.po

2019-05-10 11:07:41 -0700  Vsevolod Volkov  <vvv@mutt.org.ua> (33eeaeec)

        * Updated Ukrainian translation

M	po/uk.po

2019-05-10 14:03:48 +0000  Olaf Hering  <olaf@aepfle.de> (ce6e1db0)

        * Update de.po
        
        Unwrap lines, because it is 2019.
        
        Signed-off-by: Olaf Hering <olaf@aepfle.de>

M	po/de.po

2019-05-09 19:21:34 -0700  Kevin McCarthy  <kevin@8t8.us> (690c2945)

        * Check for NULL addresses in mutt_addrcmp().
        
        While looking into ticket #140, I noticed default_to() could pass a
        NULL env->from to mutt_addrcmp() if $reply_self is set and there is a
        reply_to address.

M	send.c

2019-05-09 13:48:44 -0700  Kevin McCarthy  <kevin@8t8.us> (f5687827)

        * Add note about %r for the pgp_list_*_command vars.
        
        In this case %r is the list of search strings.

M	init.h

2019-05-09 13:25:25 -0700  Kevin McCarthy  <kevin@8t8.us> (471fc11a)

        * Change sample muttrc path to match other manual references.
        
        The other parts of the manual refer to /usr/local/share/doc in their
        examples.  For consistency, change the starter muttrc and manual entry
        to do the same.
        
        In the future, we may wish to have a config variable instead, so users
        don't have to worry about system vs locally installed paths when
        sourcing the gpg.rc, for instance.

M	contrib/sample.muttrc-starter
M	doc/manual.xml.head

2019-05-08 19:28:21 -0700  Kevin McCarthy  <kevin@8t8.us> (80eac184)

        * Add sample starter muttrc to contrib.
        
        Add a mini-section with a link to the manual instead, under
        "Configuration".

M	contrib/Makefile.am
A	contrib/sample.muttrc-starter
M	doc/manual.xml.head

2019-05-07 15:59:40 -0700  Kevin McCarthy  <kevin@8t8.us> (988e06b2)

        * Fix typo in manual.

M	doc/manual.xml.head

2019-05-06 15:35:38 -0700  Kevin McCarthy  <kevin@8t8.us> (089d7cee)

        * Add a quick-starter config section to the manual.

M	doc/manual.xml.head

2019-05-06 12:37:04 -0700  Kevin McCarthy  <kevin@8t8.us> (2f8bc797)

        * Add a test for NULL parameter attribute too.
        
        Just to be sure we don't have any issues with the new continuation
        code.  The mutt_set_parameter() code doesn't actively prevent a null
        attribute.

M	sendlib.c

2019-05-05 13:26:59 -0700  Kevin McCarthy  <kevin@8t8.us> (a9cbf67c)

        * Add a manual section on OpenPGP and S/MIME configuration.
        
        Talk about the required config variables, GPGME and classic mode,
        agents, pinentry programs, and smime_keys.

M	doc/manual.xml.head

2019-05-02 12:31:29 -0700  Kevin McCarthy  <kevin@8t8.us> (6898c083)

        * Write rfc2231 parameter continuations for long parameters.
        
        Previously, Mutt would truncate long attachment filenames, to avoid
        writing an illegal length header line.  This commit is a followup to
        4dcb3ba1, where I reverted an incorrect fix for the problem.
        
        rfc2231_encode_string() now returns a list of continuations, with
        encoding and continuation number suffixes already appended to the
        attribute.  The function tries to keep the line length less than 78
        characters, but the code is a bit imprecise as a trade off for
        simplicity and readability.
        
        Modify mutt_write_mime_header() to loop through the continuations.

M	rfc2231.c
M	rfc2231.h
M	sendlib.c

2019-05-02 15:44:16 +0200  Vincent Lefevre  <vincent@vinc17.net> (861a7e47)

        * Missing word in UPDATING file.

M	UPDATING

2019-05-01 15:23:04 -0700  Kevin McCarthy  <kevin@8t8.us> (629e6e36)

        * Minor UPDATING file fix.

M	UPDATING

2019-05-01 06:52:37 -0700  Kevin McCarthy  <kevin@8t8.us> (01f1832f)

        * UPDATING file cleanups.
        
        Second quick pass.

M	UPDATING

2019-04-30 20:04:12 -0700  Kevin McCarthy  <kevin@8t8.us> (aa2985a3)

        * Update UPDATING file.
        
        This is a first pass to get something out quickly to testers.

M	UPDATING

2019-04-29 13:03:44 +0200  Vincent Lefevre  <vincent@vinc17.net> (98b3493a)

        * Updated French translation.

M	po/fr.po

2019-04-29 12:49:22 +0200  Vincent Lefevre  <vincent@vinc17.net> (c8eeb0d9)

        * Corrected minor errors in text related to $forward_attachments.

M	doc/manual.xml.head
M	send.c

2019-04-28 15:48:33 -0700  Kevin McCarthy  <kevin@8t8.us> (92d7c484)

        * Fix mailbox search to not recompile for a repeated search.
        
        The LastSearchExpn was being compared, but was never set to the
        expanded search value.  This was causing the search to be recompiled
        even if it were for the same previous expanded search string.

M	pattern.c

2019-04-28 14:59:13 -0700  Kevin McCarthy  <kevin@8t8.us> (9008e23b)

        * Note the other parts of generated message-ids in the manual.

M	doc/manual.xml.head

2019-04-25 19:41:04 -0700  Kevin McCarthy  <kevin@8t8.us> (24965a7d)

        * Add $forward_attachments quadoption for inline-forward mode.
        
        When set or answered yes, non text-decodable parts will be attached to
        the new message.  The default value is "ask-yes", because I believe
        this is something people will want to use, and should be made aware of
        the new possible behavior.
        
        The option presents a nice middle ground between previous
        inline-forwarding behavior (where all the non-text parts were
        dropped), and $mime_forward where the entire email was included as an
        attachment.
        
        This was previously difficult to achieve, but after the
        recv-attachment refactoring: (a19e5266^..faabd621)
        it became possible to use the ATTACH_CONTEXT and the recvattach.c
        helper mutt_generate_recvattach_list() to properly deal with nesting
        and multiple file-handles.

M	attach.h
M	doc/manual.xml.head
M	init.h
M	mutt.h
M	recvattach.c
M	send.c

2019-04-25 09:26:43 -0700  Kevin McCarthy  <kevin@8t8.us> (f706c235)

        * Add note about IMAP browser and trailing delimiters.
        
        This is the same behavior as the regular browser, but I think still
        deserves a quick mention.
        
        Thanks to Charles for pointing out this section of the manual and
        suggesting the addition.

M	doc/manual.xml.head

2019-04-23 19:06:11 -0700  Kevin McCarthy  <kevin@8t8.us> (6e90579f)

        * Add RECURSIVEMATCH LIST-EXTENDED selection option to query.
        
        LSUB is required to include mailboxes with subscribed children, but
        the just added "LIST (SUBSCRIBED)" by default does not do that.
        
        To match previous behavior, add RECURSIVEMATCH to make sure the
        children are included.  Fix the parser to trim off the CHILDINFO
        suffix in the response.
        
        Parse \NonExistent LIST attribute the same as \NoSelect.

M	imap/browse.c
M	imap/command.c

2019-04-23 22:15:31 +0100  Jeremy Sowden  <jeremy@azazel.net> (e1bc4339)

        * Enable the use of toggle-write from the pager.
        
        Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

M	curs_main.c
M	functions.h

2019-04-24 03:04:05 +1000  Naveen Nathan  <mutt@t.lastninja.net> (82b759e8)

        * Improve hierarchy information accuracy in IMAP browser
        
        Currently the IMAP browser relies on LIST and LSUB (for listing subscribed
        folders) which may not provide the required hierarchy information.
        
        RFC3348 section 3 goes as far as stating that a client mustn't rely on LSUB
        for hierarchy information.
        
        This patch implements the LIST command extensions specified in RFC5258
        requiring that a server must respond with hierarchy information for
        listed folders (whether or not filtering on subscribed folders).

M	imap/browse.c
M	imap/command.c
M	imap/imap_private.h

2018-08-19 09:25:53 -0700  Kevin McCarthy  <kevin@8t8.us> (ba55d5df)

        * Add $imap_fetch_chunk_size to allow FETCHing new headers in chunks.
        
        For extremely large mailboxes, some implementations will time out just
        while fetching the new headers, because the client doesn't send any
        commands for 30 minutes while downloading the large number of headers.
        
        Rewrite imap_fetch_msn_seqset() to return chunks of size
        $imap_fetch_chunk_size.
        
        The change requires trusting the server will follow the RFC and not
        send an EXPUNGE during or between the FETCH chunks; otherwise we'll
        miss MSNs between the chunks because the shift.
        
        We could in theory continue to set "msn_begin = idata->max_msn + 1",
        but that makes the assumption there are no holes in the header cache
        that we are filling in during a chunk.  Personally I am dubious about
        "header cache holes", but the IMAP code has explicitly mentioned and
        handled them since prior to my involvement.
        
        Since the RFC forbids the interleaving EXPUNGE I believe it's safe
        enough to set "msn_begin = fetch_msn_end + 1" until proven otherwise.

M	globals.h
M	imap/message.c
M	init.h

2019-04-22 11:02:28 -0700  Kevin McCarthy  <kevin@8t8.us> (905c1fff)

        * Add note about $mailcap_path generation to the documentation.

M	init.h

2019-04-21 20:05:13 -0700  Kevin McCarthy  <kevin@8t8.us> (6975aad8)

        * Convert mutt_parse_adrlist() to use BUFFER for simple parsing.

M	parse.c

2019-04-21 19:45:49 -0700  Kevin McCarthy  <kevin@8t8.us> (caeed0c5)

        * Change mutt_parse_mailboxes() to use the path BUFFER argument.
        
        Instead of copying to a buf[] variable, since the various utility
        methods are now available to BUFFER too.

M	buffy.c

2019-04-18 16:30:38 +0100  Julian Gilbey  <julian-git@d-and-j.net> (9351dd01)

        * Handle the same secret key appearing in multiple public keyrings
        
        Allow for S/MIME keys and the possibility of missing subkeys

M	crypt-gpgme.c

2019-04-21 11:00:23 -0700  Kevin McCarthy  <kevin@8t8.us> (9da6bde1)

        * Fix unistring library configure test.
        
        The "action-if-found" argument of AC_SEARCH_LIBS is run even if the
        result is "none required" (i.e. the test function is already in LIBS,
        in this case -lidn2).
        
        This was causing "-lunistring" to be appended on a system without the
        library installed, generating a build error.  Fix the test to not
        append the library for the "none required" case.
        
        Thanks to Fabrice Fontaine for reporting this issue and helping me
        test the fix.

M	configure.ac

2019-04-20 15:43:59 -0700  Kevin McCarthy  <kevin@8t8.us> (4dcb3ba1)

        * Revert "Fix truncation of long filenames in attachments."
        
        This reverts commit f476d0aecd6f88db5291427fced21a8e834ca181.
        
        The commit allowed the generation of illegal length header lines.  A
        correct fix needs to implement parameter continuations.
        
        I will work on that, but in case I don't get it in before 1.12,
        preserve the hard truncation.

M	sendlib.c

2019-04-20 13:00:28 -0700  Kevin McCarthy  <kevin@8t8.us> (9865e90e)

        * Convert remailer to use BUFFER for Mixmaster invocation.

M	remailer.c

2019-04-20 12:34:35 -0700  Kevin McCarthy  <kevin@8t8.us> (20bedeac)

        * Convert compress to use mutt_buffer_quote_filename().
        
        Because the compress expandos operate differently than the rest of
        mutt, requiring manual outer quoting, add a parameter to the function
        to toggle outer quoting off.
        
        Remove the now unused escape_path() function.

M	compress.c
M	muttlib.c
M	protos.h

2019-04-20 11:11:07 -0700  Kevin McCarthy  <kevin@8t8.us> (bc776fad)

        * Update po headers.
        
        Add missing Language header.
        
        Make the Project-Id-Version header consistent.

M	po/bg.po
M	po/ca.po
M	po/cs.po
M	po/de.po
M	po/el.po
M	po/es.po
M	po/et.po
M	po/eu.po
M	po/fi.po
M	po/ga.po
M	po/hu.po
M	po/id.po
M	po/it.po
M	po/ja.po
M	po/ko.po
M	po/lt.po
M	po/pl.po
M	po/pt_BR.po
M	po/ru.po
M	po/sk.po
M	po/tr.po
M	po/uk.po
M	po/zh_CN.po
M	po/zh_TW.po

2019-04-19 16:06:43 -0700  Kevin McCarthy  <kevin@8t8.us> (3eb82b30)

        * Increase mutt_pgpring path size to silence warning.
        
        This is an independent binary, and doesn't have access to the BUFFER
        functions.  The binary is used by very few (if any) people at this
        point, so just make the minimum fix to silence the warning.

M	pgppubring.c

2019-04-19 15:04:43 -0700  Kevin McCarthy  <kevin@8t8.us> (44037a53)

        * Convert pager help string to use BUFFER.
        
        Remove the awkward string truncation warnings, and remove helpstr and
        tmphelp char arrays from the stack.
        
        Because the pager is fairly long lived, allocate the helpstr instead
        of using the pool.

M	pager.c

2019-04-19 14:25:05 -0700  Kevin McCarthy  <kevin@8t8.us> (14a68f8f)

        * Convert certficate prompts to use menu dialog helper and BUFFER.

M	mutt_ssl.c
M	mutt_ssl_gnutls.c

2019-04-19 13:14:54 -0700  Kevin McCarthy  <kevin@8t8.us> (04257603)

        * Add menu dialog helper to add rows.
        
        Remove the manual max calculation and dialog row allocation.
        
        Add a NONULL check because the helper uses safe_strdup() to add a row.

M	menu.c
M	mutt_menu.h

2019-04-18 18:42:57 -0700  Kevin McCarthy  <kevin@8t8.us> (16818f75)

        * Convert migrated lib.c functions to use BUFFER.
        
        Fix mutt_getcwd() to return NULL on failure.
        
        Change mutt_rmtree(), mutt_mkwrapdir(), safe_open(), and
        safe_symlink() to use BUFFER so they don't have filename length
        limitations.

M	muttlib.c
M	protos.h

2019-04-18 15:35:22 -0700  Kevin McCarthy  <kevin@8t8.us> (92a5f7bb)

        * Relocate lib.c functions to muttlib.c to enable BUFFER use.
        
        Relocate safe_symlink(), safe_open(), mutt_rmtree() and their
        dependent functions.
        
        This rearrangement is a bit awkward.  Another approach for the future
        might be to convert lib.c into a muttlib.c aware file, and just copy
        the functions pgppubring.c uses inside itself.

M	lib.c
M	lib.h
M	muttlib.c
M	protos.h

2019-04-17 16:07:05 -0700  Kevin McCarthy  <kevin@8t8.us> (098272e0)

        * Increase prompt size to remove warning.

M	commands.c

2019-04-16 20:22:08 -0700  Kevin McCarthy  <kevin@8t8.us> (3c6bdc90)

        * Convert change folder operations to use BUFFER.
        
        Store the folder name inside a BUFFER and use the various BUFFER
        enhanced functions.

M	curs_main.c

2019-04-16 19:25:36 -0700  Kevin McCarthy  <kevin@8t8.us> (70ef4e9e)

        * Add mutt_buffer_enter_fname().

M	curs_lib.c
M	protos.h

2019-04-16 18:11:35 -0700  Kevin McCarthy  <kevin@8t8.us> (940bdaa1)

        * Convert other users of BUFFY->pathbuf to use BUFFERS.
        
        A few functions in browser.c, buffy.c, and monitor.c were using
        BUFFY->pathbuf but were potentially truncating via fixed size buffers.
        Convert those to use BUFFERS too.
        
        buffy_get() was creating epath and expanding it, apparently to match
        against expanded BUFFY list entries, was wasn't using the epath.  I
        believe this is a bug, and have switched the comparison to epath.

M	browser.c
M	buffy.c
M	monitor.c

2019-04-16 13:41:10 -0700  Kevin McCarthy  <kevin@8t8.us> (797d16a3)

        * Convert BUFFER->path to a BUFFER.
        
        Rename to BUFFER->pathbuf to make it clear the field is a BUFFER, and
        to make sure to catch and review all usages.
        
        There are still uses of pathbuf that are truncating and need to be
        fixed, for example in browser.c and buffy.c.
        
        Fix up sidebar usage in one place, that was pointing inside the
        BUFFY->path with a char *.  At the same time, change their "short
        folder" computation to also use a BUFFER.

M	browser.c
M	buffy.c
M	buffy.h
M	imap/browse.c
M	imap/command.c
M	imap/imap.c
M	mh.c
M	sidebar.c

2019-04-16 12:23:42 -0700  Kevin McCarthy  <kevin@8t8.us> (6317a303)

        * Change BUFFY->realpath to be const char *.
        
        BUFFY->path is a fixed array (which will be converted to a BUFFER in
        the next commit).  This is needed to call mutt_expand_path().
        
        However, BUFFY->realpath has no such need, and so it is a bit
        wasteful (not to mention not big enough) to store as such.

M	buffy.c
M	buffy.h

2019-04-15 18:43:49 -0700  Kevin McCarthy  <kevin@8t8.us> (4084bda4)

        * Add a note about $prompt_after to the $pager documentation.

M	init.h

2019-04-14 15:38:15 -0700  Kevin McCarthy  <kevin@8t8.us> (e8c8d559)

        * Rename mutt_buffer_rfc1524_expand_command()
        
        Remove buffer prefix now that all callers are converted.

M	attach.c
