ensure .keywords is always a dict
diff --git a/Lib/test/test_functools.py b/Lib/test/test_functools.py
index c0a9a3b..7b3cb96 100644
--- a/Lib/test/test_functools.py
+++ b/Lib/test/test_functools.py
@@ -89,9 +89,11 @@
# exercise special code paths for no keyword args in
# either the partial object or the caller
p = self.thetype(capture)
+ self.assertEqual(p.keywords, {})
self.assertEqual(p(), ((), {}))
self.assertEqual(p(a=1), ((), {'a':1}))
p = self.thetype(capture, a=1)
+ self.assertEqual(p.keywords, {'a':1})
self.assertEqual(p(), ((), {'a':1}))
self.assertEqual(p(b=2), ((), {'a':1, 'b':2}))
# keyword args in the call override those in the partial object
diff --git a/Misc/NEWS b/Misc/NEWS
index e76f187..b5a149c 100644
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -27,6 +27,8 @@
Library
-------
+- The keywords attribute of functools.partial is now always a dictionary.
+
- Issue #24134: assertRaises() and assertRaisesRegexp() checks are not longer
successful if the callable is None.
diff --git a/Modules/_functoolsmodule.c b/Modules/_functoolsmodule.c
index 6397ba9..2ba92ba 100644
--- a/Modules/_functoolsmodule.c
+++ b/Modules/_functoolsmodule.c
@@ -132,17 +132,13 @@
Py_DECREF(pto);
return NULL;
}
- if (kw != NULL) {
- pto->kw = PyDict_Copy(kw);
- if (pto->kw == NULL) {
- Py_DECREF(pto);
- return NULL;
- }
- } else {
- pto->kw = Py_None;
- Py_INCREF(Py_None);
+ pto->kw = (kw != NULL) ? PyDict_Copy(kw) : PyDict_New();
+ if (pto->kw == NULL) {
+ Py_DECREF(pto);
+ return NULL;
}
+
pto->weakreflist = NULL;
pto->dict = NULL;