Re: [PATCH] reply: Convert JSON format to use sprinter

2012-08-12 Thread David Bremner
Austin Clements amdra...@mit.edu writes: Almost all of reply was already being formatted using the sprinter. This patch converts the top-level dictionary to use the sprinter interface. pushed, d ___ notmuch mailing list notmuch@notmuchmail.org

Re: [PATCH] reply: Convert JSON format to use sprinter

2012-08-07 Thread Tomi Ollila
On Tue, Aug 07 2012, Austin Clements amdra...@mit.edu wrote: Almost all of reply was already being formatted using the sprinter. This patch converts the top-level dictionary to use the sprinter interface. --- LGTM. Tomi One last sprinter piece that had slipped through the cracks.

Re: [PATCH] reply: Convert JSON format to use sprinter

2012-08-07 Thread Mark Walters
On Tue, 07 Aug 2012, Austin Clements amdra...@mit.edu wrote: Almost all of reply was already being formatted using the sprinter. This patch converts the top-level dictionary to use the sprinter interface. --- One last sprinter piece that had slipped through the cracks. Looks good to me +1