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
Wrong warning 'Variable "$_" is not available' in closure #11499
Comments
From qallpaqa@gmail.comCreated by qallpaqa@gmail.comI wonder if it is not valid to give the warning of 'Variable "$_" is I can't figure out whether it's a proper warning or just a bug even It doesn't take place on Perl 5.12. #!perl sub main { # take place just in main() sub func { main(); Perl Info
|
From @cpansproutOn Tue Jul 12 21:24:27 2011, qallpaqa@gmail.com wrote:
I believe that�s a genuine bug. It was caused by this commit: commit 05d04d9 Don't clone the contents of lexicals in pads. |
The RT System itself - Status changed from 'new' to 'open' |
From @iabynOn Tue, Jul 12, 2011 at 10:58:59PM -0700, Father Chrysostomos via RT wrote:
Odd, that commit only affected the pad duping code, which would only be I think I've fixed the real issue with the following commit: commit 87e4a53 RT #4682: given() didn't scope $_ correctly There is a second issue that is brought up by the original bug report, I don't see a simple way to fix that. -- |
@iabyn - Status changed from 'open' to 'resolved' |
From @nwc10On Tue, Sep 06, 2011 at 12:29:00PM +0100, Dave Mitchell wrote:
Meaning that the third section of this code is buggy? $ cat ../test/94682.pl sub DESTROY { sub should_close { sub should_not_close { sub buggy { print "Should close\n"; print "Should not close\n"; print "Should not close either\n"; __END__ Should not close Should not close either In that the sub buggy() should not return a closure, hence the third (and this should become a TODO test somewhere?) Nicholas Clark |
From @iabynOn Thu, May 10, 2012 at 02:27:05PM +0100, Nicholas Clark wrote:
Correct.
Yes. op/closure.t ??? -- |
Migrated from rt.perl.org#94682 (status was 'resolved')
Searchable as RT94682$
The text was updated successfully, but these errors were encountered: