Re: [notmuch] [PATCH] notmuch-show: add tags to json output

Subject: Re: [notmuch] [PATCH] notmuch-show: add tags to json output

Date: Mon, 01 Mar 2010 13:26:52 -0400

To: Sebastian Spaeth, notmuch@notmuchmail.org

Cc:

From: David Bremner


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"


Thread: