[Melbourne-pm] Does anyone use Perl::Critic and Test::Perl::Critic

Scott Penrose scottp at dd.com.au
Wed Mar 15 21:31:08 PST 2006


Hey Guys

I like the idea of Perl::Critic and have added it into my test code.  
But I have a question of how it works.

For example, one of the rules is:
> Perl::Critic::Policy::CodeLayout::ProhibitParensWithBuiltins
>
> Write open $handle, $path instead of open($handle, $path) [Severity 1]
It produces errors like this:

> Builtin function called with parens at line 217, column 13. See  
> page 13 of PBP

But if you o to Page 13 of the book it says:

> ... so they can legitimately be called withou parentheses, except  
> where it's necessary to enforce precedence.

But the rule does not do this - it forces them all, it does not look  
at the precedence.

Is this a limitation of Perl::Critic.

On a similar note is CVS/RCS - The book says use CVS/RCS - the Rule  
say you must have $Revision$, $Date$ etc at the top of every file -  
they don't really match.

Scott
-- 
* - *  http://www.osdc.com.au - Open Source Developers Conference * - *
Scott Penrose
Welcome to the Digital Dimension
http://www.dd.com.au/
scottp at dd.com.au

Dismaimer: Contents of this mail and signature are bound to change  
randomly. Whilst every attempt has been made to control said  
randomness, the author wishes to remain blameless for the number of  
eggs that damn chicken laid. Oh and I don't want to hear about  
butterflies either.

Please do not send me Word or PowerPoint attachments.
See http://www.gnu.org/philosophy/no-word-attachments.html

Microsoft is not the answer. It's the question. And the answer is no.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: PGP.sig
Type: application/pgp-signature
Size: 186 bytes
Desc: This is a digitally signed message part
Url : http://mail.pm.org/pipermail/melbourne-pm/attachments/20060316/1fb1e5ca/PGP.bin


More information about the Melbourne-pm mailing list