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

Owner: Nobody in particular
Requestors: EDAVIS [...]

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

Subject: Suggested policy: redundant () in postfix control flow
MIME-Version: 1.0
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Web
Message-ID: <rt-4.0.16-15357-1377102289-1455.0-0-0 [...]>
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 378
Download (untitled) / with headers
text/plain 378b
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.

This service is sponsored and maintained by Best Practical Solutions and runs on infrastructure.

Please report any issues with to