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