[#43412] [Ruby 1.9 - Feature #4592][Open] Tempfileを直接保存したい — Takeyuki Fujioka <xibbar@...>

15 messages 2011/04/21

[#43453] ThreadGroup の強化案 — Hidetoshi NAGAI <nagai@...>

永井@知能.九工大です.

14 messages 2011/04/28
[#43901] ThreadGroup#make_local_space! (Re: ThreadGroup の強化案) — Hidetoshi NAGAI <nagai@...> 2011/06/25

永井@知能.九工大です.少々長いメールで失礼します.

[#43920] Re: ThreadGroup#make_local_space! (Re: ThreadGroup の強化案) — Urabe Shyouhei <shyouhei@...> 2011/06/26

ひっじょうに根本的な質問をするのですけれども、ThreadGroupって何に使うん

[#43978] Re: ThreadGroup#make_local_space! — Hidetoshi NAGAI <nagai@...> 2011/06/28

永井@知能.九工大です.

[ruby-dev:43406] [Ruby 1.9 - Bug #4591][Open] (1.5...2).max #=> 1 (Range#max)

From: redmine@...
Date: 2011-04-20 15:42:24 UTC
List: ruby-dev #43406
Issue #4591 has been reported by Masaya Tarui.

----------------------------------------
Bug #4591: (1.5...2).max #=> 1 (Range#max)
http://redmine.ruby-lang.org/issues/4591

Author: Masaya Tarui
Status: Open
Priority: Normal
Assignee: 
Category: core
Target version: 
ruby -v: ruby 1.9.3dev (2011-04-20 trunk 31311) [i386-mswin32_100]



現在、(1.5...2).maxが1になります。
beginより小さい値が返ってくるのは違和感があります。

終端を含まないRangeについて、endがIntegerである時にend-1を返していますが、
それはbeginもIntegerである事が想定されてると思います。
結局 beginがIntegerじゃないときは、endがIntegerでない時と同様に
maxが定義できなさそうなので、やはり同様にErrorにした方が親切かと思います。

以下のパッチを入れていいですか?

Index: range.c
===================================================================
--- range.c     (リビジョン 31313)
+++ range.c     (作業コピー)
@@ -670,6 +670,9 @@
                rb_raise(rb_eTypeError, "cannot exclude non Integer end value");

            }
            if (c == 0) return Qnil;
+           if (!FIXNUM_P(b) && !rb_obj_is_kind_of(b,rb_cInteger)) {
+               rb_raise(rb_eTypeError, "cannot exclude end value with non Integer begin value");
+           }
            if (FIXNUM_P(e)) {
                return LONG2NUM(FIX2LONG(e) - 1);
            }
Index: test/ruby/test_range.rb
===================================================================
--- test/ruby/test_range.rb     (リビジョン 31313)
+++ test/ruby/test_range.rb     (作業コピー)
@@ -68,6 +68,8 @@
     assert_equal(2.0, (1.0..2.0).max)
     assert_equal(nil, (2.0..1.0).max)
     assert_raise(TypeError) { (1.0...2.0).max }
+    assert_raise(TypeError) { (1...1.5).max }
+    assert_raise(TypeError) { (1.5...2).max }

     assert_equal(-0x80000002, ((-0x80000002)...(-0x80000001)).max)



-- 
http://redmine.ruby-lang.org

In This Thread

Prev Next