diff --git a/ChangeLog b/ChangeLog index 957ddbb141..a09985f13b 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +Sat Mar 19 21:53:35 2016 Nobuyoshi Nakada + + * numeric.c (fix_cmp): invert the result as the comarison is + inverted. + Sat Mar 19 18:32:00 2016 Kenta Murata * numeric.c (int_to_f): raise NotImplementedError when a receiver diff --git a/numeric.c b/numeric.c index 53af5c5666..38e05b6292 100644 --- a/numeric.c +++ b/numeric.c @@ -3425,7 +3425,12 @@ fix_cmp(VALUE x, VALUE y) return INT2FIX(-1); } else if (RB_TYPE_P(y, T_BIGNUM)) { - return rb_big_cmp(y, x); + VALUE cmp = rb_big_cmp(y, x); + switch (cmp) { + case INT2FIX(+1): return INT2FIX(-1); + case INT2FIX(-1): return INT2FIX(+1); + } + return cmp; } else if (RB_TYPE_P(y, T_FLOAT)) { return rb_integer_float_cmp(x, y); diff --git a/test/ruby/test_fixnum.rb b/test/ruby/test_fixnum.rb index 024f905a43..3a1c2fa8bc 100644 --- a/test/ruby/test_fixnum.rb +++ b/test/ruby/test_fixnum.rb @@ -214,6 +214,7 @@ class TestFixnum < Test::Unit::TestCase assert_equal(0, 1 <=> 1) assert_equal(-1, 1 <=> 4294967296) + assert_equal(-1, 1 <=> 1 << 100) assert_equal(0, 1 <=> 1.0) assert_nil(1 <=> nil)