diff options
author | knu <knu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2008-05-27 11:18:50 +0000 |
---|---|---|
committer | knu <knu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2008-05-27 11:18:50 +0000 |
commit | 619c73370397d74a9eed9dedcd942ff00871c2ba (patch) | |
tree | 0349edbbc8a3999faab878de860422d65ef8f5e3 | |
parent | 33df14986592d8fa044bbbb9db1aee0d2354096c (diff) | |
download | ruby-619c73370397d74a9eed9dedcd942ff00871c2ba.tar.gz ruby-619c73370397d74a9eed9dedcd942ff00871c2ba.tar.xz ruby-619c73370397d74a9eed9dedcd942ff00871c2ba.zip |
* array.c (rb_ary_slice_bang): Return an empty array instead of
nil when pos is valid and len is adjusted from a valid value to
zero; caught by RubySpec.
git-svn-id: http://svn.ruby-lang.org/repos/ruby/trunk@16643 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 6 | ||||
-rw-r--r-- | array.c | 5 |
2 files changed, 9 insertions, 2 deletions
@@ -1,3 +1,9 @@ +Tue May 27 20:18:30 2008 Akinori MUSHA <knu@iDaemons.org> + + * array.c (rb_ary_slice_bang): Return an empty array instead of + nil when pos is valid and len is adjusted from a valid value to + zero; caught by RubySpec. + Tue May 27 19:12:37 2008 Nobuyoshi Nakada <nobu@ruby-lang.org> * Makefile.in (MKPREP), common.mk, win32/Makefile.sub (prelude.c): get @@ -1815,13 +1815,14 @@ rb_ary_slice_bang(int argc, VALUE *argv, VALUE ary) pos += orig_len; if (pos < 0) return Qnil; } - else if (orig_len <= pos) return Qnil; + else if (orig_len < pos) return Qnil; if (orig_len < pos + len) { len = orig_len - pos; } + if (len == 0) return rb_ary_new2(0); arg2 = rb_ary_new4(len, RARRAY_PTR(ary)+pos); RBASIC(arg2)->klass = rb_obj_class(ary); - rb_ary_splice(ary, pos, len, Qundef); /* Qnil/rb_ary_new2(0) */ + rb_ary_splice(ary, pos, len, Qundef); return arg2; } |