Adam, One comment below.
On Sun, 8 Jan 2012 00:52:42 -0700, Adam Wolfe Gordon <awg+notmuch at xvx.ca> wrote: > From: Adam Wolfe Gordon <awg at xvx.ca> > > Using the new JSON reply format allows emacs to quote HTML parts > nicely by first parsing them with w3m, then quoting them. This is > very useful for users who regularly receive HTML-only email. > > The behavior for messages that contain plain text parts should be > unchanged, except that an additional quoted line is added to the end > of the reply message. The test has been updated to reflect this. > --- > emacs/notmuch-mua.el | 62 +++++++++++++++++++++++++++++++++++++++---------- > test/emacs | 1 + > 2 files changed, 50 insertions(+), 13 deletions(-) > > diff --git a/emacs/notmuch-mua.el b/emacs/notmuch-mua.el > index 7114e48..7f894cb 100644 > --- a/emacs/notmuch-mua.el > +++ b/emacs/notmuch-mua.el > @@ -19,6 +19,7 @@ > ;; > ;; Authors: David Edmondson <dme at dme.org> > > +(require 'json) > (require 'message) > > (require 'notmuch-lib) > @@ -71,27 +72,62 @@ list." > (push header message-hidden-headers))) > notmuch-mua-hidden-headers)) > > +(defun w3m-region (start end)) ;; From `w3m.el'. What is the purpose of the above line? If it is to make the compiler aware of the function, you should use ?declare-function? instead. Defun will erase the original definition of the w3m-region function. -- Aaron Ecay
