Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

qr/(?{...})/ regressions in 5.18 #14292

Closed
p5pRT opened this issue Dec 2, 2014 · 4 comments
Labels

Comments

@p5pRT
Copy link
Collaborator

@p5pRT p5pRT commented Dec 2, 2014

Migrated from rt.perl.org#123344 (status was 'resolved')

Searchable as RT123344$

@p5pRT

This comment has been minimized.

Copy link
Collaborator Author

@p5pRT p5pRT commented Dec 2, 2014

From @cpansprout

I fixed these in two recent commits​:

3bc8ec9​:

$ perl5.18.1 -e 'qr/@​a(?{})/'
Bizarre copy of ARRAY in subroutine exit at -e line 1.

d100ca4​:

$ ./perl -e 'qr/\Q(?{})/'
Assertion failed​: (AvFILLp(PL_comppad) == 0), function Perl_pmruntime, file op.c, line 5395.
Abort trap​: 6

I recommend that we backport the fixes to both maint-5.20 and maint-5.18.

d100ca4 looks complex. If necessary, we could just delete the assertion from the maint branches, instead of backporting the new assertion.

--

Father Chrysostomos

@p5pRT

This comment has been minimized.

Copy link
Collaborator Author

@p5pRT p5pRT commented Dec 6, 2014

@steve-m-hay - Status changed from 'new' to 'pending release'

@p5pRT

This comment has been minimized.

Copy link
Collaborator Author

@p5pRT p5pRT commented Jun 2, 2015

From @khwilliamson

Thanks for submitting this ticket

The issue should be resolved with the release today of Perl v5.22, available at http​://www.perl.org/get.html
If you find that the problem persists, feel free to reopen this ticket

--
Karl Williamson for the Perl 5 porters team

@p5pRT

This comment has been minimized.

Copy link
Collaborator Author

@p5pRT p5pRT commented Jun 2, 2015

@khwilliamson - Status changed from 'pending release' to 'resolved'

@p5pRT p5pRT closed this Jun 2, 2015
@p5pRT p5pRT added the Severity Low label Oct 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.