search

xslate / p5-Mouse

Lightweight class builder for Perl, as a subset of Moose
https://metacpan.org/release/Mouse
Other
46 stars 32 forks source link

Noisy on bleadperl #38

Closed andk closed 9 years ago

andk commented 9 years ago

As per subject. Sample PASS report with the warnings:

http://www.cpantesters.org/cpan/report/48501326

Related bleadperl ticket: https://rt.perl.org/rt3/Ticket/Display.html?id=123213

HTH&&Thanks, (Disclaimer: this was discovered by statistical analysis, I'm not speaking as a user)

eserte commented 9 years ago

The "Constants from lexical variables potentially modified elsewhere are deprecated" warning may also affect other modules with nowarnings-like tests (seen with TOBYINK/Role-Inspector-0.005.tar.gz, for example).

gfx commented 9 years ago

Fixed in c00b048, which looks strange, though. In fact, I believe this is a bug in Perl.

eserte commented 9 years ago

I would say Perl is just too dumb to recognize that $xs is not changed after using in the constant generation.

gfx commented 9 years ago

I should have used constant.pm to define constant subroutines :(

eserte commented 9 years ago

Can we have a CPAN release with this fix? There is a number of CPAN modules which have fail reports because of this warning in Mouse, e.g. https://rt.cpan.org/Ticket/Display.html?id=101085