bug-gnubg
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Bug-gnubg] Coding Styles - Indentation / Tabs / Spaces


From: Michael Petch
Subject: Re: [Bug-gnubg] Coding Styles - Indentation / Tabs / Spaces
Date: Fri, 26 Oct 2012 19:00:59 -0600
User-agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:13.0) Gecko/20120614 Thunderbird/13.0.1

On 2012-10-26 18:37, Joseph Heled wrote:
> Hi,
> 
> I am in favour of
>   -- uniform style in a project

If I was the lone person on my own project I would have globally
reformatted everything and put it away already. I agree, having
everything in one format would be nice. And then suggest it as the
coding style for our project for the code contributors.

>   -- not mixing tabs and spaces. I would say use spaces only.

Yes, this is often a debate in itself. I have no problem with all
spaces. Many of the modern editors support the conversion of a hard tab
into spaces. The advantage is that you can read it even in editors that
can't have the tab width adjusted and it still looks correct. I don't
think extra file size is as much of a concern. I don't know how many are
saving it onto punchcards ;-), nor is it destined for webpages.

>   -- reasonably cramming as much code into HxW as possible : my
> eyesight forces me to relatively large font, even on a large screen.
>      So I *hate* lines with a single curly (wasting the most precious
> resource, vertical lines). I also prefer 2 spaces indentation - could
> probably live with 4.
> 

So curly braces on control statements would help. And you'd probably
prefer to have blank lines minimized (These options exist in the
automatic code formatters as well)

Regarding tabs I'm not a personal fan of two, and I think 8 is too many
for our project given the indentation depth of some of our functionss. I
will point out that the Google c-style is a tab width of 2, so it does
have a following. I have to throw my hat into the ring and say I have a
personal preference of 4. I do use 8 on my own projects, but I wouldn't
suggest it here.

-- 
Michael Petch
CApp::Sysware Consulting Ltd.
OpenPGP FingerPrint=D81C 6A0D 987E 7DA5 3219 6715 466A 2ACE 5CAE 3304





reply via email to

[Prev in Thread] Current Thread [Next in Thread]