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

People
Owner: Nobody in particular
Requestors: blue [...] thisisnotmyrealemail.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in:
  • 1.106
  • 1.108
  • 1.109
  • 1.111
Fixed in:
  • 1.110_001
  • 1.112_001
  • 1.112_002
  • 1.113



Subject: Subroutines::ProhibitAmpersandSigils: improperly interprets list of sub refs
This code confuses the Subroutines::ProhibitAmpersandSigils policy: @SIG{qw(__DIE__ __WARN__)} = \(&Carp::confess, &Carp::cluck);
Proposed fix committed as svn revision 3648. The perlref docs say that '\( a )' is equivalent to '( \a )', and a little fiddling with YAML suggests that '\( ( a ) )' is too, so the proposed fix simply returns on finding a list preceded by a backslash.
This wasn't as fixed as I thought it was. The update committed as SVN update 3964 allows not only '\( &sub1 )' but also '\( &sub1, &sub2 )'. Maybe it's fixed NOW.