aboutsummaryrefslogtreecommitdiff
path: root/scripts
diff options
context:
space:
mode:
authorWenchao Xia <wenchaoqemu@gmail.com>2014-06-24 16:33:59 -0700
committerLuiz Capitulino <lcapitulino@redhat.com>2014-06-27 09:27:56 -0400
commitd6f9c82c62023c1ed5af76847c55b0a663bdd2cc (patch)
treecc3d9a6d8738475f7f2dd751db3a5ed937f39aab /scripts
parent1dbbe04525d35b6888a6a89f97302306d06eafae (diff)
downloadqemu-d6f9c82c62023c1ed5af76847c55b0a663bdd2cc.zip
qemu-d6f9c82c62023c1ed5af76847c55b0a663bdd2cc.tar.gz
qemu-d6f9c82c62023c1ed5af76847c55b0a663bdd2cc.tar.bz2
qapi script: clean up in scripts
This patch improve docs and uses c_type(argentry, is_param=True) in script. Signed-off-by: Wenchao Xia <wenchaoqemu@gmail.com> Reviewed-by: Eric Blake <eblake@redhat.com> Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com>
Diffstat (limited to 'scripts')
-rw-r--r--scripts/qapi-event.py5
-rw-r--r--scripts/qapi.py2
2 files changed, 3 insertions, 4 deletions
diff --git a/scripts/qapi-event.py b/scripts/qapi-event.py
index 3a1cd61..601e307 100644
--- a/scripts/qapi-event.py
+++ b/scripts/qapi-event.py
@@ -26,9 +26,8 @@ def _generate_event_api_name(event_name, params):
api_name += "bool has_%s,\n" % c_var(argname)
api_name += "".ljust(l)
- if argentry == "str":
- api_name += "const "
- api_name += "%s %s,\n" % (c_type(argentry), c_var(argname))
+ api_name += "%s %s,\n" % (c_type(argentry, is_param=True),
+ c_var(argname))
api_name += "".ljust(l)
api_name += "Error **errp)"
diff --git a/scripts/qapi.py b/scripts/qapi.py
index 54b97cb..f2c6d1f 100644
--- a/scripts/qapi.py
+++ b/scripts/qapi.py
@@ -255,7 +255,7 @@ def check_event(expr, expr_info):
if structured:
raise QAPIExprError(expr_info,
"Nested structure define in event is not "
- "supported now, event '%s', argname '%s'"
+ "supported, event '%s', argname '%s'"
% (expr['event'], argname))
def check_union(expr, expr_info):