diff options
| author | Markus Armbruster <armbru@redhat.com> | 2024-02-16 15:58:37 +0100 |
|---|---|---|
| committer | Markus Armbruster <armbru@redhat.com> | 2024-02-26 10:43:56 +0100 |
| commit | 0b82a7440c22056745a925d0b1c070e18534aa0e (patch) | |
| tree | 15e2a8f30582b8b1583dbedcd656cff6e60d9198 | |
| parent | fedc04c9fcbd9802d21848b4d55c436c003f4961 (diff) | |
| download | focaccia-qemu-0b82a7440c22056745a925d0b1c070e18534aa0e.tar.gz focaccia-qemu-0b82a7440c22056745a925d0b1c070e18534aa0e.zip | |
qapi: Merge adjacent untagged sections
The parser mostly doesn't create adjacent untagged sections, and merging the ones it does create is hardly worth the bother. I'm doing it to avoid behavioral change in the next commit. Signed-off-by: Markus Armbruster <armbru@redhat.com> Message-ID: <20240216145841.2099240-14-armbru@redhat.com> Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
| -rw-r--r-- | scripts/qapi/parser.py | 3 | ||||
| -rw-r--r-- | tests/qapi-schema/doc-empty-features.out | 2 |
2 files changed, 3 insertions, 2 deletions
diff --git a/scripts/qapi/parser.py b/scripts/qapi/parser.py index de2ce3ec2c..48cc9a6367 100644 --- a/scripts/qapi/parser.py +++ b/scripts/qapi/parser.py @@ -719,6 +719,9 @@ class QAPIDoc: self._start_symbol_section(self.features, name) def _start_section(self, tag: Optional[str] = None) -> None: + if not tag and not self._section.tag: + # extend current section + return if tag in ('Returns', 'Since') and self.has_section(tag): raise QAPIParseError(self._parser, "duplicated '%s' section" % tag) diff --git a/tests/qapi-schema/doc-empty-features.out b/tests/qapi-schema/doc-empty-features.out index 0f85623dde..473f59552d 100644 --- a/tests/qapi-schema/doc-empty-features.out +++ b/tests/qapi-schema/doc-empty-features.out @@ -14,6 +14,4 @@ command foo None -> None gen=True success_response=True boxed=False oob=False preconfig=False doc symbol=foo body= - - section=None not a description |