Re: perltidy - Mailing list pgsql-hackers

From Magnus Hagander
Subject Re: perltidy
Date
Msg-id i2o9837222c1004190931t66323f53l8370ac3645ebe2d5@mail.gmail.com
Whole thread Raw
In response to Re: perltidy  (Peter Eisentraut <peter_e@gmx.net>)
Responses Re: perltidy
List pgsql-hackers
On Mon, Apr 19, 2010 at 18:24, Peter Eisentraut <peter_e@gmx.net> wrote:
> On mån, 2010-04-19 at 17:02 +0200, Magnus Hagander wrote:
>> The MSVC scripts currently have a "perltidy coding style" so we run
>> perltidy with a specific set of arguments to format the code. (This is
>> in the README). Kind of like pgindent.
>>
>> Should we be doing this on all the perlscripts we use?
>>
>> And if we do, we should obviously use the same one everywhere -
>> probably just use the one we have for the msvc stuff today. Anything
>> in particular about that one that people hate?
>
> I tried it on create_help.pl and couldn't find a good combination of
> options that I liked.  It either adds too much whitespace or removes too
> much, or both.  Maybe that can be fined tuned.  I wouldn't want to use
> the -bl option; it's not a typical Perl style.

I doubt we're ever going to find a style taht everybody likes. Heck,
everybody certainly don't like our C style. We just need to decide on
one that's good enough and then go with it.

I don't recall exactly why -bl was added to the msvc style, but
probably to make it look more like our C code ;)

I don't care too much exactly *what* we do, but I think we should have
a common style...


-- Magnus HaganderMe: http://www.hagander.net/Work: http://www.redpill-linpro.com/


pgsql-hackers by date:

Previous
From: Peter Eisentraut
Date:
Subject: Re: perltidy
Next
From: Andres Freund
Date:
Subject: Re: idle in txn query cancellation