|
 |
9ae3a8 |
From f1567daa732a9f61641e6089525db0ae7f75aa06 Mon Sep 17 00:00:00 2001
|
|
|
9ae3a8 |
From: Kevin Wolf <kwolf@redhat.com>
|
|
|
9ae3a8 |
Date: Mon, 9 Sep 2013 14:28:09 +0200
|
|
|
9ae3a8 |
Subject: [PATCH 18/38] qapi-visit.py: Implement 'base' for unions
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
RH-Author: Kevin Wolf <kwolf@redhat.com>
|
|
|
9ae3a8 |
Message-id: <1378736903-18489-19-git-send-email-kwolf@redhat.com>
|
|
|
9ae3a8 |
Patchwork-id: 54205
|
|
|
9ae3a8 |
O-Subject: [RHEL-7.0 qemu-kvm PATCH 18/32] qapi-visit.py: Implement 'base' for unions
|
|
|
9ae3a8 |
Bugzilla: 1005818
|
|
|
9ae3a8 |
RH-Acked-by: Fam Zheng <famz@redhat.com>
|
|
|
9ae3a8 |
RH-Acked-by: Max Reitz <mreitz@redhat.com>
|
|
|
9ae3a8 |
RH-Acked-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
Bugzilla: 1005818
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
This implements the visitor part of base types for unions. Parsed into
|
|
|
9ae3a8 |
QMP, this example schema definition...
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
{ 'type': 'BlockOptionsBase', 'data': { 'read-only': 'bool' } }
|
|
|
9ae3a8 |
{ 'type': 'BlockOptionsQcow2, 'data': { 'lazy-refcounts': 'bool' } }
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
{ 'union': 'BlockOptions',
|
|
|
9ae3a8 |
'base': 'BlockOptionsBase',
|
|
|
9ae3a8 |
'data': {
|
|
|
9ae3a8 |
'raw': 'BlockOptionsRaw'
|
|
|
9ae3a8 |
'qcow2': 'BlockOptionsQcow2'
|
|
|
9ae3a8 |
} }
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
...would describe the following JSON object:
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
{ "type": "qcow2",
|
|
|
9ae3a8 |
"read-only": true,
|
|
|
9ae3a8 |
"data": { "lazy-refcounts": false } }
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
|
|
|
9ae3a8 |
Reviewed-by: Eric Blake <eblake@redhat.com>
|
|
|
9ae3a8 |
(cherry picked from commit 0aef92b90d24858eea1ebd52a51bc31563f1fb52)
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
|
|
|
9ae3a8 |
---
|
|
|
9ae3a8 |
scripts/qapi-visit.py | 30 +++++++++++++++++++++++++-----
|
|
|
9ae3a8 |
1 file changed, 25 insertions(+), 5 deletions(-)
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
9ae3a8 |
---
|
|
|
9ae3a8 |
scripts/qapi-visit.py | 30 +++++++++++++++++++++++++-----
|
|
|
9ae3a8 |
1 files changed, 25 insertions(+), 5 deletions(-)
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
diff --git a/scripts/qapi-visit.py b/scripts/qapi-visit.py
|
|
|
9ae3a8 |
index 5adc2af..be56ba4 100644
|
|
|
9ae3a8 |
--- a/scripts/qapi-visit.py
|
|
|
9ae3a8 |
+++ b/scripts/qapi-visit.py
|
|
|
9ae3a8 |
@@ -151,7 +151,13 @@ void visit_type_%(name)s(Visitor *m, %(name)s * obj, const char *name, Error **e
|
|
|
9ae3a8 |
''',
|
|
|
9ae3a8 |
name=name)
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
-def generate_visit_union(name, members):
|
|
|
9ae3a8 |
+def generate_visit_union(expr):
|
|
|
9ae3a8 |
+
|
|
|
9ae3a8 |
+ name = expr['union']
|
|
|
9ae3a8 |
+ members = expr['data']
|
|
|
9ae3a8 |
+
|
|
|
9ae3a8 |
+ base = expr.get('base')
|
|
|
9ae3a8 |
+
|
|
|
9ae3a8 |
ret = generate_visit_enum('%sKind' % name, members.keys())
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
ret += mcgen('''
|
|
|
9ae3a8 |
@@ -164,14 +170,28 @@ void visit_type_%(name)s(Visitor *m, %(name)s ** obj, const char *name, Error **
|
|
|
9ae3a8 |
visit_start_struct(m, (void **)obj, "%(name)s", name, sizeof(%(name)s), &err;;
|
|
|
9ae3a8 |
if (!err) {
|
|
|
9ae3a8 |
if (obj && *obj) {
|
|
|
9ae3a8 |
- visit_type_%(name)sKind(m, &(*obj)->kind, "type", &err;;
|
|
|
9ae3a8 |
- if (!err) {
|
|
|
9ae3a8 |
- switch ((*obj)->kind) {
|
|
|
9ae3a8 |
''',
|
|
|
9ae3a8 |
name=name)
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
+
|
|
|
9ae3a8 |
push_indent()
|
|
|
9ae3a8 |
push_indent()
|
|
|
9ae3a8 |
+ push_indent()
|
|
|
9ae3a8 |
+
|
|
|
9ae3a8 |
+ if base:
|
|
|
9ae3a8 |
+ struct = find_struct(base)
|
|
|
9ae3a8 |
+ push_indent()
|
|
|
9ae3a8 |
+ ret += generate_visit_struct_fields("", struct['data'])
|
|
|
9ae3a8 |
+ pop_indent()
|
|
|
9ae3a8 |
+
|
|
|
9ae3a8 |
+ pop_indent()
|
|
|
9ae3a8 |
+ ret += mcgen('''
|
|
|
9ae3a8 |
+ visit_type_%(name)sKind(m, &(*obj)->kind, "type", &err;;
|
|
|
9ae3a8 |
+ if (!err) {
|
|
|
9ae3a8 |
+ switch ((*obj)->kind) {
|
|
|
9ae3a8 |
+''',
|
|
|
9ae3a8 |
+ name=name)
|
|
|
9ae3a8 |
+
|
|
|
9ae3a8 |
for key in members:
|
|
|
9ae3a8 |
ret += mcgen('''
|
|
|
9ae3a8 |
case %(abbrev)s_KIND_%(enum)s:
|
|
|
9ae3a8 |
@@ -344,7 +364,7 @@ for expr in exprs:
|
|
|
9ae3a8 |
ret = generate_declaration(expr['type'], expr['data'])
|
|
|
9ae3a8 |
fdecl.write(ret)
|
|
|
9ae3a8 |
elif expr.has_key('union'):
|
|
|
9ae3a8 |
- ret = generate_visit_union(expr['union'], expr['data'])
|
|
|
9ae3a8 |
+ ret = generate_visit_union(expr)
|
|
|
9ae3a8 |
ret += generate_visit_list(expr['union'], expr['data'])
|
|
|
9ae3a8 |
fdef.write(ret)
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
--
|
|
|
9ae3a8 |
1.7.1
|
|
|
9ae3a8 |
|