Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

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

Report information
The Basics
Id: 82121
Status: open
Priority: 0/
Queue: Perl-Critic-StricterSubs

People
Owner: Nobody in particular
Requestors: GWS [...] cpan.org
Cc:
AdminCc:

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



Subject: RequireExplicitInclusion: false positive according to the policy's docs
Perl::Criric reports line 15 of the included source as a RequireExplicitInclusion violation. The package in question is, however, included in the same file (arguably for good reasons in this case, the general "1 package per module file" policy not withstanding). In any case, according to the docs of RequireExplicitInclusion: "It is assumed that a module will contain no more than one package. This Policy will not complain about any problems in a module containing multiple "package" statements." If I understand correctly, then there should be no violation reported here.
Subject: ZVtest.pm
package ZVtest; use strict; use warnings; # Class variable: my $zv_grammar = do { use Regexp::Grammars 1.021; qr{ foo }xmsi }; sub new { my( $class ) = @_; my %params; my $self = bless( \%params, $class ); $params{_GRAMMAR} = $zv_grammar; $params{_ACTIONS} = ZVActions->new( foo => sub { return 'foo' } ); return $self; } sub compile { my( $self, $line ) = @_; return unless $line =~ $self->{_GRAMMAR}->with_actions( $self->{_ACTIONS} ); return ( \%/, $^N ); } 1; package ZVActions; sub new { my( $class ) = @_; my %params; return bless( \%params, $class ); } sub foo { my( $self ) = @_; my $foo = [ 'foo' // 42 ]; return; } sub bar { my( $self ) = @_; return; } 1;
Subject: Re: [rt.cpan.org #82121] RequireExplicitInclusion: false positive according to the policy's docs
Date: Wed, 2 Jan 2013 14:43:34 -0800
To: bug-Perl-Critic-StricterSubs [...] rt.cpan.org
From: Jeffrey Ryan Thalhammer <jeff [...] imaginative-software.com>
On Dec 19, 2012, at 3:24 PM, Gisbert W. Selke via RT wrote: Show quoted text
> Perl::Criric reports line 15 of the included source as a RequireExplicitInclusion violation.
Thanks, I'll look into this in the next few days. Happy New Year! -Jeff
Subject: Abwesenheitsnotiz: [rt.cpan.org #82121] RequireExplicitInclusion: false positive according to the policy's docs
Date: Thu, 3 Jan 2013 17:58:31 +0100
To: "Jeffrey Thalhammer via RT" <bug-Perl-Critic-StricterSubs [...] rt.cpan.org>
From: "Selke, Gisbert W." <Gisbert.Selke [...] wido.bv.aok.de>
Ich bin zurzeit nicht erreichbar. Ab dem 4. Januar 2013 stehe ich Ihnen gerne wieder zur Verfügung. Ihre Mail wird nicht weitergeleitet. Bitte wenden Sie sich bis dahin an Frau Pao-Hui Oßwald-Huang (pao-hui.osswald-huang(at)wido.bv.aok.de). Vielen Dank. ------------------------------------------------------------------------------- I am currently not on location. I shall be back 4th January 2013, when I shall be happy to be at your service. Your mail will not be forwarded. In urgent cases, please, get in touch in the meantime with Ms Pao-Hui Oßwald-Huang (pao-hui.osswald-huang(at)wido.bv.aok.de). Thank you. ------------------------------------------------------------------------------- Je suis absent jusqu'au 4 Janvier 2013. Pendant ce temps, en cas d'urgence, veuillez prendre contact avec Mme. Pao-Hui Oßwald-Huang (pao-hui.osswald-huang(at)wido.bv.aok.de). Merci. ------------------------------------------------------------------------------- Gisbert W. Selke, Wissenschaftliches Institut der AOK (WIdO) Rosenthaler Straße 31, 10178 Berlin, Germany Tel.: +49-30-34646-2114 (Sekretariat: -2393); Fax: +49-30-34646-2144 gisbert.selke(at)wido.bv.aok.de http://www.wido.de