Le 11/11/2010 22:10, Robert Haas a écrit :
> On Tue, Nov 2, 2010 at 5:52 PM, Guillaume Lelarge
> <guillaume@lelarge.info> wrote:
>> Le 24/10/2010 07:49, Dave Page a écrit :
>>> On Sun, Oct 24, 2010 at 2:22 PM, Guillaume Lelarge
>>> <guillaume@lelarge.info> wrote:
>>>> IIRC, we should use spaces and not tabs. If I look into the code, I see
>>>> many tabs in it. I can of course configure my editor to display them as
>>>> four tabs, but I would really like to get rid of them.
>>>>
>>>> Any objections? because it would be a hell of a patch (90427 replacements)
>>>
>>> Go for it. The project standard is for spaces.
>>>
>>> Though, if we're going to make such a big change, I have to wonder if
>>> tabs would be a more convenient choice in hindsight...
>>>
>>
>> Actually, I don't like tabs :)
>>
>> But I don't care that much. What I would like to have is a nice source
>> code, and, now, it doesn't look that nice.
>>
>> So, if it is tabs, so be it :)
>
> I like tabs, but that's just me, and I don't work on pgAdmin. It
> sucks trying to move back and forth between projects with different
> styles, though, and pgsql uses tabs.
>
So go for it.
Just waiting to know Dave's opinion on all this before moving on.
> Another thing to consider if you're thinking about a massive fixup
> commit is removing trailing whitespace.
>
Oh, great idea. There's probably an option for that too.
--
Guillaume
http://www.postgresql.fr
http://dalibo.com