ARRRGHHHH

I just discovered this piece of code in the CVS

 } else {
   *(int*)0=0;  // unexpected tag, boom
 }
}

I will leave the name calling and expletives out  but

THIS IS RUDE ARROGANT AMATURISH PROGRAMMING 
TO THE MAX and completely UNACCEPTABLE in a complex
project like FlightGear where not everyone is familiar with
every line of code

Would the programmer responsible either

1) CHANGE THIS ASAP to at least print an error message
  or 
2) defend this hack publicly 

=================

There is some good news however

After discovering that the above was responsible for
unexplained crashing at startup with some configurations 
I can announce that I have a new Height above Ground algorithm 
that is  MUCH faster < read order of magnitude > 

FWIW - this should translate into a 5-10% boost in frame rate
depending on current Scenery complexity,   angle of view 
< even a  geForce2 becomes fill-limited quickly by the cloud rendering 
   so tilting the view down a bit should help frame-rate for most >
etc. 

Note for the FDM writers
This means that queries for multiple < 3 or 4 > gear locations
should be quicker then just the single query was before

Cheers

Norman 


_______________________________________________
Flightgear-devel mailing list
[EMAIL PROTECTED]
http://mail.flightgear.org/mailman/listinfo/flightgear-devel

Reply via email to