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.err | |
| 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.err')
| -rw-r--r-- | tests/qapi-schema/args-name-clash.err | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/tests/qapi-schema/args-name-clash.err b/tests/qapi-schema/args-name-clash.err index e69de29bb2..d953e8d241 100644 --- a/tests/qapi-schema/args-name-clash.err +++ b/tests/qapi-schema/args-name-clash.err @@ -0,0 +1 @@ +tests/qapi-schema/args-name-clash.json:4: 'a_b' (parameter of oops) collides with 'a-b' (parameter of oops) |