Andy Lester

icon indicating a website link http://petdance.com/ icon indicating an email [email protected]

icon location Palatine, IL icon location I wrote ack (@beyondgrep). I'm shepherding Perl::Critic (@Perl-Critic). I do other Perl stuff.

Results 496 comments of Andy Lester

ProhibitEmptyQuotes should be configurable

I'm not understanding what you're suggesting. What is an example of code that trips this warning that you would like to be able to configure past?

ProhibitEmptyQuotes should be configurable

@mkende So you have this code local $Data::Dumper::Pad = ''; and even though the policy throws a warning, the code is OK with you because that's easier to read than...

ProhibitEmptyQuotes should be configurable

So here's our situation as I see it. * ProhibitEmptyQuotes is really ProhibitStringsThatAreNothingButWhitespace. We can't change it. * You want a way to allow a zero-length string. * You don't...

ProhibitEmptyQuotes should be configurable

The downside is it's more complex, and it's not a feature anyone has asked for.

Modules::ProhibitAutomaticExportation should allow explicitly empty @EXPORT

> makes it much clearer that I meant to do that Yes, agreed 100%.

Can't find in non-utf8 files

ack's default behavior is to not look inside binary files. That's why the `return -T $file->name` is there. Using Perl's `-T` operator is clearly not ideal, but I don't know...