Perl::Critic::Policy::CodeLayout::ProhibitParensWithBuiltins - Write open $handle, $path instead of open($handle, $path).
This Policy is part of the core Perl::Critic distribution.
Conway suggests that all built-in functions be called without
parentheses around the argument list. This reduces visual clutter and
disambiguates built-in functions from user functions. Exceptions are
made for my, local, and our which require parentheses when
called with multiple arguments.
open($handle, '>', $filename); #not ok open $handle, '>', $filename; #ok split(/$pattern/, @list); #not ok split /$pattern/, @list; #ok
This Policy is not configurable except for the standard options.
Coding with parentheses can sometimes lead to verbose and awkward constructs, so I think the intent of Conway's guideline is to remove only the unnecessary parentheses. This policy makes exceptions for some common situations where parentheses are usually required. However, you may find other situations where the parentheses are necessary to enforce precedence, but they cause still violations. In those cases, consider using the '## no critic' comments to silence Perl::Critic.
Jeffrey Ryan Thalhammer <thaljef@cpan.org>
Copyright (c) 2005-2008 Jeffrey Ryan Thalhammer. All rights reserved.
This program is free software; you can redistribute it and/or modify it under the same terms as Perl itself. The full text of this license can be found in the LICENSE file included with this module.