diff options
| author | Markus Armbruster <armbru@redhat.com> | 2019-09-27 15:46:22 +0200 |
|---|---|---|
| committer | Markus Armbruster <armbru@redhat.com> | 2019-09-28 17:17:18 +0200 |
| commit | d7bc17c602f128bb358376e6976b3b5dee1ad732 (patch) | |
| tree | dcb866d6555d49919c90479e62de679e36c3b966 /tests/qapi-schema/bad-ident.err | |
| parent | c9efc984ca5b5f75f184680e1ab9a7784d241578 (diff) | |
qapi: Improve reporting of invalid name errors
Split check_name() into check_name_is_str() and check_name_str(), keep
check_name() as a wrapper.
Move add_name()'s call into its caller check_exprs(), and inline.
This permits delaying check_name_str() there, so its error message
gains an "in definition" line.
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20190927134639.4284-10-armbru@redhat.com>
Diffstat (limited to 'tests/qapi-schema/bad-ident.err')
| -rw-r--r-- | tests/qapi-schema/bad-ident.err | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/tests/qapi-schema/bad-ident.err b/tests/qapi-schema/bad-ident.err index c4190602b5..6878889854 100644 --- a/tests/qapi-schema/bad-ident.err +++ b/tests/qapi-schema/bad-ident.err @@ -1 +1,2 @@ +tests/qapi-schema/bad-ident.json: In struct '*oops': tests/qapi-schema/bad-ident.json:2: 'struct' does not allow optional name '*oops' |