yuzu/unicorn
1
0
Fork 0
mirror of https://github.com/yuzu-emu/unicorn.git synced 2025-03-23 05:25:11 +00:00
Code Issues Packages Projects Releases Wiki Activity

qapi: Factor out QAPISchemaObjectType.check_clash()

Browse source 
Consolidate two common sequences of clash detection into a
new QAPISchemaObjectType.check_clash() helper method.

No change to generated code.

Backports commit c2183d2e62b6d9d66f80bc0bcf4fc7ec3c5d76d4 from qemu
This commit is contained in:
Eric Blake 2018-02-19 20:23:33 -05:00 committed by Lioncash
parent e1f3c46a16
commit 51b4313b34
No known key found for this signature in database
GPG key ID: 4E3C3CC1031BA9C7
1 changed files with 7 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
qemu/scripts/qapi.py
View file

@ -981,10 +981,8 @@ class QAPISchemaObjectType(QAPISchemaType):
if self._base_name:
self.base = schema.lookup_type(self._base_name)
assert isinstance(self.base, QAPISchemaObjectType)
assert not self.base.variants # not implemented
self.base.check(schema)
for m in self.base.members:
m.check_clash(seen)
self.base.check_clash(schema, seen)
self.members = seen.values()
for m in self.local_members:
m.check(schema)
@ -994,6 +992,11 @@ class QAPISchemaObjectType(QAPISchemaType):
assert self.variants.tag_member in self.members
self.variants.check_clash(schema, seen)
def check_clash(self, schema, seen):
assert not self.variants # not implemented
for m in self.members:
m.check_clash(seen)
def is_implicit(self):
# See QAPISchema._make_implicit_object_type()
return self.name[0] == ':'
@ -1064,11 +1067,8 @@ class QAPISchemaObjectTypeVariants(object):
for v in self.variants:
# Reset seen map for each variant, since qapi names from one
# branch do not affect another branch
vseen = dict(seen)
assert isinstance(v.type, QAPISchemaObjectType)
assert not v.type.variants # not implemented
for m in v.type.members:
m.check_clash(vseen)
v.type.check_clash(schema, dict(seen))
class QAPISchemaObjectTypeVariant(QAPISchemaObjectTypeMember):