mirrors/cpython
1
0
Fork 0
mirror of https://github.com/python/cpython.git synced 2025-08-04 00:48:58 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 8

don't memoize objects that are their own copies (closes #12422)

Browse source 
Patch mostly by Alex Gaynor.
This commit is contained in:
Benjamin Peterson 2011-06-27 16:22:46 -05:00
parent 31877c9d0e
commit e90ec366fb
3 changed files with 27 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

19
Lib/test/test_copy.py
View file

@ -321,9 +321,24 @@ class TestCopy(unittest.TestCase):
def test_deepcopy_keepalive(self):
memo = {}
x = 42
x = []
y = copy.deepcopy(x, memo)
self.assertTrue(memo[id(x)] is x)
self.assertIs(memo[id(memo)][0], x)
def test_deepcopy_dont_memo_immutable(self):
memo = {}
x = [1, 2, 3, 4]
y = copy.deepcopy(x, memo)
self.assertEqual(y, x)
# There's the entry for the new list, and the keep alive.
self.assertEqual(len(memo), 2)
memo = {}
x = [(1, 2)]
y = copy.deepcopy(x, memo)
self.assertEqual(y, x)
# Tuples with immutable contents are immutable for deepcopy.
self.assertEqual(len(memo), 2)
def test_deepcopy_inst_vanilla(self):
class C: