diff options
author | Tim Peters <tim.peters@gmail.com> | 2001-12-18 22:32:40 +0000 |
---|---|---|
committer | Tim Peters <tim.peters@gmail.com> | 2001-12-18 22:32:40 +0000 |
commit | 14a87cf9d03b0ce9f2244f69d2d3299e35e29fa6 (patch) | |
tree | b3a2a34c92428a28242bb01c734f65944f96cd2b /Lib/tempfile.py | |
parent | 1c8fde932793d05dff97a12299e239d9fdb4aeaf (diff) | |
download | cpython-14a87cf9d03b0ce9f2244f69d2d3299e35e29fa6.tar.gz |
TemporaryFileWrapper: cache the value of os.unlink for use by __del__,
to prevent mysterious errors at shutdown due to "os.unlink" turning into
"None.unlink".
Diffstat (limited to 'Lib/tempfile.py')
-rw-r--r-- | Lib/tempfile.py | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/Lib/tempfile.py b/Lib/tempfile.py index 96245d8c39..d4880637f0 100644 --- a/Lib/tempfile.py +++ b/Lib/tempfile.py @@ -128,6 +128,13 @@ class TemporaryFileWrapper: In particular, it seeks to automatically remove the file when it is no longer needed. """ + + # Cache the unlinker so we don't get spurious errors at shutdown + # when the module-level "os" in None'd out. Note that this must + # be referenced as self.unlink, because the name TemporaryFileWrapper + # may also get None'd out before __del__ is called. + unlink = os.unlink + def __init__(self, file, path): self.file = file self.path = path @@ -137,7 +144,7 @@ class TemporaryFileWrapper: if not self.close_called: self.close_called = 1 self.file.close() - os.unlink(self.path) + self.unlink(self.path) def __del__(self): self.close() |