diff options
author | Eric Blake <eblake@redhat.com> | 2015-05-04 09:05:28 -0600 |
---|---|---|
committer | Markus Armbruster <armbru@redhat.com> | 2015-05-05 18:39:01 +0200 |
commit | 3e391d355644b2bff7c9f187759aadb46c6e051f (patch) | |
tree | 6b4c8cf72241f97797f60a05289d5e3782692933 /scripts | |
parent | 895a2a80e0e054f0d5d3715aa93d10d15e49f9f7 (diff) | |
download | qemu-3e391d355644b2bff7c9f187759aadb46c6e051f.zip qemu-3e391d355644b2bff7c9f187759aadb46c6e051f.tar.gz qemu-3e391d355644b2bff7c9f187759aadb46c6e051f.tar.bz2 |
qapi: Forbid 'type' in schema
Referring to "type" as both a meta-type (built-in, enum, union,
alternate, or struct) and a specific type (the name that the
schema uses for declaring structs) is confusing. Finish up the
conversion to using "struct" in qapi schema by removing the hack
in the generator that allowed 'type'.
Signed-off-by: Eric Blake <eblake@redhat.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Diffstat (limited to 'scripts')
-rw-r--r-- | scripts/qapi.py | 14 |
1 files changed, 0 insertions, 14 deletions
diff --git a/scripts/qapi.py b/scripts/qapi.py index e50fec8..333f59a 100644 --- a/scripts/qapi.py +++ b/scripts/qapi.py @@ -617,20 +617,6 @@ def parse_schema(input_file): for expr_elem in schema.exprs: expr = expr_elem['expr'] info = expr_elem['info'] - - # back-compat hack until all schemas have been converted; - # preserve the ordering of the original expression - if expr.has_key('type'): - seen_type = False - for (key, value) in expr.items(): - if key == 'type': - seen_type = True - del expr['type'] - expr['struct'] = value - elif seen_type: - del expr[key] - expr[key] = value - if expr.has_key('enum'): check_keys(expr_elem, 'enum', ['data']) add_enum(expr['enum'], info, expr['data']) |