Skip to content

gh-104050: Add more type hints to Argument Clinic DSLParser() #106354

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 19 commits into from
Jul 3, 2023
Merged

gh-104050: Add more type hints to Argument Clinic DSLParser() #106354

Changes from 3 commits
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
15d4b03
gh-104050: Add more type hints to Argument Clinic DSLParser()
erlend-aasland Jul 3, 2023
4744bd9
Fix shadowing post partition
erlend-aasland Jul 3, 2023
7df949b
Tighten up parse_converter() return annotation
erlend-aasland Jul 3, 2023
7667704
Loosen up state_parameters_start()
erlend-aasland Jul 3, 2023
0480ed2
ast.expr is not str
erlend-aasland Jul 3, 2023
ec7c1d8
Annotate Parameter 'kind' param as inspect._ParameterKind
erlend-aasland Jul 3, 2023
ec38c62
Annotate DSLParser 'function' attr as 'Function | None'
erlend-aasland Jul 3, 2023
b1bd72a
Make sure Parameter() gets name as str
erlend-aasland Jul 3, 2023
3f1a0c0
Function param self_converter is CConverter or None
erlend-aasland Jul 3, 2023
d694f04
Make sure str is used as key in parameter dict
erlend-aasland Jul 3, 2023
f116de8
Merge branch 'main' into clinic/even-more-typing
AlexWaygood Jul 3, 2023
e587ba4
Annotation for c_basename
erlend-aasland Jul 3, 2023
cd5d675
More accurate type for self.self_converter
erlend-aasland Jul 3, 2023
f25dd99
Address review: fix return_converter type
erlend-aasland Jul 3, 2023
dcd5d32
from __future__ imports must occur at the beginning of the file!
erlend-aasland Jul 3, 2023
4c3844e
Pull in main
erlend-aasland Jul 3, 2023
89ba85a
Pull in main
erlend-aasland Jul 3, 2023
140bafc
Wrap touched lines at 79 chars
erlend-aasland Jul 3, 2023
4a7edf9
Apply suggestions from code review
erlend-aasland Jul 3, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 7 additions & 6 deletions Tools/clinic/clinic.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4526,7 +4526,7 @@ def state_dsl_start(self, line: str | None) -> None:

self.next(self.state_modulename_name, line)

def state_modulename_name(self, line):
def state_modulename_name(self, line: str | None) -> None:
# looking for declaration, which establishes the leftmost column
# line should be
# modulename.fnname [as c_basename] [-> return annotation]
Expand All @@ -4543,7 +4543,7 @@ def state_modulename_name(self, line):
# this line is permitted to start with whitespace.
# we'll call this number of spaces F (for "function").

if not line.strip():
if not self.valid_line(line):
return

self.indent.infer(line)
Expand Down Expand Up @@ -4588,9 +4588,9 @@ def state_modulename_name(self, line):

line, _, returns = line.partition('->')

full_name, _, c_basename = line.partition(' as ')
full_name = full_name.strip()
c_basename = c_basename.strip() or None
left, _, right = line.partition(' as ')
full_name = left.strip()
c_basename = right.strip() or None

if not is_legal_py_identifier(full_name):
fail("Illegal function name:", full_name)
Expand Down Expand Up @@ -5021,7 +5021,7 @@ def bad_node(self, node):
key = f"{parameter_name}_as_{c_name}" if c_name else parameter_name
self.function.parameters[key] = p

KwargDict = dict[str | None, Any]
KwargDict = dict[str, Any]

@staticmethod
def parse_converter(annotation: ast.expr | None) -> tuple[str, bool, KwargDict]:
Expand All @@ -5031,6 +5031,7 @@ def parse_converter(annotation: ast.expr | None) -> tuple[str, bool, KwargDict]:
case ast.Name(name):
return name, False, {}
case ast.Call(func=ast.Name(name)):
assert isinstance(ast.expr, str)
symbols = globals()
kwargs = {
node.arg: eval_ast_expr(node.value, symbols)
Expand Down