On Sat, Nov 07 2015, Mark Walters wrote: > From: David Edmondson <dme@dme.org> > > No code uses the 'no-buttons argument to > `notmuch-show-insert-bodypart', so remove it. Looks fine. > --- > emacs/notmuch-show.el | 7 ++----- > 1 file changed, 2 insertions(+), 5 deletions(-) > > diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el > index 9fc79e0..f8184e2 100644 > --- a/emacs/notmuch-show.el > +++ b/emacs/notmuch-show.el > @@ -956,9 +956,7 @@ (defun notmuch-show-insert-bodypart (msg part depth &optional hide) > > HIDE determines whether to show or hide the part and the button > as follows: If HIDE is nil, show the part and the button. If HIDE > -is t, hide the part initially and show the button. If HIDE is > -'no-buttons, show the part but do not add any buttons (this is > -useful for quoting in replies)." > +is t, hide the part initially and show the button." > > (let* ((content-type (downcase (plist-get part :content-type))) > (mime-type (notmuch-show-mime-type part)) > @@ -969,8 +967,7 @@ (defun notmuch-show-insert-bodypart (msg part depth &optional hide) > (beg (point)) > ;; We omit the part button for the first (or only) part if > ;; this is text/plain, or HIDE is 'no-buttons. > - (button (when (and (not (equal hide 'no-buttons)) > - (notmuch-show-insert-header-p part hide)) > + (button (when (notmuch-show-insert-header-p part hide) > (notmuch-show-insert-part-header nth mime-type content-type (plist-get part :filename)))) > ;; Hide the part initially if HIDE is t, or if it is too long > ;; and we have a button to allow toggling (thus reply which > -- > 2.1.4