|Think about Loose Coupling|
Re: Module Announcement: Perl-Critic-1.01by stvn (Monsignor)
|on Jan 26, 2007 at 13:13 UTC||Need Help??|
Leaving the whole thing about RCS keywords (my grandfather used RCS, I use subversion ;) and specifc POD sections (maybe I don't want to name them the way TheDamian does) aside, I am surpised at a couple of things in the output:
The magic punctuatinon it is talking about is $@ without which I could not really catch exceptions. I think that is a little severe.
I don't even get that one because the line in question is basically something like this:
I would have to say that this string was far from useless.
Now, I can understand this for large subroutines, but the sub in queston was basically this:
I would argue that adding return to something like that would be just adding noise. Of course I know you are supposed to be able to customize Perl::Critic to your own tastes, but I wonder if the default settings are maybe not too severe? My first thoughts when I saw all the output was similar to BrowserUK, I felt like I was looking at some overpriced Java middleware exception stack trace.