On Mon, 10 Jul 2006, Melchior Franz wrote:

> Update of /var/cvs/FlightGear-0.9/FlightGear/src/Airports
> In directory baron:/tmp/cvs-serv5045
>
> Modified Files:
>       simple.cxx simple.hxx
> Log Message:
> - unify coding style (though not to the last detail)
> - remove trailing spaces
> - fix mixed indentation (tabs -> 8 spaces)

Noted that, I used to send out tabified code. Personally, I prefer
tabs, but I'll :set expandtab ts=8 if that's people want in the next
patches.

> - throw out braindead FSF coding style that has somehow sneaked in
:-) "First off, I'd suggest printing out a copy of the GNU coding
standards, and NOT read it. Burn them, it's a great symbolic gesture."
        /usr/src/linux/Documentation/CodingStyle

How about a uniform set of GNU indent(1) settings to use on ANY files
that one is patching? this way they all will be unified eventually
(but I don't think it is worth doing a major reindenting of all the
files that are not touched otherwise); or maybe the one applying
the patch could do it.

Such settings should probably be stored in the cvs within source/scripts.
I have no problem with the kernel style (see /usr/src/linux/scripts/Lindent ).

V



-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
Flightgear-devel mailing list
Flightgear-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/flightgear-devel

Reply via email to