Skip to content

Commit 900a002

Browse files
committed
Address AA-Turner's coding style review
1 parent cdadfa1 commit 900a002

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

Tools/clinic/clinic.py

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2156,7 +2156,7 @@ def print_block(
21562156
21572157
""")
21582158

2159-
if clinic:
2159+
if clinic is not None:
21602160
# Emit optional includes
21612161
for include, reason in sorted(clinic.includes.items()):
21622162
line = f'#include "{include}"'
@@ -2305,7 +2305,7 @@ def __init__(self, clinic: Clinic) -> None: ...
23052305
def parse(self, block: Block) -> None: ...
23062306

23072307

2308-
clinic : Clinic | None = None
2308+
clinic: Clinic | None = None
23092309
class Clinic:
23102310

23112311
presets_text = """
@@ -3058,7 +3058,8 @@ class CConverter(metaclass=CConverterAutoRegister):
30583058
# Only set by self_converter.
30593059
signature_name: str | None = None
30603060

3061-
# Optional #include "name" // reason
3061+
# Optional (name, reason) include which generate a line like:
3062+
# "#include "name" // reason"
30623063
include: tuple[str, str] | None = None
30633064

30643065
# keep in sync with self_converter.__init__!
@@ -3366,7 +3367,7 @@ def parser_name(self) -> str:
33663367
return self.name
33673368

33683369
def add_include(self, name: str, reason: str) -> None:
3369-
if self.include:
3370+
if self.include is not None:
33703371
raise ValueError("a converter only supports a single include")
33713372
self.include = (name, reason)
33723373

0 commit comments

Comments
 (0)