diff options
author | Markus Armbruster <armbru@redhat.com> | 2024-02-16 15:58:36 +0100 |
---|---|---|
committer | Markus Armbruster <armbru@redhat.com> | 2024-02-26 10:43:56 +0100 |
commit | fedc04c9fcbd9802d21848b4d55c436c003f4961 (patch) | |
tree | 317ebb4e49ec019b166a66f8729bc3f800c7a0ba | |
parent | 66227e90478b34a2bc4bbd4f11f5ea637184c20b (diff) | |
download | qemu-fedc04c9fcbd9802d21848b4d55c436c003f4961.zip qemu-fedc04c9fcbd9802d21848b4d55c436c003f4961.tar.gz qemu-fedc04c9fcbd9802d21848b4d55c436c003f4961.tar.bz2 |
qapi: Call QAPIDoc.check() always
We currently call QAPIDoc.check() only for definition documentation.
Calling it for free-form documentation as well is simpler. No change,
because it doesn't actually do anything there.
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Message-ID: <20240216145841.2099240-13-armbru@redhat.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
-rw-r--r-- | scripts/qapi/schema.py | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/scripts/qapi/schema.py b/scripts/qapi/schema.py index 6a83695..8ba5665 100644 --- a/scripts/qapi/schema.py +++ b/scripts/qapi/schema.py @@ -95,10 +95,6 @@ class QAPISchemaEntity: for f in self.features: doc.connect_feature(f) - def check_doc(self): - if self.doc: - self.doc.check() - def _set_module(self, schema, info): assert self._checked fname = info.fname if info else QAPISchemaModule.BUILTIN_MODULE_NAME @@ -1223,9 +1219,10 @@ class QAPISchema: for ent in self._entity_list: ent.check(self) ent.connect_doc() - ent.check_doc() for ent in self._entity_list: ent.set_module(self) + for doc in self.docs: + doc.check() def visit(self, visitor): visitor.visit_begin(self) |