From: David Edmondson Date: Thu, 19 Jan 2012 09:34:07 +0000 (+0000) Subject: emacs: Make the part content available to `mm-inlinable-p'. X-Git-Tag: debian/0.12_rc1-1~153 X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=63342a3c06cca53720f8be2ae8c4cf2c66d137f1;p=notmuch.git emacs: Make the part content available to `mm-inlinable-p'. 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. --- diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el index e6a5b31f..c37479a5 100644 --- a/emacs/notmuch-show.el +++ b/emacs/notmuch-show.el @@ -429,14 +429,15 @@ current buffer, if possible." (with-temp-buffer (let* ((charset (plist-get part :content-charset)) (handle (mm-make-handle (current-buffer) `(,content-type (charset . ,charset))))) - (if (and (mm-inlinable-p handle) - (mm-inlined-p handle)) - (let ((content (notmuch-show-get-bodypart-content msg part nth))) - (insert content) - (set-buffer display-buffer) - (mm-display-part handle) - t) - nil))))) + ;; If the user wants the part inlined, insert the content and + ;; test whether we are able to inline it (which includes both + ;; capability and suitability tests). + (when (mm-inlined-p handle) + (insert (notmuch-show-get-bodypart-content msg part nth)) + (when (mm-inlinable-p handle) + (set-buffer display-buffer) + (mm-display-part handle) + t)))))) (defvar notmuch-show-multipart/alternative-discouraged '(