Project

General

Profile

Bug #4591

(1.5...2).max #=> 1 (Range#max)

Added by tarui (Masaya Tarui) over 9 years ago. Updated about 9 years ago.

Status:
Closed
Priority:
Normal
Target version:
ruby -v:
-
Backport:
[ruby-dev:43406]

Description

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


Related issues

Related to Ruby master - Bug #974: Range#max で終了しないことがあるClosed01/04/2009Actions
Related to Ruby master - Bug #4577: (int...float).max should not raise an errorRejectedmatz (Yukihiro Matsumoto)04/14/2011Actions

Also available in: Atom PDF