diff options
author | mame <mame@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2012-11-15 13:50:55 +0000 |
---|---|---|
committer | mame <mame@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2012-11-15 13:50:55 +0000 |
commit | 412b023742171585b123cf20fcbb9b94476582ac (patch) | |
tree | f4839c7206cf01409fd0ebbe6bd06e2e5719388c /array.c | |
parent | 768801b020d2ec28af2bb719cadf5bc3c126c3d6 (diff) | |
download | ruby-412b023742171585b123cf20fcbb9b94476582ac.tar.gz |
* range.c (range_bsearch): fix some bugs: a documentation bug, a wrong
condition, missed break in switch/case, and workaround for GCC
optimization. See [ruby-core:49364] in detail. A great patch from
Heesob Park. [Bug #7352] [Feature #4766]
* array.c (rb_ary_bsearch): fix similar bug (missed break).
* test/ruby/test_range.rb: add two test cases for above.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@37662 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'array.c')
-rw-r--r-- | array.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -2451,7 +2451,7 @@ rb_ary_bsearch(VALUE ary) 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; + case 1: smaller = 1; break; case -1: smaller = 0; } } |