summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRaymond Hettinger <python@rcn.com>2010-10-30 08:17:46 +0000
committerRaymond Hettinger <python@rcn.com>2010-10-30 08:17:46 +0000
commit2ad17e19eec7f873005a215a00c13520c679d4fb (patch)
tree5afb2827106033f8bdb24c518775c6c20b0242d5
parentcbba8d4c7a5b77ef60c088b5070b919b32a1b861 (diff)
downloadcpython-git-2ad17e19eec7f873005a215a00c13520c679d4fb.tar.gz
Issue 10221: Improve error message for dict.pop().
-rw-r--r--Misc/NEWS3
-rw-r--r--Objects/dictobject.c3
2 files changed, 4 insertions, 2 deletions
diff --git a/Misc/NEWS b/Misc/NEWS
index eeff4a7760..dc5dca453f 100644
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -10,6 +10,9 @@ What's New in Python 2.7.1?
Core and Builtins
-----------------
+- Issue #10221: dict.pop(k) now has a key error message that includes the
+ missing key (same message d[k] returns for missing keys).
+
- Issue #10125: Don't segfault when the iterator passed to ``file.writelines()``
closes the file.
diff --git a/Objects/dictobject.c b/Objects/dictobject.c
index def3da9af0..3670e974ad 100644
--- a/Objects/dictobject.c
+++ b/Objects/dictobject.c
@@ -1980,8 +1980,7 @@ dict_pop(PyDictObject *mp, PyObject *args)
Py_INCREF(deflt);
return deflt;
}
- PyErr_SetString(PyExc_KeyError,
- "pop(): dictionary is empty");
+ set_key_error(key);
return NULL;
}
if (!PyString_CheckExact(key) ||