From a43eeaef7529f979eb17276cd04bd87fabb25968 Mon Sep 17 00:00:00 2001 From: matz Date: Wed, 24 Mar 2004 01:08:20 +0000 Subject: [PATCH] * lib/tempfile.rb (Tempfile::_close): should not clear @tmpname until the file is really removed. [ruby-core:02684] git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@6007 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ChangeLog | 5 +++++ lib/tempfile.rb | 11 +++++++---- 2 files changed, 12 insertions(+), 4 deletions(-) diff --git a/ChangeLog b/ChangeLog index 2280aeb154..00b2b13d04 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +Wed Mar 24 10:05:22 2004 Yukihiro Matsumoto + + * lib/tempfile.rb (Tempfile::_close): should not clear @tmpname + until the file is really removed. [ruby-core:02684] + Wed Mar 24 04:12:44 2004 Yukihiro Matsumoto * object.c (rb_mod_cvar_get): new method Module#class_variable_get. diff --git a/lib/tempfile.rb b/lib/tempfile.rb index 2169a20ec5..8ebabc9ea9 100644 --- a/lib/tempfile.rb +++ b/lib/tempfile.rb @@ -76,12 +76,13 @@ class Tempfile < SimpleDelegator def _close # :nodoc: @tmpfile.close if @tmpfile - @data[1] = @tmpfile = nil - @data = @tmpname = nil + @tmpfile = nil + @data[1] = nil if @data + @data = nil end protected :_close - # Closes the file. If the optional flag is true, unlinks the file + #Closes the file. If the optional flag is true, unlinks the file # after closing. # # If you don't explicitly unlink the temporary file, the removal @@ -99,6 +100,7 @@ class Tempfile < SimpleDelegator _close @clean_proc.call ObjectSpace.undefine_finalizer(self) + @tmpname = nil end # Unlinks the file. On UNIX-like systems, it is often a good idea @@ -108,7 +110,8 @@ class Tempfile < SimpleDelegator def unlink # keep this order for thread safeness File.unlink(@tmpname) if File.exist?(@tmpname) - @@cleanlist.delete(@tmpname) if @@cleanlist + @@cleanlist.delete(@tmpname) + @tmpname = nil end alias delete unlink