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 1 commit
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
Prev Previous commit
Next Next commit
Make sure str is used as key in parameter dict
  • Loading branch information
@erlend-aasland
erlend-aasland committed Jul 3, 2023
commit d694f043ae25308dda607fad4ea1d8d85d79982f
2 changes: 1 addition & 1 deletion Tools/clinic/clinic.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4653,7 +4653,7 @@ def state_modulename_name(self, line: str | None) -> None:
kwargs['type'] = cls.typedef
sc = self.function.self_converter = self_converter(name, name, self.function, **kwargs)
p_self = Parameter(name, inspect.Parameter.POSITIONAL_ONLY, function=self.function, converter=sc)
self.function.parameters[sc.name] = p_self
self.function.parameters[name] = p_self

(cls or module).functions.append(self.function)
self.next(self.state_parameters_start)
Expand Down