Skip to content

Commit 68d499e

Browse files
committed
Address AA-Turner's coding style review
1 parent 830fb64 commit 68d499e

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
@@ -2175,7 +2175,7 @@ def print_block(
21752175
21762176
""")
21772177

2178-
if clinic:
2178+
if clinic is not None:
21792179
# Emit optional includes
21802180
for include, reason in sorted(clinic.includes.items()):
21812181
line = f'#include "{include}"'
@@ -2324,7 +2324,7 @@ def __init__(self, clinic: Clinic) -> None: ...
23242324
def parse(self, block: Block) -> None: ...
23252325

23262326

2327-
clinic : Clinic | None = None
2327+
clinic: Clinic | None = None
23282328
class Clinic:
23292329

23302330
presets_text = """
@@ -3088,7 +3088,8 @@ class CConverter(metaclass=CConverterAutoRegister):
30883088
# Only set by self_converter.
30893089
signature_name: str | None = None
30903090

3091-
# Optional #include "name" // reason
3091+
# Optional (name, reason) include which generate a line like:
3092+
# "#include "name" // reason"
30923093
include: tuple[str, str] | None = None
30933094

30943095
# keep in sync with self_converter.__init__!
@@ -3396,7 +3397,7 @@ def parser_name(self) -> str:
33963397
return self.name
33973398

33983399
def add_include(self, name: str, reason: str) -> None:
3399-
if self.include:
3400+
if self.include is not None:
34003401
raise ValueError("a converter only supports a single include")
34013402
self.include = (name, reason)
34023403

0 commit comments

Comments
 (0)