diff options
| author | Markus Armbruster <armbru@redhat.com> | 2024-02-16 15:58:30 +0100 |
|---|---|---|
| committer | Markus Armbruster <armbru@redhat.com> | 2024-02-26 10:43:56 +0100 |
| commit | bf00dc19f3aacf014b308d57bb0debf250339396 (patch) | |
| tree | 64ce6eb2bab7a32ea06d1aabef8a5fbd641c8a54 | |
| parent | 15333abed9112f99e0b1af4327154af733b987d3 (diff) | |
| download | focaccia-qemu-bf00dc19f3aacf014b308d57bb0debf250339396.tar.gz focaccia-qemu-bf00dc19f3aacf014b308d57bb0debf250339396.zip | |
qapi: Improve error position for bogus invalid "Returns" section
When something other than a command has a "Returns" section, the error message points to the beginning of the definition comment. Point to the "Returns" section instead. Signed-off-by: Markus Armbruster <armbru@redhat.com> Message-ID: <20240216145841.2099240-7-armbru@redhat.com> Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
| -rw-r--r-- | scripts/qapi/parser.py | 10 | ||||
| -rw-r--r-- | tests/qapi-schema/doc-invalid-return.err | 2 |
2 files changed, 8 insertions, 4 deletions
diff --git a/scripts/qapi/parser.py b/scripts/qapi/parser.py index 82db595dcf..a771013959 100644 --- a/scripts/qapi/parser.py +++ b/scripts/qapi/parser.py @@ -759,9 +759,13 @@ class QAPIDoc: self.features[feature.name].connect(feature) def check_expr(self, expr: QAPIExpression) -> None: - if self.has_section('Returns') and 'command' not in expr: - raise QAPISemError(self.info, - "'Returns:' is only valid for commands") + if 'command' not in expr: + sec = next((sec for sec in self.sections + if sec.name == 'Returns'), + None) + if sec: + raise QAPISemError(sec.info, + "'Returns:' is only valid for commands") def check(self) -> None: diff --git a/tests/qapi-schema/doc-invalid-return.err b/tests/qapi-schema/doc-invalid-return.err index 2ad89c5941..bc5826de20 100644 --- a/tests/qapi-schema/doc-invalid-return.err +++ b/tests/qapi-schema/doc-invalid-return.err @@ -1 +1 @@ -doc-invalid-return.json:3: 'Returns:' is only valid for commands +doc-invalid-return.json:5: 'Returns:' is only valid for commands |