Complete.Org: Mailing Lists: Archives: freeciv-dev: January 2002:
[Freeciv-Dev] Re: Coding Style Guidelines
Home

[Freeciv-Dev] Re: Coding Style Guidelines

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: Chris Richards <chrisr@xxxxxxxxxxxxxxxx>
Cc: Mike Kaufman <mkaufman@xxxxxxxxxxxxxx>, freeciv-dev <freeciv-dev@xxxxxxxxxxx>
Subject: [Freeciv-Dev] Re: Coding Style Guidelines
From: Raimar Falke <hawk@xxxxxxxxxxxxxxxxxxxxxxx>
Date: Sat, 12 Jan 2002 10:43:04 +0100
Reply-to: rf13@xxxxxxxxxxxxxxxxxxxxxx

On Fri, Jan 11, 2002 at 09:29:17PM -0600, Chris Richards wrote:
> Sub-subject: spacing and -ts0
> 
> Since some people aren't strict K&R (re: style guide discussion), is
> it poo-poo'd to use spacing as follows:
> 
Indent will do
> if (foo) {
    ^ enforce a space here
>   need_to_call (maximize (foo));
                ^         ^ will remove these spaces
> }
> 
> Essentially I add a space before the '('.  K&R doesn't mind the space
> after the 'if' but it changes it for the function calls.

IMHO it isn't important what style we use. It is important that we use
one style and that this style covers a lot of the problems.

> Secondly, indent it goes crazy with tabs.  Personally, I think tabs
> should be frowned upon.  And 'indent' does strange things with tabs.
> For example, I ran some cruft through indent and this is what it
> produced:
> 
> void bar()
> {
> SSSSif (foo) {
> Tneed_to_call(maximize(foo));
> SSSS}
> }
> 
> Where 'S' is space and 'T' is tab because tab is 8 spaces by default.
> So for every 8 spaces it indents, it adds a tab filling the remainder
> with 4 spaces if necessary.

> So, I think that any invocation of indent should specify -ts0.

I agree.

        Raimar

-- 
 email: rf13@xxxxxxxxxxxxxxxxx
 "Make it idiot-proof and someone will make a better idiot."


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