On Mon, 01 Mar 2010 13:26:52 -0400, David Bremner <bremner@unb.ca> wrote:
> On Mon,  1 Mar 2010 16:39:06 +0100, Sebastian Spaeth <Sebastian@SSpaeth.de> wrote:
> > The previous json patches forgot to add the notmuch tags to the json
> > output. This is inconsistent to the text output so here they are. We
> > just output a 'tags' field that sends a space delimited list of
> > json-encoded notmuch tags.
> > 
> > Signed-off-by: Sebastian Spaeth <Sebastian@SSpaeth.de>
> 
> I agree we need tags in json output, but I think that making the client
> parse the string of (space delimited?) tags defeats a bit the purpose of
> having json output.  What do you think about 
> 
>        "tags": [ "inbox", "unread" ] 
> 
> instead of 
> 
>         "tags": "inbox unread"

I agree that having the JSON "tags" output field be an array of strings
makes the most sense.

jamie.