Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Perl-Critic CPAN distribution.

Report information
The Basics
Id: 88007
Status: new
Priority: 0/
Queue: Perl-Critic

People
Owner: Nobody in particular
Requestors: EDAVIS [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: Suggested policy: redundant () in postfix control flow
Because 'if', 'foreach' and so on require parentheses in their block form, newcomers may write code like say 'debug flag is on' if ($debug); The extra () round the condition are not needed and not the usual Perl style. There should be a policy to warn about them. It doesn't matter if the policy doesn't catch 100% of cases as long as it warns about the most common ones.