8000 gh-85795: Add support for `super()` in `NamedTuple` subclasses by bswck · Pull Request #129352 · python/cpython · GitHub
[go: up one dir, main page]
Skip to content

gh-85795: Add support for super() in NamedTuple subclasses #129352

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

Closed
wants to merge 16 commits into from
Closed

gh-85795: Add support for super() in NamedTuple subclasses #129352

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
e84d487
Add support for `super()` in `NamedTuple` subclasses
bswck Jan 27, 2025
577bdc7
Rewrite the test to be more neutral
bswck Jan 28, 2025
b77ea9f
Add a check that `__classcell__` isn't leaked
bswck Jan 28, 2025
31f6607
Separate leakage test and mark it as CPython-only
bswck Jan 28, 2025
74578e9
Shorten the test name, we know it's a namedtuple test from context
bswck Jan 28, 2025
7696b87
We test for dunder class too
bswck Jan 28, 2025
2f801dc
Language correctness
bswck Jan 28, 2025
f8cf62f
Minor aesthetical
bswck Jan 28, 2025
cc20a2d
Remove unintended attribute access
bswck Jan 28, 2025
efb9ad2
Encapsulate the `_classcell` parameter
bswck Jan 29, 2025
d9cdb27
Use a sentinel for missing `__classcell__`
bswck Jan 29, 2025
074df4c
Convert new documentation into a `versionchanged` block
bswck Jan 29, 2025
998474d
Add a comment discussing reasons for `__classcell__` manipulation
bswck Jan 29, 2025
ee93519
Reformat for optimal line length
bswck Jan 29, 2025
e3fc197
Add extra param `stack_offset` for fetching correct frames
bswck Jan 29, 2025
c2a0c50
Use `next` version ref instead of `3.14`
bswck Jan 29, 2025
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
Reformat for optimal line length 
Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>
  • Loading branch information
@bswck @AlexWaygood
bswck and AlexWaygood committed Jan 29, 2025
commit ee93519bccb4914b41ab2c3b51fc3813edce693d
3 changes: 2 additions & 1 deletion Lib/collections/__init__.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -360,7 +360,8 @@ def __ror__(self, other):

_nmtuple_classcell_sentinel = object()

def _namedtuple(typename, field_names, *, rename=False, defaults=None, module=None, classcell=_nmtuple_classcell_sentinel):
def _namedtuple(typename, field_names, *, rename=False, defaults=None, module=None,
classcell=_nmtuple_classcell_sentinel):
# Validate the field names. At the user's option, either generate an error
# message or automatically replace the field name with a valid name.
if isinstance(field_names, str):
Expand Down
3 changes: 2 additions & 1 deletion Lib/typing.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2929,7 +2929,8 @@ def __round__(self, ndigits: int = 0) -> T:
pass


def _make_nmtuple(name, fields, annotate_func, module, defaults = (), classcell=collections._nmtuple_classcell_sentinel):
def _make_nmtuple(name, fields, annotate_func, module, defaults = (),
classcell=collections._nmtuple_classcell_sentinel):
nm_tpl = collections._namedtuple(name, fields, defaults=defaults,
module=module, classcell=classcell)
nm_tpl.__annotate__ = nm_tpl.__new__.__annotate__ = annotate_func
Expand Down
Loading
0