Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
qapi/parser: Drop two bad type hints for now
Two type hints fail centos-stream-8-x86_64 CI.  They are actually
broken.  Changing them to Optional[re.Match[str]] fixes them locally
for me, but then CI fails differently.  Drop them for now.

Fixes: 3e32dca (qapi: Rewrite parsing of doc comment section symbols and tags)
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Message-Id: <20230517061600.1782455-1-armbru@redhat.com>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Tested-by: Igor Mammedov <imammedo@redhat.com>
  • Loading branch information
Markus Armbruster committed May 17, 2023
1 parent a729c0c commit 28d2a4f
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions scripts/qapi/parser.py
Expand Up @@ -563,11 +563,11 @@ def end_comment(self) -> None:
self._switch_section(QAPIDoc.NullSection(self._parser))

@staticmethod
def _match_at_name_colon(string: str) -> re.Match:
def _match_at_name_colon(string: str):
return re.match(r'@([^:]*): *', string)

@staticmethod
def _match_section_tag(string: str) -> re.Match:
def _match_section_tag(string: str):
return re.match(r'(Returns|Since|Notes?|Examples?|TODO): *', string)

def _append_body_line(self, line: str) -> None:
Expand Down

0 comments on commit 28d2a4f

Please sign in to comment.