diff options
author | Raymond Hettinger <rhettinger@users.noreply.github.com> | 2019-06-11 02:01:06 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-06-11 02:01:06 -0700 |
commit | 1b615b2f035d207941e44259a92ed0bdcc56ac45 (patch) | |
tree | b15f171a96c871413d5535b8ee1bbb2c6799a7ac | |
parent | c59b1bbbb37bbfc7eb7a9386bb51c7fea838fc2a (diff) | |
download | cpython-git-1b615b2f035d207941e44259a92ed0bdcc56ac45.tar.gz |
[3.7] bpo-37219: Remove erroneous optimization for differencing an empty set (GH-13965) (GH-13968)
-rw-r--r-- | Lib/test/test_set.py | 6 | ||||
-rw-r--r-- | Misc/NEWS.d/next/Core and Builtins/2019-06-11-01-37-34.bpo-37219.jPSufq.rst | 1 | ||||
-rw-r--r-- | Objects/setobject.c | 8 |
3 files changed, 7 insertions, 8 deletions
diff --git a/Lib/test/test_set.py b/Lib/test/test_set.py index bb1081f034..e4766ab190 100644 --- a/Lib/test/test_set.py +++ b/Lib/test/test_set.py @@ -895,6 +895,12 @@ class TestBasicOps: self.assertEqual(self.set, copy, "%s != %s" % (self.set, copy)) + def test_issue_37219(self): + with self.assertRaises(TypeError): + set().difference(123) + with self.assertRaises(TypeError): + set().difference_update(123) + #------------------------------------------------------------------------------ class TestBasicOpsEmpty(TestBasicOps, unittest.TestCase): diff --git a/Misc/NEWS.d/next/Core and Builtins/2019-06-11-01-37-34.bpo-37219.jPSufq.rst b/Misc/NEWS.d/next/Core and Builtins/2019-06-11-01-37-34.bpo-37219.jPSufq.rst new file mode 100644 index 0000000000..ef8f52dce6 --- /dev/null +++ b/Misc/NEWS.d/next/Core and Builtins/2019-06-11-01-37-34.bpo-37219.jPSufq.rst @@ -0,0 +1 @@ +Remove errorneous optimization for empty set differences. diff --git a/Objects/setobject.c b/Objects/setobject.c index 357e48ca3c..1e4781fe11 100644 --- a/Objects/setobject.c +++ b/Objects/setobject.c @@ -1478,10 +1478,6 @@ PyDoc_STRVAR(isdisjoint_doc, static int set_difference_update_internal(PySetObject *so, PyObject *other) { - if (PySet_GET_SIZE(so) == 0) { - return 0; - } - if ((PyObject *)so == other) return set_clear_internal(so); @@ -1556,10 +1552,6 @@ set_difference(PySetObject *so, PyObject *other) Py_ssize_t pos = 0, other_size; int rv; - if (PySet_GET_SIZE(so) == 0) { - return set_copy(so); - } - if (PyAnySet_Check(other)) { other_size = PySet_GET_SIZE(other); } |