From: redmine@...
Date: 2011-04-21T00:42:24+09:00
Subject: [ruby-dev:43406] [Ruby 1.9 - Bug #4591][Open] (1.5...2).max #=> 1 (Range#max)


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