[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 12/16] qapi: Call QAPIDoc.check() always
From: |
Markus Armbruster |
Subject: |
[PATCH 12/16] qapi: Call QAPIDoc.check() always |
Date: |
Fri, 16 Feb 2024 15:58:36 +0100 |
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>
---
scripts/qapi/schema.py | 7 ++-----
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/scripts/qapi/schema.py b/scripts/qapi/schema.py
index 6a836950a9..8ba5665bc6 100644
--- a/scripts/qapi/schema.py
+++ b/scripts/qapi/schema.py
@@ -95,10 +95,6 @@ def connect_doc(self, doc=None):
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 @@ def check(self):
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)
--
2.43.0
- [PATCH 16/16] qapi: Divorce QAPIDoc from QAPIParseError, (continued)
- [PATCH 16/16] qapi: Divorce QAPIDoc from QAPIParseError, Markus Armbruster, 2024/02/16
- [PATCH 03/16] tests/qapi-schema: Cover 'Features:' not followed by descriptions, Markus Armbruster, 2024/02/16
- [PATCH 15/16] qapi: Reject multiple and empty feature descriptions, Markus Armbruster, 2024/02/16
- [PATCH 08/16] qapi: Rename QAPIDoc.Section.name to .tag, Markus Armbruster, 2024/02/16
- [PATCH 02/16] tests/qapi-schema: Cover duplicate 'Features:' line, Markus Armbruster, 2024/02/16
- [PATCH 12/16] qapi: Call QAPIDoc.check() always,
Markus Armbruster <=
- [PATCH 14/16] qapi: Rewrite doc comment parser, Markus Armbruster, 2024/02/16
- [PATCH 10/16] qapi: Require descriptions and tagged sections to be indented, Markus Armbruster, 2024/02/16