diff options
| author | Anton Nefedov <anton.nefedov@virtuozzo.com> | 2018-06-18 11:40:05 +0300 |
|---|---|---|
| committer | Markus Armbruster <armbru@redhat.com> | 2018-06-22 16:33:46 +0200 |
| commit | 800877bb1639d38ffaebe312a37b61c66bb10c83 (patch) | |
| tree | 766371f9d2bb317d702ed6c4317d9ae75a0bd67f /tests/qapi-schema/qapi-schema-test.out | |
| parent | fe170d8bfaa12d63117cad8707ac18c3b2f3bb8e (diff) | |
| download | qemu-800877bb1639d38ffaebe312a37b61c66bb10c83.zip qemu-800877bb1639d38ffaebe312a37b61c66bb10c83.tar.gz qemu-800877bb1639d38ffaebe312a37b61c66bb10c83.tar.bz2 | |
qapi: allow empty branches in flat unions
It often happens that just a few discriminator values imply extra data in
a flat union. Existing checks did not make possible to leave other values
uncovered. Such cases had to be worked around by either stating a dummy
(empty) type or introducing another (subset) discriminator enumeration.
Both options create redundant entities in qapi files for little profit.
With this patch it is not necessary anymore to add designated union
fields for every possible value of a discriminator enumeration.
Signed-off-by: Anton Nefedov <anton.nefedov@virtuozzo.com>
Message-Id: <1529311206-76847-2-git-send-email-anton.nefedov@virtuozzo.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Diffstat (limited to 'tests/qapi-schema/qapi-schema-test.out')
| -rw-r--r-- | tests/qapi-schema/qapi-schema-test.out | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/tests/qapi-schema/qapi-schema-test.out b/tests/qapi-schema/qapi-schema-test.out index 542a19c..0dbcdaf 100644 --- a/tests/qapi-schema/qapi-schema-test.out +++ b/tests/qapi-schema/qapi-schema-test.out @@ -23,7 +23,7 @@ object UserDefOne base UserDefZero member string: str optional=False member enum1: EnumOne optional=True -enum EnumOne ['value1', 'value2', 'value3'] +enum EnumOne ['value1', 'value2', 'value3', 'value4'] object UserDefZero member integer: int optional=False object UserDefTwoDictDict @@ -52,6 +52,7 @@ object UserDefFlatUnion case value1: UserDefA case value2: UserDefB case value3: UserDefB + case value4: q_empty object UserDefUnionBase base UserDefZero member string: str optional=False |