Skip to content

gh-114053: Fix another edge case involving get_type_hints, PEP 695 and PEP 563 #120272

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
AlexWaygood merged 10 commits into from
Jun 25, 2024
Merged

gh-114053: Fix another edge case involving get_type_hints, PEP 695 and PEP 563 #120272

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
879abeb
gh-114053: Fix another edge case involving `get_type_hints`, PEP 695 …
AlexWaygood Jun 8, 2024
065914e
Merge branch 'main' into gettypehints-bug
AlexWaygood Jun 9, 2024
373b23c
Add failing test
AlexWaygood Jun 9, 2024
4f95b1c
Fix failing test
AlexWaygood Jun 9, 2024
ffdba91
remove redundant assertions in test
AlexWaygood Jun 9, 2024
5505d97
Don't mutate globals!
AlexWaygood Jun 9, 2024
b05ca1a
Merge branch 'gettypehints-bug' of https://github.com/AlexWaygood/cpy…
AlexWaygood Jun 9, 2024
bccaea2
simplify
AlexWaygood Jun 9, 2024
2fbcd7b
Fix yet another bug and add many more tests
AlexWaygood Jun 9, 2024
432e749
Merge branch 'main' into gettypehints-bug
AlexWaygood Jun 25, 2024
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
simplify
  • Loading branch information
@AlexWaygood
AlexWaygood committed Jun 9, 2024
commit bccaea2f237699fd3be6db9ab4646ac61e28f28a
12 changes: 6 additions & 6 deletions Lib/typing.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1069,15 +1069,15 @@ def _evaluate(self, globalns, localns, type_params=_sentinel, *, recursive_guard
# but should in turn be overridden by names in the class scope
# (which here are called `globalns`!)
if type_params:
name_to_param_map = {param.__name__: param for param in type_params}
if self.__forward_is_class__:
globalns, localns = dict(globalns), dict(localns)
for name, param in name_to_param_map.items():
if name not in globalns:
globalns[name] = param
localns.pop(name, None)
for param in type_params:
param_name = param.__name__
if param_name not in globalns:
Comment thread
AlexWaygood marked this conversation as resolved.
Outdated
globalns[param_name] = param
localns.pop(param_name, None)
else:
localns = name_to_param_map | localns
localns = {param.__name__: param for param in type_params} | localns

type_ = _type_check(
eval(self.__forward_code__, globalns, localns),
Expand Down