diff options
| author | Markus Armbruster | 2019-09-27 15:46:33 +0200 |
|---|---|---|
| committer | Markus Armbruster | 2019-09-28 17:17:19 +0200 |
| commit | 3f58cc29a8d2f01dc498ff4624a25e72448059a1 (patch) | |
| tree | 39d5b75b46390d8009553c6962fe74d5eb6efa79 /tests/qapi-schema/enum-missing-data.err | |
| parent | qapi: Improve reporting of invalid flags (diff) | |
| download | qemu-3f58cc29a8d2f01dc498ff4624a25e72448059a1.tar.gz qemu-3f58cc29a8d2f01dc498ff4624a25e72448059a1.tar.xz qemu-3f58cc29a8d2f01dc498ff4624a25e72448059a1.zip | |
qapi: Improve reporting of missing / unknown definition keys
Have check_exprs() call check_keys() later, so its error messages gain
an "in definition" line.
Both check_keys() and check_name_is_str() check the definition's name
is a string. Since check_keys() now runs after check_name_is_str()
rather than before, its check is dead. Bury it. Checking values in
check_keys() is unclean anyway.
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20190927134639.4284-21-armbru@redhat.com>
Diffstat (limited to 'tests/qapi-schema/enum-missing-data.err')
| -rw-r--r-- | tests/qapi-schema/enum-missing-data.err | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/tests/qapi-schema/enum-missing-data.err b/tests/qapi-schema/enum-missing-data.err index 3c3c52d037..ffde1082c3 100644 --- a/tests/qapi-schema/enum-missing-data.err +++ b/tests/qapi-schema/enum-missing-data.err @@ -1 +1,2 @@ +tests/qapi-schema/enum-missing-data.json: In enum 'MyEnum': tests/qapi-schema/enum-missing-data.json:2: key 'data' is missing from enum 'MyEnum' |