Skip Menu |
 

This queue is for tickets about the PPI-XS CPAN distribution.

Report information
The Basics
Id: 76561
Status: resolved
Priority: 0/
Queue: PPI-XS

People
Owner: Nobody in particular
Requestors: ANDK [...] cpan.org
KENTNL [...] cpan.org
Cc: jquelin [...] cpan.org
AdminCc:

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

Attachments
PPI-XS-0.901-remove_defined_warnings.patch



Subject: Noisy with bleadperl
Download (untitled) / with headers
text/plain 286b
Since ca. perl 5.15.7 bleadperl warns on 'defined @Foo::Bar' and PPI::XS has such a line as well: https://metacpan.org/source/ADAMK/PPI-XS-0.901/lib/PPI/XS.pm#L45 Sample noisy test output: http://www.cpantesters.org/cpan/report/28a2230c-47ea-11e1-a8a9-7b559aeef8c6 Thanks&Regards,
Download (untitled) / with headers
text/html 18.8k

Message body is not shown because it is too large.

attached patch fixes the problem.
Subject: PPI-XS-0.901-remove_defined_warnings.patch
--- lib/PPI/XS.pm~ 2009-05-05 16:38:33.000000000 +0200 +++ lib/PPI/XS.pm 2012-07-11 15:42:21.302160716 +0200 @@ -42,7 +42,7 @@ # (example) PPI::Element::exclude_this } elsif ( $VERSION < $PPI::VERSION ) { # It is newer, it has the option of excluding functions - if ( defined @PPI::XS_EXCLUDE ) { + if ( @PPI::XS_EXCLUDE ) { # It as defined things for us to exclude %EXCLUDE = map { $_ => 1 } @PPI::XS_EXCLUDE; }
Fixed in 0.902


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.