Re: [Qemu-devel] [PATCH 14/22] qapi: add query-version QMP command

[Avi Kivity]

On 03/09/2011 03:44 PM, Anthony Liguori wrote:
> > > Yeah, it's only loosely JSON as I don't use a JSON parser.
> >
> > Goes kind of against all the buzzwords you're letting fly here...
>
>
> The schema defines arguments in a dictionary because in QMP, the 
> argument order doesn't matter.  But the argument order matters in C so 
> I need to use a custom parser to preserve dictionary order.

We could extend our parser to annotate the dictionary with the original 
order.  Not worth it though.

> There's no way to do commenting in JSON and I really wanted to have 
> inline documentation.
>
> But otherwise, it's valid JSON.

We should then have a transformation that generates a valid json for 
clients to use.  We could even include the documentation as a 'doc': key.

--
error compiling committee.c: too many arguments to function