diff mbox

qapi: use valid JSON in schema

Message ID 1365621164-17079-1-git-send-email-eblake@redhat.com
State New
Headers show

Commit Message

Eric Blake April 10, 2013, 7:12 p.m. UTC 
* qapi-schema.json: JSON doesn't allow trailing commas.

Signed-off-by: Eric Blake <eblake@redhat.com>
---

a multi-line regex search for ',[ \n]*[]}]' didn't turn up any
other violations

 qapi-schema.json | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Luiz Capitulino April 11, 2013, 12:50 p.m. UTC | #1 
On Wed, 10 Apr 2013 13:12:44 -0600
Eric Blake <eblake@redhat.com> wrote:

> * qapi-schema.json: JSON doesn't allow trailing commas.
> 
> Signed-off-by: Eric Blake <eblake@redhat.com>

Applied to the qmp branch, thanks.

> ---
> 
> a multi-line regex search for ',[ \n]*[]}]' didn't turn up any
> other violations
> 
>  qapi-schema.json | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/qapi-schema.json b/qapi-schema.json
> index db542f6..044517d 100644
> --- a/qapi-schema.json
> +++ b/qapi-schema.json
> @@ -1600,7 +1600,7 @@
>  ##
>  { 'union': 'BlockdevAction',
>    'data': {
> -       'blockdev-snapshot-sync': 'BlockdevSnapshot',
> +       'blockdev-snapshot-sync': 'BlockdevSnapshot'
>     } }
> 
>  ##
diff mbox

Patch

diff --git a/qapi-schema.json b/qapi-schema.json
index db542f6..044517d 100644
--- a/qapi-schema.json
+++ b/qapi-schema.json
@@ -1600,7 +1600,7 @@ 
 ##
 { 'union': 'BlockdevAction',
   'data': {
-       'blockdev-snapshot-sync': 'BlockdevSnapshot',
+       'blockdev-snapshot-sync': 'BlockdevSnapshot'
    } }

 ##