1berry/ruby
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Do not set Enumerator::Lazy#zip to use packed format

Browse Source 
Enumerator#zip yields a single array, not multiple arguments,
so Enumerator::Lazy#zip should do the same.

Fixes [#20623]
This commit is contained in:
Jeremy Evans 2024-07-19 13:08:11 -07:00
parent 84680dc255
commit e7dda08617
Notes: git 2024-07-21 01:17:00 +00:00 
Merged: https://github.com/ruby/ruby/pull/11212
2 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
enumerator.c
View File

@ -2400,7 +2400,6 @@ lazy_zip_func(VALUE proc_entry, struct MEMO *result, VALUE memos, long memo_inde
rb_ary_push(ary, v);
}
LAZY_MEMO_SET_VALUE(result, ary);
LAZY_MEMO_SET_PACKED(result);
return result;
}

4
test/ruby/test_lazy_enumerator.rb
View File

@ -489,6 +489,10 @@ EOS
assert_equal [1, 2, 3], enum.map { |x| x / 2 }
end
def test_lazy_zip_map_yield_arity_bug_20623
assert_equal([[1, 2]], [1].lazy.zip([2].lazy).map { |x| x }.force)
end
def test_lazy_to_enum
lazy = [1, 2, 3].lazy
def lazy.foo(*args)