-
Notifications
You must be signed in to change notification settings - Fork 32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Noisy on bleadperl #38
Comments
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). |
Fixed in c00b048, which looks strange, though. In fact, I believe this is a bug in Perl. |
I would say Perl is just too dumb to recognize that $xs is not changed after using in the constant generation. |
I should have used constant.pm to define constant subroutines :( |
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 |
Changelog diff is: diff --git a/Changes b/Changes index f1e2ea3..3459479 100644 --- a/Changes +++ b/Changes @@ -1,6 +1,8 @@ Revision history for Mouse {{$NEXT}} + +2.4.2 2015-04-12T01:22:02Z - Fixed #40; 'use strict' not enabled when 'use 5.010' follows 'use Mouse' - Fixed #39; New warnings in Perl 5.21.x: redundant arguments for sprintf - Fixed #38; Avoid warnings introduced in Perl 5.21.x
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)
The text was updated successfully, but these errors were encountered: