* 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
Flightgear-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/flightgear-devel

Reply via email to