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: 42767
Status: resolved
Priority: 0/
Queue: Perl-Critic

People
Owner: Nobody in particular
Requestors: hinrik.sig [...] gmail.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: Variables::ProhibitReusedNames shouldn't catch $self in inner subs
I read the CAVEATS section of this policy's Pod. While valid, I think an exception should be made for $self.
Subject: Re: [rt.cpan.org #42767] Variables::ProhibitReusedNames shouldn't catch $self in inner subs
Date: Sun, 25 Jan 2009 01:37:33 -0600
To: bug-Perl-Critic [...] rt.cpan.org
From: Elliot Shank <perl [...] galumph.com>
Hinrik Orn Sigurdsson via RT wrote: Show quoted text
> I read the CAVEATS section of this policy's Pod. While valid, I think an > exception should be made for $self.
Agreed. By default, $class now gets an exception, as well. There's a new "allow" option to add to that list. This will be in the new release.
Subject: Re: [rt.cpan.org #42767] Variables::ProhibitReusedNames shouldn't catch $self in inner subs
Date: Sun, 25 Jan 2009 11:11:59 -0600
To: bug-Perl-Critic [...] rt.cpan.org
From: Elliot Shank <perl [...] galumph.com>
Elliot Shank via RT wrote: Show quoted text
> Agreed. By default, $class now gets an exception, as well. There's > a new "allow" option to add to that list. This will be in the new > release.
s/new release/next release/
This has been completed and released in Perl-Critic-1.097_002. http://search.cpan.org/~elliotjs/Perl-Critic-1.097_002 Thank you for submitting this ticket.