On Thu, Apr 30, 2009 at 09:40:50AM -0700, David Fetter wrote:
> On Thu, Apr 30, 2009 at 11:39:33AM -0500, Andy Lester wrote:
> > On Apr 30, 2009, at 6:41 AM, Robert Haas wrote:
> >>> Please clean up this code at least to the point where it's
> >>> strict-clean, which means putting "use strict;" right after the
> >>> shebang line and not checking it in until it runs that way.
> >> And "use warnings;", too.
> > I'll prob'ly come up with a policy file for Perl::Critic and a make
> > target for perlcritic.
> The current code has a bunch of 5s in it, so it's a target-rich
> environment :)
Here's a patch that gets it to pass perlcritic -4 and still (as far as
I can tell) work.
David Fetter <david(at)fetter(dot)org> http://fetter.org/
Phone: +1 415 235 3778 AIM: dfetter666 Yahoo!: dfetter
Skype: davidfetter XMPP: david(dot)fetter(at)gmail(dot)com
Remember to vote!
Consider donating to Postgres: http://www.postgresql.org/about/donate
In response to
pgsql-hackers by date
|Next:||From: James Mansion||Date: 2009-04-30 19:26:02|
|Subject: Re: idea: global temp tables|
|Previous:||From: David Fetter||Date: 2009-04-30 16:40:50|
|Subject: Re: Keyword list sanity check|