Re: [PATCH] emacs: Make the part content available to `mm-inlinable-p'.

Subject: Re: [PATCH] emacs: Make the part content available to `mm-inlinable-p'.

Date: Thu, 19 Jan 2012 12:31:12 -0500

To: David Edmondson, notmuch@notmuchmail.org

Cc:

From: Aaron Ecay

On Thu, 19 Jan 2012 09:34:07 +0000, David Edmondson <dme@dme.org> wrote:
> The `mm-inlinable-p' function works better if it has access to the
> data of the relevant part, so load that content before calling it.
> 
> Don't load the content for parts that the user has indicated no desire
> to inline.
> 
> This fixes the display of attached image/jpeg parts, for example.

I had suggested the same approach in a message that only Dmitry got
(accursed new reply bindings!)  LGTM.

-- 
Aaron Ecay

• Previous message (by thread): Re: [PATCH] emacs: Make the part content available to `mm-inlinable-p'.

Thread:

• David Edmondson—[PATCH] emacs: Make the part content available to the mm-inline* checks. [inbox, notmuch::obsolete, notmuch::patch, unread]
  • David Edmondson—Re: [PATCH] emacs: Make the part content available to the mm-inline* checks. [inbox, signed, unread]
  • David Edmondson—[PATCH v2] emacs: Make the part content available to the mm-inline* checks. [inbox, notmuch::obsolete, notmuch::patch, unread]
    • Dmitry Kurochkin—Re: [PATCH v2] emacs: Make the part content available to the mm-inline* checks. [inbox, unread]
      • David Edmondson—Re: [PATCH v2] emacs: Make the part content available to the mm-inline* checks. [inbox, signed, unread]
        • Dmitry Kurochkin—Re: [PATCH v2] emacs: Make the part content available to the mm-inline* checks. [inbox, unread]
          • Austin Clements—Re: [PATCH v2] emacs: Make the part content available to the mm-inline* checks. [inbox, unread]
            • Dmitry Kurochkin—Re: [PATCH v2] emacs: Make the part content available to the mm-inline* checks. [inbox, unread]
            • Jameson Graef Rollins—Re: [PATCH v2] emacs: Make the part content available to the mm-inline* checks. [inbox, signed, unread]
              • David Edmondson—Re: [PATCH v2] emacs: Make the part content available to the mm-inline* checks. [inbox, signed, unread]
          • David Edmondson—Re: [PATCH v2] emacs: Make the part content available to the mm-inline* checks. [inbox, signed, unread]
            • Dmitry Kurochkin—Re: [PATCH v2] emacs: Make the part content available to the mm-inline* checks. [inbox, unread]
              • David Edmondson—[PATCH] emacs: Make the part content available to `mm-inlinable-p'. [inbox, notmuch::patch, notmuch::pushed, unread]
                • Dmitry Kurochkin—Re: [PATCH] emacs: Make the part content available to `mm-inlinable-p'. [inbox, notmuch::review, unread]
                • Aaron Ecay—Re: [PATCH] emacs: Make the part content available to `mm-inlinable-p'. [inbox, notmuch::review, unread]
                • Tomi Ollila—Re: [PATCH] emacs: Make the part content available to `mm-inlinable-p'. [inbox, notmuch::review, unread]
                • Jameson Graef Rollins—Re: [PATCH] emacs: Make the part content available to `mm-inlinable-p'. [inbox, signed, unread]
                  • David Edmondson—Re: [PATCH] emacs: Make the part content available to `mm-inlinable-p'. [inbox, signed, unread]
                • David Bremner—Re: [PATCH] emacs: Make the part content available to `mm-inlinable-p'. [inbox, unread]