diff options
| author | Markus Armbruster <armbru@redhat.com> | 2019-09-13 22:13:42 +0200 |
|---|---|---|
| committer | Markus Armbruster <armbru@redhat.com> | 2019-09-24 14:07:22 +0200 |
| commit | f03255362ae3bfd6f105c0fc855c713944f99717 (patch) | |
| tree | 82cf6ddf03b23e7556833f4c52500730d35aa737 /tests/qapi-schema/alternate-empty.json | |
| parent | 675b214bc6ba2c1d8ac499e339a8cb99c7f23c7c (diff) | |
qapi: Permit alternates with just one branch
A union or alternate without branches makes no sense and doesn't work:
it can't be instantiated. A union or alternate with just one branch
works, but is degenerate. We accept the former, but reject the
latter. Weird. docs/devel/qapi-code-gen.txt doesn't mention the
difference. It claims an alternate definition is "is similar to a
simple union type".
Permit degenerate alternates to make them consistent with unions.
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20190913201349.24332-10-armbru@redhat.com>
Diffstat (limited to 'tests/qapi-schema/alternate-empty.json')
| -rw-r--r-- | tests/qapi-schema/alternate-empty.json | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/qapi-schema/alternate-empty.json b/tests/qapi-schema/alternate-empty.json index fff15baf16..9f445474e6 100644 --- a/tests/qapi-schema/alternate-empty.json +++ b/tests/qapi-schema/alternate-empty.json @@ -1,2 +1,2 @@ -# alternates must list at least two types to be useful -{ 'alternate': 'Alt', 'data': { 'i': 'int' } } +# alternates cannot be empty +{ 'alternate': 'Alt', 'data': { } } |