bpo-38210: Fix intersection operation with dict view and iterator. (GH-16602)
diff --git a/Lib/test/test_dictviews.py b/Lib/test/test_dictviews.py
index b15cfeb..7cf0192 100644
--- a/Lib/test/test_dictviews.py
+++ b/Lib/test/test_dictviews.py
@@ -214,6 +214,17 @@
self.assertTrue(de.items().isdisjoint(de.items()))
self.assertTrue(de.items().isdisjoint([1]))
+ def test_set_operations_with_iterator(self):
+ origin = {1: 2, 3: 4}
+ self.assertEqual(origin.keys() & iter([1, 2]), {1})
+ self.assertEqual(origin.keys() | iter([1, 2]), {1, 2, 3})
+ self.assertEqual(origin.keys() ^ iter([1, 2]), {2, 3})
+
+ items = origin.items()
+ self.assertEqual(items & iter([(1, 2)]), {(1, 2)})
+ self.assertEqual(items ^ iter([(1, 2)]), {(3, 4)})
+ self.assertEqual(items | iter([(1, 2)]), {(1, 2), (3, 4)})
+
def test_recursive_repr(self):
d = {}
d[42] = d.values()
diff --git a/Misc/NEWS.d/next/Core and Builtins/2019-10-06-15-01-57.bpo-38210.Xgc6F_.rst b/Misc/NEWS.d/next/Core and Builtins/2019-10-06-15-01-57.bpo-38210.Xgc6F_.rst
new file mode 100644
index 0000000..768c6d4
--- /dev/null
+++ b/Misc/NEWS.d/next/Core and Builtins/2019-10-06-15-01-57.bpo-38210.Xgc6F_.rst
@@ -0,0 +1,2 @@
+Remove unecessary intersection and update set operation in dictview with
+empty set. (Contributed by Dong-hee Na in :issue:`38210`.)
diff --git a/Objects/dictobject.c b/Objects/dictobject.c
index 2f86946..5c3f1fb 100644
--- a/Objects/dictobject.c
+++ b/Objects/dictobject.c
@@ -4225,14 +4225,6 @@
it = PyObject_GetIter(other);
- _Py_IDENTIFIER(intersection_update);
- tmp = _PyObject_CallMethodIdOneArg(result, &PyId_intersection_update, other);
- if (tmp == NULL) {
- Py_DECREF(result);
- return NULL;
- }
- Py_DECREF(tmp);
-
if (PyDictKeys_Check(self)) {
dict_contains = dictkeys_contains;
}