diff options
author | ocean <ocean@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2005-11-15 07:38:38 +0000 |
---|---|---|
committer | ocean <ocean@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2005-11-15 07:38:38 +0000 |
commit | ea4a254de5b2b369c96e1e7591230bee54a2b4ab (patch) | |
tree | 76408bf6c65ef6e132fef3ffc740ea9a95de3893 | |
parent | d0f784a071bbca9c2fbc4dc1ff7a2027fc8004c8 (diff) | |
download | ruby-ea4a254de5b2b369c96e1e7591230bee54a2b4ab.tar.gz |
* array.c (rb_ary_fill): previous commit disabled this usage:
a = [0,1,2,3,4,5,6,7,8,9]
a.fill {|i| a[i] * 10} #=> [nil, nil, ...., nil]
previous commit has advantange for early garbage collection, but
potensially would break some script. so I reverted behavior.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@9543 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 10 | ||||
-rw-r--r-- | array.c | 7 |
2 files changed, 11 insertions, 6 deletions
@@ -1,3 +1,13 @@ +Tue Nov 15 16:36:03 2005 Hirokazu Yamamoto <ocean@m2.ccsnet.ne.jp> + + * array.c (rb_ary_fill): previous commit disabled this usage: + + a = [0,1,2,3,4,5,6,7,8,9] + a.fill {|i| a[i] * 10} #=> [nil, nil, ...., nil] + + previous commit has advantange for early garbage collection, but + potensially would break some script. so I reverted behavior. + Tue Nov 15 16:15:23 2005 Yukihiro Matsumoto <matz@ruby-lang.org> * file.c (file_load_ok): use eaccess() instead of actually opening @@ -2137,13 +2137,8 @@ rb_ary_fill(int argc, VALUE *argv, VALUE ary) REALLOC_N(RARRAY(ary)->ptr, VALUE, end); RARRAY(ary)->aux.capa = end; } - RARRAY(ary)->len = end; - } - if (beg > RARRAY(ary)->len) { rb_mem_clear(RARRAY(ary)->ptr + RARRAY(ary)->len, end - RARRAY(ary)->len); - } - else { - rb_mem_clear(RARRAY(ary)->ptr + beg, end - beg); + RARRAY(ary)->len = end; } if (block_p) { |