On Sun, 25 Mar 2012 00:43:28 +0000, Mark Walters <markwalters1009 at gmail.com> wrote: > The function notmuch-match-content-type was comparing content types > case sensitively. Fix it so it tests case insensitively. > > This fixes a bug where emacs would not include any body when replying > to a message with content-type TEXT/PLAIN. > --- > emacs/notmuch-lib.el | 5 +++-- > 1 files changed, 3 insertions(+), 2 deletions(-) > > diff --git a/emacs/notmuch-lib.el b/emacs/notmuch-lib.el > index c146748..a754de7 100644 > --- a/emacs/notmuch-lib.el > +++ b/emacs/notmuch-lib.el > @@ -185,8 +185,9 @@ the user hasn't set this variable with the old or new > value." > (st2 (notmuch-split-content-type t2))) > (if (or (string= (cadr st1) "*") > (string= (cadr st2) "*")) > - (string= (car st1) (car st2)) > - (string= t1 t2)))) > + ;; Comparison of content types should be case insensitive. > + (string= (downcase (car st1)) (downcase (car st2))) > + (string= (downcase t1) (downcase t2))))) > > (defvar notmuch-multipart/alternative-discouraged > '(
LGTM. jamie. -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 835 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120326/025634a8/attachment.pgp>
