[Python-checkins] cpython (3.4): ensure .keywords is always a dict

Sat May 9 06:29:12 CEST 2015

https://hg.python.org/cpython/rev/9dfda4e7169a
changeset: 95929:9dfda4e7169a
branch: 3.4
parent: 95926:c78137bff841
user: Benjamin Peterson <benjamin at python.org>
date: Sat May 09 00:25:18 2015 -0400
summary:
 ensure .keywords is always a dict
files:
 Lib/test/test_functools.py | 2 ++
 Misc/NEWS | 2 ++
 Modules/_functoolsmodule.c | 14 +++++---------
 3 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/Lib/test/test_functools.py b/Lib/test/test_functools.py
--- a/Lib/test/test_functools.py
+++ b/Lib/test/test_functools.py
@@ -77,9 +77,11 @@
 # exercise special code paths for no keyword args in
 # either the partial object or the caller
 p = self.partial(capture)
+ self.assertEqual(p.keywords, {})
 self.assertEqual(p(), ((), {}))
 self.assertEqual(p(a=1), ((), {'a':1}))
 p = self.partial(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
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -57,6 +57,8 @@
 - Issue #9246: On POSIX, os.getcwd() now supports paths longer than 1025 bytes.
 Patch written by William Orr.
 
+- The keywords attribute of functools.partial is now always a dictionary.
+
 - Issues #24099, #24100, and #24101: Fix free-after-use bug in heapq's siftup
 and siftdown functions.
 
diff --git a/Modules/_functoolsmodule.c b/Modules/_functoolsmodule.c
--- a/Modules/_functoolsmodule.c
+++ b/Modules/_functoolsmodule.c
@@ -54,17 +54,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;
 
-- 
Repository URL: https://hg.python.org/cpython
