Commit a8377182 authored by Raymond Hettinger's avatar Raymond Hettinger

Issue #9825: Remove __del__() from OrderedDict.

parent d7ffbd0d
...@@ -173,9 +173,6 @@ class OrderedDict(dict, MutableMapping): ...@@ -173,9 +173,6 @@ class OrderedDict(dict, MutableMapping):
all(_imap(_eq, self.iteritems(), other.iteritems())) all(_imap(_eq, self.iteritems(), other.iteritems()))
return dict.__eq__(self, other) return dict.__eq__(self, other)
def __del__(self):
self.clear() # eliminate cyclical references
################################################################################ ################################################################################
### namedtuple ### namedtuple
......
...@@ -43,6 +43,13 @@ Core and Builtins ...@@ -43,6 +43,13 @@ Core and Builtins
Library Library
------- -------
- Issue #9825: removed __del__ from the definition of collections.OrderedDict.
This prevents user-created self-referencing ordered dictionaries from
becoming permanently uncollectable GC garbage. The downside is that
removing __del__ means that the internal doubly-linked list has to wait for
GC collection rather than freeing memory immediately when the refcnt drops
to zero.
- Issue #9816: random.Random.jumpahead(n) did not produce a sufficiently - Issue #9816: random.Random.jumpahead(n) did not produce a sufficiently
different internal state for small values of n. Fixed by salting the different internal state for small values of n. Fixed by salting the
value. value.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment