[#12164] patch for ext/gdbm — Koji Arai <JCA02266@...>

新井です。

24 messages 2001/02/04
[#12168] Re: patch for ext/gdbm — matz@... (Yukihiro Matsumoto) 2001/02/05

まつもと ゆきひろです

[#12176] Re: patch for ext/gdbm — Koji Arai <JCA02266@...> 2001/02/05

新井です。

[#12179] Re: patch for ext/gdbm — matz@... (Yukihiro Matsumoto) 2001/02/06

まつもと ゆきひろです

[#12219] Re: patch for ext/gdbm — Koji Arai <JCA02266@...> 2001/02/12

新井です。

[#12220] Re: patch for ext/gdbm — Koji Arai <JCA02266@...> 2001/02/12

新井です。

[#12256] set_trace_func — keiju@... (Keiju ISHITSUKA)

けいじゅ@日本ラショナルソフトウェアです.

15 messages 2001/02/17

[#12293] crash on proc without a block — Kenichi Komiya <kom@...1.accsnet.ne.jp>

15 messages 2001/02/25

[#12323] Re: [ruby-list:28364] class definition extension — "K.Kosako" <kosako@...>

ruby-listから移動しました。

13 messages 2001/02/28
[#12324] Re: [ruby-list:28364] class definition extension — matz@... (Yukihiro Matsumoto) 2001/02/28

まつもと ゆきひろです

[ruby-dev:12242] CGI::Session#close

From: Shugo Maeda <shugo@...>
Date: 2001-02-14 09:54:54 UTC
List: ruby-dev #12242
前田です。

CGI::Session#closeなんですが、条件が逆じゃないでしょうか。
CGI::Session#deleteの方は、closeされていたら、どうせ、

path = @f.path

の部分でエラーになる(deleteを呼んだのに消さずにreturnするよりもエ
ラーになった方がよいと思います)ので外してみました。

commitしてもよろしいでしょうか?


--- session.rb	2001/02/14 05:53:58	1.6.2.3
+++ session.rb	2001/02/14 09:49:29
@@ -134,14 +134,13 @@
       end
 
       def close
-	return unless @f.closed?
+	return if @f.closed?
 	update
 	@f.close
       end
 
       def delete
 	path = @f.path
-	return unless @f.closed?
 	@f.close
 	File::unlink path
       end


-- 
前田 修吾

In This Thread

Prev Next