|
From: | Avi Kivity |
Subject: | Re: [Qemu-devel] [PATCH 14/22] qapi: add query-version QMP command |
Date: | Wed, 09 Mar 2011 15:51:58 +0200 |
User-agent: | Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.13) Gecko/20101209 Fedora/3.1.7-0.35.b3pre.fc14 Lightning/1.0b3pre Thunderbird/3.1.7 |
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
[Prev in Thread] | Current Thread | [Next in Thread] |