diff options
| author | Eric Blake | 2015-11-18 09:52:51 +0100 |
|---|---|---|
| committer | Markus Armbruster | 2015-12-17 08:21:27 +0100 |
| commit | 27b60ab93bd1d5d8c85f009aac7a97ffd2c53c86 (patch) | |
| tree | e53959d5527ff0ba5f9f9da180cb84ae65727205 /tests/qapi-schema/args-name-clash.out | |
| parent | qapi: Track owner of each object member (diff) | |
| download | qemu-27b60ab93bd1d5d8c85f009aac7a97ffd2c53c86.tar.gz qemu-27b60ab93bd1d5d8c85f009aac7a97ffd2c53c86.tar.xz qemu-27b60ab93bd1d5d8c85f009aac7a97ffd2c53c86.zip | |
qapi: Detect collisions in C member names
Detect attempts to declare two object members that would result
in the same C member name, by keying the 'seen' dictionary off
of the C name rather than the qapi name. It also requires passing
info through the check_clash() methods.
This addresses a TODO and fixes the previously-broken
args-name-clash test. The resulting error message demonstrates
the utility of the .describe() method added previously. No change
to generated code.
Signed-off-by: Eric Blake <eblake@redhat.com>
Message-Id: <1447836791-369-17-git-send-email-eblake@redhat.com>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Diffstat (limited to 'tests/qapi-schema/args-name-clash.out')
| -rw-r--r-- | tests/qapi-schema/args-name-clash.out | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/tests/qapi-schema/args-name-clash.out b/tests/qapi-schema/args-name-clash.out index 9b2f6e4d5f..e69de29bb2 100644 --- a/tests/qapi-schema/args-name-clash.out +++ b/tests/qapi-schema/args-name-clash.out @@ -1,6 +0,0 @@ -object :empty -object :obj-oops-arg - member a-b: str optional=False - member a_b: str optional=False -command oops :obj-oops-arg -> None - gen=True success_response=True |