diff options
| author | Eric Blake | 2016-02-18 07:48:17 +0100 |
|---|---|---|
| committer | Markus Armbruster | 2016-02-19 11:08:56 +0100 |
| commit | 46534309e667fd860720f983c2c9aefe0354340d (patch) | |
| tree | def73f0f34e3c1fd607c2624d350312e2a0a0791 /include | |
| parent | qapi: Forbid empty unions and useless alternates (diff) | |
| download | qemu-46534309e667fd860720f983c2c9aefe0354340d.tar.gz qemu-46534309e667fd860720f983c2c9aefe0354340d.tar.xz qemu-46534309e667fd860720f983c2c9aefe0354340d.zip | |
qapi: Forbid 'any' inside an alternate
The whole point of an alternate is to allow some type-safety while
still accepting more than one JSON type. Meanwhile, the 'any'
type exists to bypass type-safety altogether. The two are
incompatible: you can't accept every type, and still tell which
branch of the alternate to use for the parse; fix this to give a
sane error instead of a Python stack trace.
Note that other types that can't be alternate members are caught
earlier, by check_type().
Signed-off-by: Eric Blake <eblake@redhat.com>
Message-Id: <1455778109-6278-4-git-send-email-eblake@redhat.com>
[Commit message tweaked]
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Diffstat (limited to 'include')
0 files changed, 0 insertions, 0 deletions