Skip to content

Commit

Permalink
Address AA-Turner's coding style review
Browse files Browse the repository at this point in the history
  • Loading branch information
vstinner committed Aug 25, 2023
1 parent cdadfa1 commit 900a002
Showing 1 changed file with 5 additions and 4 deletions.
9 changes: 5 additions & 4 deletions Tools/clinic/clinic.py
Original file line number Diff line number Diff line change
Expand Up @@ -2156,7 +2156,7 @@ def print_block(

""")

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


clinic : Clinic | None = None
clinic: Clinic | None = None
class Clinic:

presets_text = """
Expand Down Expand Up @@ -3058,7 +3058,8 @@ class CConverter(metaclass=CConverterAutoRegister):
# Only set by self_converter.
signature_name: str | None = None

# Optional #include "name" // reason
# Optional (name, reason) include which generate a line like:
# "#include "name" // reason"
include: tuple[str, str] | None = None

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

def add_include(self, name: str, reason: str) -> None:
if self.include:
if self.include is not None:
raise ValueError("a converter only supports a single include")
self.include = (name, reason)

Expand Down

0 comments on commit 900a002

Please sign in to comment.