On Fri, May 4, 2018 at 5:28 PM, Mike Blackwell wrote:
> I didn't see a .perlcriticrc file in the project, so ran with our local
> settings.
>
> With those, perlcritic is pretty unhappy, even at -4, though I don't see
> anything that pops out as potentially bug-inducing. The ones I'd probably
> lo
I didn't see a .perlcriticrc file in the project, so ran with our local
settings.
With those, perlcritic is pretty unhappy, even at -4, though I don't see
anything that pops out as potentially bug-inducing. The ones I'd probably
look fixing at for starters would be the two argument form of open,
Mike Blackwell writes:
> In my experience, that would more commonly be written with the lower
> precedence "or" operator (with or without the param list parens):
> unlink $temp_name or die "unlink: $temp_name: $!";
Yeah, I thought about that, but the pre-existing rename call had ||
and I didn't
In my experience, that would more commonly be written with the lower
precedence "or" operator (with or without the param list parens):
unlink $temp_name or die "unlink: $temp_name: $!";
__
*Mike Blackwell | Technical
