Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
qapi: Move detection of doc / expression name mismatch
Move the check whether the doc matches the expression name from check_definition_doc() to check_exprs(). This changes the error location from the comment to the expression. Makes sense as the message talks about the expression: "Definition of '%s' follows documentation for '%s'". It's also a step towards getting rid of check_docs(). Signed-off-by: Markus Armbruster <armbru@redhat.com> Reviewed-by: Eric Blake <eblake@redhat.com> Message-Id: <1489582656-31133-33-git-send-email-armbru@redhat.com>
- Loading branch information
Markus Armbruster
committed
Mar 16, 2017
1 parent
e7823a2
commit 7947016
Showing
2 changed files
with
19 additions
and
11 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1 +1 @@ | ||
tests/qapi-schema/doc-bad-symbol.json:3: Definition of 'foo' follows documentation for 'food' | ||
tests/qapi-schema/doc-bad-symbol.json:6: Definition of 'foo' follows documentation for 'food' |