diff options
author | Eric Blake <eblake@redhat.com> | 2015-11-18 01:52:48 -0700 |
---|---|---|
committer | Markus Armbruster <armbru@redhat.com> | 2015-12-17 08:21:27 +0100 |
commit | 10565ca92a8d3f8a34559329acfbdb25a791b594 (patch) | |
tree | 92becea28277b3be70530d6b75e0b2171ecb4042 /qapi | |
parent | c2183d2e62b6d9d66f80bc0bcf4fc7ec3c5d76d4 (diff) | |
download | qemu-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