bwlp/qemu.git/tests/qapi-schema/struct-member-if-invalid.json, branch spice_video_codecsExperimental fork of QEMU with video encoding patcheshttps://git.openslx.org/bwlp/qemu.git/atom/tests/qapi-schema/struct-member-if-invalid.json?h=spice_video_codecs2019-09-24T12:07:23+00:00qapi: Fix missing 'if' checks in struct, union, alternate 'data'2019-09-24T12:07:23+00:00Markus Armbruster2019-09-14T15:34:59+00:00urn:sha1:dec0012ef8d644b5dde1b68ee8dab3f8c12e0b6b
Commit 87adbbffd4..3e270dcacc "qapi: Add 'if' to (implicit
struct|union|alternate) members" (v4.0.0) neglected test coverage, and
promptly failed to check the conditions. Review fail.
Recent commit "tests/qapi-schema: Demonstrate insufficient 'if'
checking" added test coverage, demonstrating the bug. Fix it by add
the missing check_if().
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Message-Id: <20190914153506.2151-13-armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
tests/qapi-schema: Demonstrate insufficient 'if' checking2019-09-24T12:07:23+00:00Markus Armbruster2019-09-14T15:34:52+00:00urn:sha1:ef91ab0d5fe1a263e9e3cc1d14ea702a0a9f2261
Cover invalid 'if' in struct members, features, union and alternate
branches. Four out of four are broken. Mark FIXME.
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Message-Id: <20190914153506.2151-6-armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
[Comment typo fixed]