aboutsummaryrefslogtreecommitdiff
path: root/qapi
diff options
context:
space:
mode:
authorEric Blake <eblake@redhat.com>2015-11-18 01:52:48 -0700
committerMarkus Armbruster <armbru@redhat.com>2015-12-17 08:21:27 +0100
commit10565ca92a8d3f8a34559329acfbdb25a791b594 (patch)
tree92becea28277b3be70530d6b75e0b2171ecb4042 /qapi
parentc2183d2e62b6d9d66f80bc0bcf4fc7ec3c5d76d4 (diff)
downloadqemu-10565ca92a8d3f8a34559329acfbdb25a791b594.zip
qemu-10565ca92a8d3f8a34559329acfbdb25a791b594.tar.gz
qemu-10565ca92a8d3f8a34559329acfbdb25a791b594.tar.bz2
qapi: Hoist tag collision check to Variants.check()
Checking that a given QAPISchemaObjectTypeVariant.name is a member of the corresponding QAPISchemaEnumType of the owning QAPISchemaObjectTypeVariants.tag_member ensures that there are no collisions in the generated C union for those tag values (since the enum itself should have no collisions). However, ever since its introduction in f51d8c3d, this was the only additional action of of Variant.check(), beyond calling the superclass Member.check(). This forces a difference in .check() signatures, just to pass the enum type down. Simplify things by instead doing the tag name check as part of Variants.check(), at which point we can rely on inheritance instead of overriding Variant.check(). Signed-off-by: Eric Blake <eblake@redhat.com> Message-Id: <1447836791-369-14-git-send-email-eblake@redhat.com> Signed-off-by: Markus Armbruster <armbru@redhat.com>
Diffstat (limited to 'qapi')
0 files changed, 0 insertions, 0 deletions