summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorVictor Stinner <victor.stinner@gmail.com>2016-03-19 01:01:48 (GMT)
committerVictor Stinner <victor.stinner@gmail.com>2016-03-19 01:01:48 (GMT)
commitf664dc58344fc615f8f5649752bf5cb54efe5b3b (patch)
tree33168a0657832d06ceb69098b13108ffa93046ef
parenteedf13fe231b49b10c9a836ae8c9120d215deb84 (diff)
downloadcpython-f664dc58344fc615f8f5649752bf5cb54efe5b3b.zip
cpython-f664dc58344fc615f8f5649752bf5cb54efe5b3b.tar.gz
cpython-f664dc58344fc615f8f5649752bf5cb54efe5b3b.tar.bz2
ResourceWarning: Revert change on socket and scandir
io.FileIO has a safe implementation of destructor, but not socket nor scandir.
-rw-r--r--Modules/posixmodule.c4
-rw-r--r--Modules/socketmodule.c3
2 files changed, 4 insertions, 3 deletions
diff --git a/Modules/posixmodule.c b/Modules/posixmodule.c
index 3f22d14..65b20be 100644
--- a/Modules/posixmodule.c
+++ b/Modules/posixmodule.c
@@ -12111,8 +12111,8 @@ ScandirIterator_dealloc(ScandirIterator *iterator)
*/
++Py_REFCNT(iterator);
PyErr_Fetch(&exc, &val, &tb);
- if (PyErr_ResourceWarning((PyObject *)iterator, 1,
- "unclosed scandir iterator %R", iterator)) {
+ if (PyErr_WarnFormat(PyExc_ResourceWarning, 1,
+ "unclosed scandir iterator %R", iterator)) {
/* Spurious errors can appear at shutdown */
if (PyErr_ExceptionMatches(PyExc_Warning))
PyErr_WriteUnraisable((PyObject *) iterator);
diff --git a/Modules/socketmodule.c b/Modules/socketmodule.c
index 657b04b..77a6b31 100644
--- a/Modules/socketmodule.c
+++ b/Modules/socketmodule.c
@@ -4170,7 +4170,8 @@ sock_dealloc(PySocketSockObject *s)
Py_ssize_t old_refcount = Py_REFCNT(s);
++Py_REFCNT(s);
PyErr_Fetch(&exc, &val, &tb);
- if (PyErr_ResourceWarning(s, 1, "unclosed %R", s))
+ if (PyErr_WarnFormat(PyExc_ResourceWarning, 1,
+ "unclosed %R", s))
/* Spurious errors can appear at shutdown */
if (PyErr_ExceptionMatches(PyExc_Warning))
PyErr_WriteUnraisable((PyObject *) s);