diff options
| author | Eric Blake | 2015-05-04 17:05:37 +0200 |
|---|---|---|
| committer | Markus Armbruster | 2015-05-05 18:39:02 +0200 |
| commit | ff55d72eaf9628e7d58e7b067b361cdbf789c9f4 (patch) | |
| tree | 39ff570f5e091ca43323e7270f80bb9176a9907f /include/exec/exec-all.h | |
| parent | qapi: Support (subset of) \u escapes in strings (diff) | |
| download | qemu-ff55d72eaf9628e7d58e7b067b361cdbf789c9f4.tar.gz qemu-ff55d72eaf9628e7d58e7b067b361cdbf789c9f4.tar.xz qemu-ff55d72eaf9628e7d58e7b067b361cdbf789c9f4.zip | |
qapi: Check for member name conflicts with a base class
Our type inheritance for both 'struct' and for flat 'union' merges
key/value pairs from the base class with those from the type in
question. Although the C code currently boxes things so that there
is a distinction between which member is referred to, the QMP wire
format does not allow passing a key more than once in a single
object. Besides, if we ever change the generated C code to not be
quite so boxy, we'd want to avoid duplicate member names there,
too.
Fix a testsuite entry added in an earlier patch, as well as adding
a couple more tests to ensure we have appropriate coverage. Ensure
that collisions are detected, regardless of whether there is a
difference in opinion on whether the member name is optional.
Signed-off-by: Eric Blake <eblake@redhat.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Diffstat (limited to 'include/exec/exec-all.h')
0 files changed, 0 insertions, 0 deletions