qapi/parser: clarify _end_section() logic
authorJohn Snow <jsnow@redhat.com>
Thu, 30 Sep 2021 20:57:09 +0000 (16:57 -0400)
committerMarkus Armbruster <armbru@redhat.com>
Sat, 2 Oct 2021 05:33:42 +0000 (07:33 +0200)
commit1e20a77576dedf1489ce1cdb6abc4b34663637a4
tree15e69c3ecdd775924ea525b4b52c0ce7785dd134
parentcd87c14cde5db42a2f13bfdbba1f3cbeb347a411
qapi/parser: clarify _end_section() logic

The "if self._section" clause in end_section is mysterious: In which
circumstances might we end a section when we don't have one?

QAPIDoc always expects there to be a "current section", only except
after a call to end_comment(). This actually *shouldn't* ever be 'None',
so let's remove that logic so I don't wonder why it's like this again in
three months.

Signed-off-by: John Snow <jsnow@redhat.com>
Message-Id: <20210930205716.1148693-7-jsnow@redhat.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
scripts/qapi/parser.py