Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Perl-Critic CPAN distribution.

Report information
The Basics
Id: 24618
Status: open
Priority: 0/
Queue: Perl-Critic
People
Owner: Nobody in particular
Requestors: smueller [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in: 1.01
Fixed in: (no value)

History Show all quoted text
  Fri Jan 26 18:15:30 2007 smueller [...] cpan.org - Ticket created
Subject: Capture variable used outside conditional (Wrong)
Hi, If you run PAR.pm through Perl::Critic, you get: Capture variable used outside conditional at line 555, column 18. See page 253 of PBP. Severity: 3 0555: CORE::exit($1) if ($@ =~/^_TK_EXIT_\((\d+)\)/); Now, the $1 is *not* used outside a conditional. Keep up the good work! Steffen P.S.: I used the online-validator at perlcritic.com. I suppose it's version 1.01.
  Fri Jan 26 20:46:51 2007 chris+rt [...] chrisdolan.net - Correspondence added
On Fri Jan 26 18:15:30 2007, SMUELLER wrote: Show quoted text
> If you run PAR.pm through Perl::Critic, you get: > > > Capture variable used outside conditional at line 555, column 18. See > page 253 of PBP. Severity: 3 > > 0555: CORE::exit($1) if ($@ =~/^_TK_EXIT_\((\d+)\)/); > > > Now, the $1 is *not* used outside a conditional.
Yeah, we're not handling postfix control structures yet in the policy. I think we may have more bug reports against that one policy than any other now! -- Chris Dolan
  Fri Jan 26 20:46:53 2007 The RT System itself - Status changed from 'new' to 'open'