diff options
author | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2012-11-16 07:28:30 +0000 |
---|---|---|
committer | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2012-11-16 07:28:30 +0000 |
commit | c9b6a01435ef814f790f770520400e7f6ccc5e20 (patch) | |
tree | b9a401a541538db3620b0b0cda992aa0e4add3c3 /range.c | |
parent | 288da355e3575a7331f75ae2a8593e59d4433d78 (diff) |
range.c: compare signedness only
* range.c (BSEARCH_CHECK): compare signedness only and relax an
assumption of the result of rb_cmpint() which compilers cannot know.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@37676 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'range.c')
-rw-r--r-- | range.c | 8 |
1 files changed, 3 insertions, 5 deletions
@@ -543,11 +543,9 @@ range_bsearch(VALUE range) smaller = 0; \ } \ else if (rb_obj_is_kind_of(v, rb_cNumeric)) { \ - switch (rb_cmpint(rb_funcall(v, id_cmp, 1, INT2FIX(0)), v, INT2FIX(0))) { \ - case 0: return val; \ - case -1: smaller = 1; break; \ - case 1: smaller = 0; \ - } \ + int cmp = rb_cmpint(rb_funcall(v, id_cmp, 1, INT2FIX(0)), v, INT2FIX(0)); \ + if (!cmp) return val; \ + smaller = cmp < 0; \ } \ else { \ smaller = RTEST(v); \ |