[ruby-list:21055] Re: tempfile.rb BUG

なかだです。

At Sat, 26 Feb 2000 22:59:33 +0900,
Kazuhiro Nishiyama <[e-mail]> wrote:
> > > http://www.ne.jp/asahi/minazuki/bakera/html/reference/charrefhelp
> > > によるとUnicodeとみなしていいみたいです。
> >   ありがとうございます。あ、これって鳩丸?
> 
> ですです。

  そうか、移転したですか。

> これだけでは寂しかったので、Ruby本みてて気づいたtempfile.rbのバグ。
> 
> closeした後、openしたままだとファイルが残ります。
> 
> require 'tempfile'
> t = Tempfile.new('sample')
> t.close
> t.open
(中略)
> これがわからなかったので、close(true)の後openしたら
> 例外Errno::ENOENTが発生するのが仕様かどうかわからな
> かったんですが。

  undef_finalizer してしまうからですが、たしかに close(true) した
あとの open の振舞はちょっと中途半端に見えます。ここはやはり、
close(true) したあとはスッパリあきらめて新しい Tempfile を使うっ
てのがいいように思えますが、とりあえずパッチ。

  ところで 3引数の File.open を使おうかと思ったんですが、
USE_CWGUSI って何?

> それからTempfile#openにある__setobj__って何？

  delegate.rb が使う委譲先のオブジェクトを指定するメソッドです。
initialize の中でも super(@tmpfile) で同じことをしてます。


Index: tempfile.rb
===================================================================
RCS file: /home/cvs/ruby/lib/tempfile.rb,v
retrieving revision 1.2
diff -u -2 -p -r1.2 tempfile.rb
--- tempfile.rb	1999/08/13 05:45:18	1.2
+++ tempfile.rb	2000/02/26 16:22:03
@@ -53,4 +53,6 @@ Tempfile#initialize
       @tmpfile = File.open(tmpname, 'w+')
       @tmpname = tmpname
+      @basename = basename
+      @tmpdir = tmpdir
       super(@tmpfile)
       Dir.rmdir(lock)
@@ -66,6 +68,10 @@ Tempfile#open
   def open
     @tmpfile.close if @tmpfile
-    @tmpfile = File.open(@tmpname, 'r+')
-    __setobj__(@tmpfile)
+    if @tmpname
+      @tmpfile = File.open(@tmpname, 'r+')
+      __setobj__(@tmpfile)
+    else
+      initialize(@basename, @tmpfile) # re-initialize again
+    end
   end
 
@@ -74,4 +80,5 @@ Tempfile#close
     @tmpfile = nil
     if real
+      @tmpname = nil
       @clean_files.call
       ObjectSpace.undefine_finalizer(self)

-- 
そうだ 強気に ちょっと インチキに☆彡
    中田 "Bugるくらいがちょうどいいかも;-)" 伸悦