Re: [PATCH v2] emacs: Make the part content available to the mm-inline* checks.

Subject: Re: [PATCH v2] emacs: Make the part content available to the mm-inline* checks.

Date: Mon, 23 Jan 2012 08:00:00 +0000

To: Jameson Graef Rollins, Austin Clements, Dmitry Kurochkin

Cc: notmuch@notmuchmail.org

From: David Edmondson

On Fri, 20 Jan 2012 12:13:49 -0800, Jameson Graef Rollins <jrollins@finestructure.net> wrote:
> On Wed, 18 Jan 2012 14:35:01 -0500, Austin Clements <amdragon@MIT.EDU> wrote:
> > Shouldn't we only be doing this for parts with inline (or not
> > attachment) content-disposition?  That's cheap to check.  Or do we
> > actually want things like image attachments to get inlined, despite
> > their disposition?
> 
> This is a good question, actually.  Should we just always ignore the
> disposition, and inline stuff if it's inlinable?  Should this be
> configurable?

We shouldn't inline things unless the content disposition is 'inline'.

I'll work on a fix for this, which will have nothing to do with whether
or not things are indexed.

• 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]