* John Denker -- Saturday 03 March 2007: > On 03/03/2007 12:32 PM, Melchior FRANZ wrote: > > Don't use cout/cerr. Use SG_LOG with appropriate log levels. > > There are 2017 comparable uses of cout in FlightGear and 376 more > in SimGear. Why don't you remove some of those?
If you weren't a newcomer then you would know that this is what I'm routinely doing. A bug doesn't become more acceptable when there are similar bugs elsewhere. What's exactly the problem with fixing that? Either *you* fix it, or someone else has to. Is it fair to load the work on others? > When I changed the indentation just now, to make it more uniform, > it increased the length of the atis.?xx patch from 474 to 487 lines. > That's almost three percent! Are you serious? Correctness is valued higher than shortness. m. ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys-and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ Flightgear-devel mailing list Flightgear-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/flightgear-devel
