This whole series looks good to me (modulo the minor comments I have made). The tests all pass. Additionally it was very simple to modify notmuch-pick to use the new parser and that also seemed to work. Incidentally it seems that notmuch-show still uses the JSON format so we may want to switch that over too. Best wishes Mark On Sat, 18 May 2013, Austin Clements <amdragon@MIT.EDU> wrote: > This series implements an incremental S-expression parser and switches > search over to it. To simplify things, it also implements better > handing of stderr for asynchronous processes so we don't have to > handle errors embedded in the S-expression stream.