[PATCH v2 0/7] emacs: JSON-based search cleanups

Jameson Graef Rollins jrollins at finestructure.net
Sat Jul 14 12:06:17 PDT 2012


On Fri, Jul 13 2012, Austin Clements <amdragon at MIT.EDU> wrote:
> This version updates the docstrings for notmuch-search-update-result
> and notmuch-search-show-result and fixes the scrolling artifact in
> notmuch-search-update-result.  I also updated the commit message of
> patch 7 to mention that it changes the behavior of
> notmuch-search-previous-thread.

Hey, Austin.  This is a really nice series.  Very nice cleanup, with a
lot of great side effects.  It definitely looks good to me, although
like Mark, I don't quite grok the defmacro in patch 4.  But this is only
my ignorance and not a comment on the patch at all.  I've also tested
and the series seems to work great.  I want to try some multi-line
search results now!

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/20120714/17de2914/attachment.pgp>


More information about the notmuch mailing list