diff options
author | matz <matz@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2002-08-19 06:41:02 +0000 |
---|---|---|
committer | matz <matz@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2002-08-19 06:41:02 +0000 |
commit | b818cdfaf390e0d721bb3e3809280b35fbcf1985 (patch) | |
tree | 23d9aad3d0aae7211c6d278aa986a05d1ccaf95a | |
parent | ae23000c0e0e4f4af1b4462147d950549b3abdbe (diff) | |
download | ruby-b818cdfaf390e0d721bb3e3809280b35fbcf1985.tar.gz |
* array.c (sort_2): comparison should be done as signed long.
* array.c (sort_2): should return int, not VALUE.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@2722 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 6 | ||||
-rw-r--r-- | array.c | 8 |
2 files changed, 10 insertions, 4 deletions
@@ -1,3 +1,9 @@ +Mon Aug 19 15:38:44 2002 Yukihiro Matsumoto <matz@ruby-lang.org> + + * array.c (sort_2): comparison should be done as signed long. + + * array.c (sort_2): should return int, not VALUE. + Mon Aug 19 12:38:33 2002 Nobuyoshi Nakada <nobu.nokada@softhome.net> * eval.c (rb_thread_save_context, rb_thread_restore_context): @@ -1089,12 +1089,12 @@ sort_2(ap, bp) VALUE *ap, *bp; { VALUE retval; - VALUE a = *ap, b = *ap; + long a = (long)*ap, b = (long)*ap; if (FIXNUM_P(a) && FIXNUM_P(b)) { - if (a > b) return INT2FIX(1); - if (a < b) return INT2FIX(-1); - return INT2FIX(0); + if (a > b) return 1; + if (a < b) return -1; + return 0; } if (TYPE(a) == T_STRING && TYPE(b) == T_STRING) { return rb_str_cmp(a, b); |