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 at 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.

Reply via email to