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.err | |
| 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.err')
| -rw-r--r-- | tests/qapi-schema/alternate-empty.err | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/qapi-schema/alternate-empty.err b/tests/qapi-schema/alternate-empty.err index bb06c5bfec..86dbc666eb 100644 --- a/tests/qapi-schema/alternate-empty.err +++ b/tests/qapi-schema/alternate-empty.err @@ -1 +1 @@ -tests/qapi-schema/alternate-empty.json:2: Alternate 'Alt' should have at least two branches in 'data' +tests/qapi-schema/alternate-empty.json:2: Alternate 'Alt' cannot have empty 'data' |