diff --git a/ChangeLog b/ChangeLog index d5149435b3..1d74783cb9 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,7 @@ +Tue Oct 29 22:11:37 2013 Masaki Matsushita + + * array.c (rb_ary_uniq_bang): use st_foreach() instead of for loop. + Tue Oct 29 20:01:58 2013 Koichi Sasada * add RUBY_TYPED_FREE_IMMEDIATELY to data types which only use diff --git a/array.c b/array.c index 7e4d4185ae..19878f0c40 100644 --- a/array.c +++ b/array.c @@ -4050,6 +4050,13 @@ rb_ary_or(VALUE ary1, VALUE ary2) return ary3; } +static int +push_key(st_data_t key, st_data_t val, st_data_t ary) +{ + rb_ary_push((VALUE)ary, (VALUE)key); + return ST_CONTINUE; +} + static int push_value(st_data_t key, st_data_t val, st_data_t ary) { @@ -4085,15 +4092,16 @@ push_value(st_data_t key, st_data_t val, st_data_t ary) static VALUE rb_ary_uniq_bang(VALUE ary) { - VALUE hash, v; - long i, j; + VALUE hash; + long hash_size; rb_ary_modify_check(ary); if (RARRAY_LEN(ary) <= 1) return Qnil; if (rb_block_given_p()) { hash = ary_make_hash_by(ary); - if (RARRAY_LEN(ary) == (i = RHASH_SIZE(hash))) { + hash_size = RHASH_SIZE(hash); + if (RARRAY_LEN(ary) == hash_size) { return Qnil; } rb_ary_modify(ary); @@ -4102,21 +4110,23 @@ rb_ary_uniq_bang(VALUE ary) rb_ary_unshare(ary); FL_SET_EMBED(ary); } - ary_resize_capa(ary, i); + ary_resize_capa(ary, hash_size); st_foreach(rb_hash_tbl_raw(hash), push_value, ary); } else { hash = ary_make_hash(ary); - if (RARRAY_LEN(ary) == (long)RHASH_SIZE(hash)) { + hash_size = RHASH_SIZE(hash); + if (RARRAY_LEN(ary) == hash_size) { return Qnil; } - for (i=j=0; i