diff options
| author | Eric Blake <eblake@redhat.com> | 2015-05-04 09:05:17 -0600 |
|---|---|---|
| committer | Markus Armbruster <armbru@redhat.com> | 2015-05-05 18:39:01 +0200 |
| commit | 4dc2e6906e1084fdd37bf67385c5dcd2c72ae22b (patch) | |
| tree | 38dbff47247cefd2448c56813de8bed894096c92 /tests/qapi-schema/redefined-builtin.out | |
| parent | cfdd5bcad515a8371af59dba9625e31a6f6f733e (diff) | |
| download | qemu-4dc2e6906e1084fdd37bf67385c5dcd2c72ae22b.zip | |
qapi: Better error messages for duplicated expressions
The previous commit demonstrated that the generator overlooked
duplicate expressions:
- a complex type or command reusing a built-in type name
- redeclaration of a type name, whether by the same or different
metatype
- redeclaration of a command or event
- collision of a type with implicit 'Kind' enum for a union
- collision with an implicit MAX enum constant
Since the c_type() function in the generator treats all names
as being in the same namespace, this patch adds a global array
to track all known names and their source, to prevent collisions
before it can cause further problems. While valid .json files
won't trigger any of these cases, we might as well be nicer to
developers that make a typo while trying to add new QAPI code.
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 'tests/qapi-schema/redefined-builtin.out')
| -rw-r--r-- | tests/qapi-schema/redefined-builtin.out | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/tests/qapi-schema/redefined-builtin.out b/tests/qapi-schema/redefined-builtin.out index b0a9aea548..e69de29bb2 100644 --- a/tests/qapi-schema/redefined-builtin.out +++ b/tests/qapi-schema/redefined-builtin.out @@ -1,3 +0,0 @@ -[OrderedDict([('type', 'size'), ('data', OrderedDict([('myint', 'size')]))])] -[] -[OrderedDict([('type', 'size'), ('data', OrderedDict([('myint', 'size')]))])] |