Stories
Slash Boxes
Comments
NOTE: use Perl; is on undef hiatus. You can read content, but you can't post it. More info will be forthcoming forthcomingly.

All the Perl that's Practical to Extract and Report

use Perl Log In

Log In

[ Create a new account ]

Beatnik (493)

Beatnik
  (email not shown publicly)
http://www.ldl48.org/

A 29 year old belgian who likes Mountain Dew, Girl Scout Cookies, Tim Hortons French Vanilla Flavoured Cappucinno, Belgian beer, Belgian chocolate, Belgian women, Magners Cider, chocolate chipped cookies and Perl. Likes snowboarding, snorkling, sailing and silence. Bach can really cheer him up! He still misses his dog.

Project Daddy of Spine [sf.net], a mod_perl based CMS.

In his superhero time (8.30 AM to 5.30 PM), he works on world peace.

Journal of Beatnik (493)

Wednesday September 06, 2006
07:39 AM

Critic

[ #30906 ]
A short list of critic so far (up to severity 3). I'm still not grasping some of the problems.. Others, I can just live with... some are just false :) I'll probably produce an updated list with statistics (even though this might get embarrasing):
  • Variable declared in conditional statement
  • Expression form of "eval"
  • AUTOLOAD method declared
  • @ISA used instead of "use base"
  • "warn" used instead of "carp"
  • "die" used instead of "croak"
  • Bareword file handle opened
  • Symbols are exported by default
  • Ambiguous name for variable or subroutine
  • Capture variable used outside conditional
  • Regular expression without "/x" flag
  • Subroutine name is a homonym for builtin function
  • Subroutine with high complexity score (X)
  • Subroutine does not end with "return"
  • Warnings disabled
  • Pragma "constant" used
  • Mixed high and low-precedence booleans
  • Heredoc terminator must be quoted
  • Loop iterator is not lexical
The Fine Print: The following comments are owned by whoever posted them. We are not responsible for them in any way.
 Full
 Abbreviated
 Hidden
More | Login | Reply
Loading... please wait.
  • Perl::Critic is hardly bug free. I've been committing failing tests as I've been delinting my own source and finding both false positives and negatives.
  • I set my test severity to 1, then disabled policies I didn't agree with n a custom rc file. I figure that way, when a new policy comes out, it will be on by default, until I either disagree/disable it, or fix code.

    So far so good.