From a255a72f0a5d33c1d28547dfb2f7d320a2ca1773 Mon Sep 17 00:00:00 2001 From: Tim Peters Date: Tue, 18 Dec 2001 22:32:40 +0000 Subject: [PATCH] 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". --- Lib/tempfile.py | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) 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() -- 2.50.1