Bug #76561 for PPI-XS: Noisy with bleadperl

Sun Apr 15 03:24:31 2012 ANDK [...] cpan.org - Ticket created

Subject:

Noisy with bleadperl

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,

Mon Apr 16 23:47:25 2012 pjs [...] cpan.org - Correspondence added

Also: http://www.cpantesters.org/cpan/report/c67e136c-8776-11e1-8c7c-398542b6749c

Mon Apr 16 23:47:26 2012 The RT System itself - Status changed from 'new' to 'open'

Thu Jun 07 00:10:07 2012 KENTNL [...] cpan.org - Requestor KENTNL added

Thu Jun 07 00:12:30 2012 KENTNL [...] cpan.org - Correspondence added

Message body is not shown because it is too large.

Wed Jul 11 09:13:39 2012 jquelin [...] cpan.org - Cc JQUELIN added

Wed Jul 11 09:44:42 2012 jquelin [...] cpan.org - Correspondence added

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; }

Tue Feb 26 19:26:05 2013 dagolden [...] cpan.org - Correspondence added

Fixed in 0.902

Tue Feb 26 19:26:07 2013 dagolden [...] cpan.org - Status changed from 'open' to 'resolved'