From: Kenta Murata <muraken@...>
Date: 2011-07-08T17:56:01+09:00
Subject: [ruby-dev:44059] [Ruby 1.9 - Bug #4591] (1.5...2).max #=> 1 (Range#max)


Issue #4591 has been updated by Kenta Murata.


> 今回のbug報告はこの時の変更に追従して変更すべきだったのを
> 変更してなかったんではないかというバグ報告として挙げていて、
> maxとはどうあるべきかまでは踏み込んでないつもりです。
> 
> そういう議論をするならどっちかというと、#4577ですかね。。

この issue が max の定義についての議論ではなく、
以前に報告された bug への対応漏れという認識で理解しました。

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

Author: Masaya Tarui
Status: Assigned
Priority: Normal
Assignee: Kenta Murata
Category: core
Target version: 1.9.3
ruby -v: -


=begin

現在、(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)

=end



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