https://github.com/python/cpython/commit/edc9d85c68af600431556f3e8edae9b0fbfdfd34
commit: edc9d85c68af600431556f3e8edae9b0fbfdfd34
branch: main
author: Nikita Sobolev <[email protected]>
committer: ericvsmith <[email protected]>
date: 2024-03-05T13:12:00-05:00
summary:
gh-109653: Just import `recursive_repr` in `dataclasses` (gh-109822)
files:
M Lib/dataclasses.py
diff --git a/Lib/dataclasses.py b/Lib/dataclasses.py
index 3335821f3f32e9..45ce5a98b51ae0 100644
--- a/Lib/dataclasses.py
+++ b/Lib/dataclasses.py
@@ -4,10 +4,9 @@
import types
import inspect
import keyword
-import functools
import itertools
import abc
-import _thread
+from reprlib import recursive_repr
from types import FunctionType, GenericAlias
@@ -245,25 +244,6 @@ def __repr__(self):
property,
})
-# This function's logic is copied from "recursive_repr" function in
-# reprlib module to avoid dependency.
-def _recursive_repr(user_function):
- # Decorator to make a repr function return "..." for a recursive
- # call.
- repr_running = set()
-
- @functools.wraps(user_function)
- def wrapper(self):
- key = id(self), _thread.get_ident()
- if key in repr_running:
- return '...'
- repr_running.add(key)
- try:
- result = user_function(self)
- finally:
- repr_running.discard(key)
- return result
- return wrapper
class InitVar:
__slots__ = ('type', )
@@ -322,7 +302,7 @@ def __init__(self, default, default_factory, init, repr,
hash, compare,
self.kw_only = kw_only
self._field_type = None
- @_recursive_repr
+ @recursive_repr()
def __repr__(self):
return ('Field('
f'name={self.name!r},'
@@ -632,7 +612,7 @@ def _repr_fn(fields, globals):
for f in fields]) +
')"'],
globals=globals)
- return _recursive_repr(fn)
+ return recursive_repr()(fn)
def _frozen_get_del_attr(cls, fields, globals):
_______________________________________________
Python-checkins mailing list -- [email protected]
To unsubscribe send an email to [email protected]
https://mail.python.org/mailman3/lists/python-checkins.python.org/
Member address: [email protected]
