Slash Boxes
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 ]

Ovid (2709)

  (email not shown publicly)
AOL IM: ovidperl (Add Buddy, Send Message)

Stuff with the Perl Foundation. A couple of patches in the Perl core. A few CPAN modules. That about sums it up.

Journal of Ovid (2709)

Friday February 06, 2009
10:53 AM

Class::Sniff for the Win!

[ #38416 ]

I always find out interesting stuff at meetings. Last night's involved me learning about someone who found unexpected problems in his code using Class::Sniff (not naming him because I didn't ask permission). This makes me happy.

There was also a general discussion of its features and here are some thoughts (some I've had, others came up last night):

  • Calling SUPER:: with MI should definitely be a code smell (perhaps an outright bug?) and not too hard to detect. I'm wondering how else this might manifest. It has some odd edge cases, though.
  • Allow Class::Sniff->new({ class => $some_instance }). I know it's trivial, but I get tired of typing 'ref' because I'm usually loading an instance first to ensure everything is loaded properly.
  • Get rid of the Tree. I don't think I need it.
  • $sniff->report({ as => 'html' });
  • Can I merge different sniff graphs to get a fully graphed system?
  • Find a clean way of excluding stuff you don't care about without excluding an entire section of the report.

Ingy and Yuval were also there. It was good to catch up with them. Yay for London, the town everyone wants to stop over at!

The Fine Print: The following comments are owned by whoever posted them. We are not responsible for them in any way.
More | Login | Reply
Loading... please wait.