summaryrefslogtreecommitdiffstats
path: root/Lib/tempfile.py
diff options
context:
space:
mode:
authorSerhiy Storchaka <storchaka@gmail.com>2014-09-24 10:26:25 (GMT)
committerSerhiy Storchaka <storchaka@gmail.com>2014-09-24 10:26:25 (GMT)
commit5e193ac0bd6bd4d4a122fd9aa2446ee70e5f2bf1 (patch)
tree32b9c686d3e209c51c3d3a0fc812e7a6ebc8ddd1 /Lib/tempfile.py
parentb87630c273ab3ff68d4dbee61fe7ec9c65eeccb8 (diff)
downloadcpython-5e193ac0bd6bd4d4a122fd9aa2446ee70e5f2bf1.zip
cpython-5e193ac0bd6bd4d4a122fd9aa2446ee70e5f2bf1.tar.gz
cpython-5e193ac0bd6bd4d4a122fd9aa2446ee70e5f2bf1.tar.bz2
Issue #22427: TemporaryDirectory no longer attempts to clean up twice when
used in the with statement in generator.
Diffstat (limited to 'Lib/tempfile.py')
-rw-r--r--Lib/tempfile.py15
1 files changed, 3 insertions, 12 deletions
diff --git a/Lib/tempfile.py b/Lib/tempfile.py
index f0e25fc..35fd8b4 100644
--- a/Lib/tempfile.py
+++ b/Lib/tempfile.py
@@ -663,11 +663,6 @@ class TemporaryDirectory(object):
in it are removed.
"""
- # Handle mkdtemp raising an exception
- name = None
- _finalizer = None
- _closed = False
-
def __init__(self, suffix="", prefix=template, dir=None):
self.name = mkdtemp(suffix, prefix, dir)
self._finalizer = _weakref.finalize(
@@ -675,10 +670,9 @@ class TemporaryDirectory(object):
warn_message="Implicitly cleaning up {!r}".format(self))
@classmethod
- def _cleanup(cls, name, warn_message=None):
+ def _cleanup(cls, name, warn_message):
_shutil.rmtree(name)
- if warn_message is not None:
- _warnings.warn(warn_message, ResourceWarning)
+ _warnings.warn(warn_message, ResourceWarning)
def __repr__(self):
@@ -691,8 +685,5 @@ class TemporaryDirectory(object):
self.cleanup()
def cleanup(self):
- if self._finalizer is not None:
- self._finalizer.detach()
- if self.name is not None and not self._closed:
+ if self._finalizer.detach():
_shutil.rmtree(self.name)
- self._closed = True