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/union-branch-if-invalid.err | |
| parent | 19e950d9d47d3efe4c8d5c6c872a53889a54363c (diff) | |
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/union-branch-if-invalid.err')
| -rw-r--r-- | tests/qapi-schema/union-branch-if-invalid.err | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/tests/qapi-schema/union-branch-if-invalid.err b/tests/qapi-schema/union-branch-if-invalid.err index 607edee382..b49cf9b664 100644 --- a/tests/qapi-schema/union-branch-if-invalid.err +++ b/tests/qapi-schema/union-branch-if-invalid.err @@ -1 +1,2 @@ +tests/qapi-schema/union-branch-if-invalid.json: In union 'Uni': tests/qapi-schema/union-branch-if-invalid.json:4: 'if' condition '' makes no sense |