On Fri, 13 Jul 2012, Austin Clements <amdragon@MIT.EDU> wrote: > This series builds on the JSON-based search series [0] to clean up > several other aspects of search-mode. It removes constraints on the > formatting of tags in the result line (you can even leave them out > entirely), it recolors lines when tags change, it adds supports for > multi-line result formats, and rendering big search buffers should be > less quadratic (it might even be linear). Much of this derives from > having a single object representation of a result (the JSON plist) and > a simple method for rendering it to the buffer. Overall this series looks excellent. I have reviewed all the patches except patch 4/7 (I will need to work out how macros work to review that) and they seem fine modulo the small comments I have made. One final comment: I couldn't persuade the defcustom search result to take a newline symbol to test multiline results. It seems to escape entries of \n, and I couldn't enter returns (and ^Q return didn't work either). It was easy by editing the option in my .emacs file and then seemed to work as expected. Best wishes Mark > > [0] 1341870162-17782-1-git-send-email-amdragon@mit.edu > > _______________________________________________ > notmuch mailing list > notmuch@notmuchmail.org > http://notmuchmail.org/mailman/listinfo/notmuch