* John Denker -- Sunday 28 January 2007:
> http://www.av8n.com/fly/fgfs/location-in-air.xml.diff
+# We need our own private variables, to have and to hold,
+# without worrying about whether the FDM will mess with
+# them. A subset of these will be copied to FDM variables
+# with similar names. Which subset depends on which radio
+# button is pushed.
Umm ... that's exactly the kind of comment that I absolutely
detest. We all know what local variables are. And we can
see what the code does. No need to repeat it in "human"
language. Nasal/c++/xml *is* already human language, or we
would all code in 1 and 0. And I will also not commit
anything that uses baby-language. ("myvar", "mynode", ...).
But I'll test the patch and see if agree with the code. :-)
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
[email protected]
https://lists.sourceforge.net/lists/listinfo/flightgear-devel
