Norman Vine wrote:
> 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
This is the YASim XML parser. You hit this line when an unrecognized
tag is found in the XML file. Certainly, printing a nice error
message and exiting cleanly would be better, but you seem to
misunderstand the point. There is nothing useful the code can do at
this point; I actually like the fact that it crashes. If you feed
line noise to other parts of the code, you can get them to crash too.
This just does it in a more debuggable way.
If folks want to see a nice error message, then I'm happy to accept
patches. I'm less happy to be screamed at in public, of course.
Andy
--
Andrew J. Ross NextBus Information Systems
Senior Software Engineer Emeryville, CA
[EMAIL PROTECTED] http://www.nextbus.com
"Men go crazy in conflagrations. They only get better one by one."
- Sting (misquoted)
_______________________________________________
Flightgear-devel mailing list
[EMAIL PROTECTED]
http://mail.flightgear.org/mailman/listinfo/flightgear-devel
