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
Bleadperl v5.22.0-142-g6228a1e breaks SEWI/Tie-DBIx-Class-0.01.tar.gz #15046
Comments
From @andkbisect commit 6228a1e pp_return: tail call pp_leavesub cpantesters http://www.cpantesters.org/cpan/report/250ba40a-3fbe-11e5-8c3e-1141b56994ca discovered by Slaven: https://rt.cpan.org/Ticket/Display.html?id=108752 perl -V Summary of my perl5 (revision 5 version 23 subversion 1) configuration: Characteristics of this binary (from libperl): |
From @tonycozOn Wed Nov 11 23:45:31 2015, andreas.koenig.7os6VVqR@franz.ak.mind.de wrote:
I haven't tracked this down yet, but there is some stack handling issue around untie() with UNTIE both in 5.22 and blead, using the attached script: tony@mars:.../git/perl$ ./perl ../126621.pl Only when UNTIE returns an empty list does untie() behave: tony@mars:.../git/perl$ ./perl ../126621.pl untie()'s return value is undocumented, but the tests that are failing in Tie::DBIx::Class are testing that value, so arguably the tests are incorrect. From looking at pp_untie, the only case it appears to return false (undef) is when the supplied value is for a defelem with no target. Tony |
The RT System itself - Status changed from 'new' to 'open' |
From @tonycozOn Wed Nov 11 23:45:31 2015, andreas.koenig.7os6VVqR@franz.ak.mind.de wrote:
Dave Mitchell fixed this in: commit 2e2d740 pp_leavesub: reset SP in void context |
@tonycoz - Status changed from 'open' to 'pending release' |
From @khwilliamsonThank you for submitting this report. You have helped make Perl better. Perl 5.24.0 may be downloaded via https://metacpan.org/release/RJBS/perl-5.24.0 |
@khwilliamson - Status changed from 'pending release' to 'resolved' |
Migrated from rt.perl.org#126621 (status was 'resolved')
Searchable as RT126621$
The text was updated successfully, but these errors were encountered: