Todd Rinaldo

Results 207 comments of Todd Rinaldo
trafficstars

I really like the idea of tracking what people do with those 2 options. I don't know the level of effort required to do so?

Given the slow nature of fixing PPI in the related case (https://github.com/Perl-Critic/PPI/issues/194), I wonder if Perl::Critic could just see the "prototype" and use its own smarts to recognize it as...

@zdm how has this been resolved?

> This issue is 8 years old, seems that it is not actual already. I cleaned my issues list from garbage. It's a pretty important issue for many people.

I couldn't agree more that this is a critic not a PPI issue. Given Perl::Critic's job isn't to determine if the code can compile, would it be possible to give...

This also breaks in the same way: sub open_wrap { my ($fh_ref) = (\shift) return open( $$fh_ref, '>', $_[1] ); } But this does not break: sub open_wrap { return...

FYI we've seen errors from B/Deparse.pm, depending on the size of the test file / planetary alignment, etc. so that does happen some times.

In at least one case, I know we have patched B.pm like it's patched here. The problem persists. http://perl5.git.perl.org/perl.git/blobdiff/c31ee3bbc31a4f8c5b4850ce38fb8c353dca688f..5cc8528c900964306cba9b53c6eaa27af540eaea:/ext/B/B.pm @rurban are you saying the bug is the lack of patching...

To be clear, some action needs taking since Devel Cover is corrupting at the moment and the problem in B (if there is one) is not fixed or identified.

My findings: ``` $>corelist attrs B::Lint::Plugin::Test B::Section DynaLoader EnableModule FindExt in java::lang::String JNI JPL::AutoLoader JPL::Class JPL::Compile Maintainers MyTestModule of Thread::Signal Thread::Specific UnicodeCD|grep -v 'Data for'|sort -r UnicodeCD was not in...