diff options
author | tarui <tarui@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2011-07-10 00:47:31 +0000 |
---|---|---|
committer | tarui <tarui@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2011-07-10 00:47:31 +0000 |
commit | d22e08f9375dbc0d0a962751d1c57f17b22f01b6 (patch) | |
tree | ba23d24361d4967fc78249a44f606d0b75bba4df | |
parent | b1d647c809a86b8068a327ce2398c89c3823dc61 (diff) |
* range.c (range_max): fix behavior with excluded end value.
[Bug #4591]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@32482 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 5 | ||||
-rw-r--r-- | range.c | 3 | ||||
-rw-r--r-- | test/ruby/test_range.rb | 2 |
3 files changed, 10 insertions, 0 deletions
@@ -1,3 +1,8 @@ +Sun Jul 10 09:46:14 2011 Masaya Tarui <tarui@ruby-lnag.org> + + * range.c (range_max): fix behavior with excluded end value. + [Bug #4591] + Sun Jul 10 09:13:18 2011 Eric Hodel <drbrain@segment7.net> * NEWS: Fix RubyGems version. [Ruby 1.9 - Bug #5004] @@ -665,6 +665,9 @@ range_max(VALUE range) 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); } diff --git a/test/ruby/test_range.rb b/test/ruby/test_range.rb index d45df8f820..738c7cdb85 100644 --- a/test/ruby/test_range.rb +++ b/test/ruby/test_range.rb @@ -68,6 +68,8 @@ class TestRange < Test::Unit::TestCase 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) |