changeset: 99395:962086677306 parent: 99391:6b08429a3932 parent: 99394:fecb07050aae user: Serhiy Storchaka date: Mon Nov 30 17:37:13 2015 +0200 files: Misc/NEWS description: Issue #25718: Fixed copying object with state with boolean value is false. diff -r 6b08429a3932 -r 962086677306 Lib/copy.py --- a/Lib/copy.py Mon Nov 30 03:18:29 2015 +0000 +++ b/Lib/copy.py Mon Nov 30 17:37:13 2015 +0200 @@ -279,7 +279,7 @@ if n > 2: state = info[2] else: - state = {} + state = None if n > 3: listiter = info[3] else: @@ -293,7 +293,7 @@ y = callable(*args) memo[id(x)] = y - if state: + if state is not None: if deep: state = deepcopy(state, memo) if hasattr(y, '__setstate__'): diff -r 6b08429a3932 -r 962086677306 Lib/test/test_copy.py --- a/Lib/test/test_copy.py Mon Nov 30 03:18:29 2015 +0000 +++ b/Lib/test/test_copy.py Mon Nov 30 17:37:13 2015 +0200 @@ -213,6 +213,9 @@ return self.foo == other.foo x = C(42) self.assertEqual(copy.copy(x), x) + # State with boolean value is false (issue #25718) + x = C(0.0) + self.assertEqual(copy.copy(x), x) # The deepcopy() method @@ -517,6 +520,12 @@ self.assertEqual(y, x) self.assertIsNot(y, x) self.assertIsNot(y.foo, x.foo) + # State with boolean value is false (issue #25718) + x = C([]) + y = copy.deepcopy(x) + self.assertEqual(y, x) + self.assertIsNot(y, x) + self.assertIsNot(y.foo, x.foo) def test_deepcopy_reflexive_inst(self): class C: diff -r 6b08429a3932 -r 962086677306 Misc/NEWS --- a/Misc/NEWS Mon Nov 30 03:18:29 2015 +0000 +++ b/Misc/NEWS Mon Nov 30 17:37:13 2015 +0200 @@ -107,6 +107,8 @@ Library ------- +- Issue #25718: Fixed copying object with state with boolean value is false. + - Issue #10131: Fixed deep copying of minidom documents. Based on patch by Marian Ganisin.