summaryrefslogtreecommitdiff
path: root/array.c
diff options
context:
space:
mode:
authormame <mame@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2008-10-29 14:30:07 +0000
committermame <mame@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2008-10-29 14:30:07 +0000
commit7451e34428cbab2bd48afde0632d15976c25c991 (patch)
tree501955cd443cb516eaf6941c52199ed9111bf7b8 /array.c
parent224471fd572c1818e99c66f756ce398745885398 (diff)
* array.c (rb_ary_sort_bang): replacing array during sort broke
invariant of array. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@20024 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'array.c')
-rw-r--r--array.c8
1 files changed, 6 insertions, 2 deletions
diff --git a/array.c b/array.c
index e04de09cc2..dcf8d0e56c 100644
--- a/array.c
+++ b/array.c
@@ -1751,14 +1751,18 @@ rb_ary_sort_bang(VALUE ary)
ruby_qsort(RARRAY_PTR(tmp), RARRAY_LEN(tmp), sizeof(VALUE),
rb_block_given_p()?sort_1:sort_2, &data);
- if (ARY_EMBED_P(ary) || ARY_EMBED_P(tmp)) {
+ if (ARY_EMBED_P(tmp)) {
assert(ARY_EMBED_P(tmp));
+ if (ARY_SHARED_P(ary)) { /* ary might be destructively operated in the given block */
+ rb_ary_unshared(ary);
+ FL_SET_EMBED(ary);
+ }
MEMCPY(RARRAY_PTR(ary), ARY_EMBED_PTR(tmp), VALUE, ARY_EMBED_LEN(tmp));
ARY_SET_LEN(ary, ARY_EMBED_LEN(tmp));
}
else {
- assert(!ARY_EMBED_P(ary));
assert(!ARY_EMBED_P(tmp));
+ if (ARY_EMBED_P(ary)) FL_UNSET_EMBED(ary);
if (RARRAY_PTR(ary) != RARRAY_PTR(tmp)) {
assert(!ARY_SHARED_P(tmp));
if (ARY_SHARED_P(ary)) {