On Tue, Nov 2, 2010 at 5:52 PM, Guillaume Lelarge <[email protected]> wrote: > Le 24/10/2010 07:49, Dave Page a écrit : >> On Sun, Oct 24, 2010 at 2:22 PM, Guillaume Lelarge >> <[email protected]> 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. Another thing to consider if you're thinking about a massive fixup commit is removing trailing whitespace. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgadmin-hackers mailing list ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgadmin-hackers
