Branch: refs/heads/yves/fix_branch_reset_list_context_assignment
Home: https://github.com/Perl/perl5
Commit: b18fb8a0f894e08e06344b8f49c7ab3548871591
https://github.com/Perl/perl5/commit/b18fb8a0f894e08e06344b8f49c7ab3548871591
Author: Yves Orton <[email protected]>
Date: 2023-01-27 (Fri, 27 Jan 2023)
Changed paths:
M pp_hot.c
M t/re/pat.t
Log Message:
-----------
pp_hot.c - fix branch reset matches in list context
I am kinda surprised this issue was not picked up by one of our
other test files. I would have expected one of the t/re/regexp.t
based patches to validate list context matches populating the list
properly. But apparently not! So when I fixed branch reset in
fe5492d916201ce31a107839a36bcb1435fe7bf0 I missed the list context
logic. This fixes the oversight. Thanks to Andreas Koenig for the
BBC report on this.
This also changes the code to use SSize_t for various length related
operations, the original code was using I32 which might break on
very very long strings. Thanks to Tony C for pointing that out.
