From: | Guillaume Lelarge <guillaume(at)lelarge(dot)info> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
Cc: | Magnus Hagander <magnus(at)hagander(dot)net>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: Source reindenting |
Date: | 2010-12-29 23:45:40 |
Message-ID: | 4D1BC824.7030309@lelarge.info |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Le 29/12/2010 18:44, Dave Page a écrit :
> On Wed, Dec 29, 2010 at 5:30 PM, Magnus Hagander <magnus(at)hagander(dot)net> wrote:
>> I just did a testrun with astyle to reindent and restyle the code. I
>> created a rule in Makefile.am per:
>>
>> diff --git a/Makefile.am b/Makefile.am
>> index be6af45..8266f8d 100644
>> --- a/Makefile.am
>> +++ b/Makefile.am
>> @@ -65,3 +65,13 @@ endif
>> # We need to ensure the help cache is world writeable
>> install-data-hook:
>> chmod 0666 $(help_cache)
>> +
>> +# Perform astyle cleanup per rules:
>> +# * -p - insert space around parenthesis
>> +# * -b - bracket style
>> +# * -S - indent switches
>> +# * -s4 - intent with spaces, 4 of them
>
> Didn't we say we were going to move to tabs?
>
Yes. Magnus probably forgot that part when I talked with him on IM this
morning (well, yesterday morning now).
>> It's going to cause merge conflicts like hell. but it's going to do
>> that whenever we run it. So should we just go ahead and run it?
>
> Why not? :-)
>
The only patches not yet commited that I know of are the GSoC projects.
I know it won't be an issue for the database designer. Could be one for
your project.
--
Guillaume
http://www.postgresql.fr
http://dalibo.com
From | Date | Subject | |
---|---|---|---|
Next Message | Magnus Hagander | 2010-12-30 08:09:07 | pgAdmin III commit: Add support for REPLICATION roles from 9.1 |
Previous Message | Dave Page | 2010-12-29 19:34:04 | Re: fix: cannot build wxWidgets if sources are located at another disk |