diff options
author | Markus Armbruster <armbru@redhat.com> | 2019-09-27 15:46:17 +0200 |
---|---|---|
committer | Markus Armbruster <armbru@redhat.com> | 2019-09-28 17:17:18 +0200 |
commit | 7be6c511943613c60b3e5b640e09bdc916be3b65 (patch) | |
tree | 03764294007e38beb8f80df950e131b7eda50da5 /tests/qapi-schema/args-name-clash.err | |
parent | 19e950d9d47d3efe4c8d5c6c872a53889a54363c (diff) | |
download | qemu-7be6c511943613c60b3e5b640e09bdc916be3b65.zip |
qapi: Prefix frontend errors with an "in definition" line
We take pains to include the offending expression in error messages,
e.g.
tests/qapi-schema/alternate-any.json:2: alternate 'Alt' member 'one' cannot use type 'any'
But not always:
tests/qapi-schema/enum-if-invalid.json:2: 'if' condition must be a string or a list of strings
Instead of improving them one by one, report the offending expression
whenever it is known, like this:
tests/qapi-schema/enum-if-invalid.json: In enum 'TestIfEnum':
tests/qapi-schema/enum-if-invalid.json:2: 'if' condition must be a string or a list of strings
Error messages that mention the offending expression become a bit
redundant, e.g.
tests/qapi-schema/alternate-any.json: In alternate 'Alt':
tests/qapi-schema/alternate-any.json:2: alternate 'Alt' member 'one' cannot use type 'any'
I'll take care of that later in this series.
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20190927134639.4284-5-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 d953e8d241..eeb4e1b4dd 100644 --- a/tests/qapi-schema/args-name-clash.err +++ b/tests/qapi-schema/args-name-clash.err @@ -1 +1,2 @@ +tests/qapi-schema/args-name-clash.json: In command 'oops': tests/qapi-schema/args-name-clash.json:4: 'a_b' (parameter of oops) collides with 'a-b' (parameter of oops) |