On 11/27/2008 12:23 PM, Syd wrote: > Since I've had to add a nasal inHG to Kpa conversion to most > altimeters I've added , I decided maybe its something altimeter.cxx > should do , so here is a small patch to do that. Could someone check and > commit please ?
This seems to be an unhappy medium. On the one hand, AFAICT this does not fully implement the "setting-kpa" node. In particular, unpleasant things are going to happen if somebody sets (via nasal) the setting-kpa node. There is no code to maintain consistency if this happens. It would be a burden on the documentation to outlaw this. On the other hand, I don't see how this is much of an improvement over just reading the "setting-inhg" node and applying the conversion factor each time. Floating-point multiplication is cheap. Or am I missing something? ------------------------------------------------------------------------- This SF.Net email is sponsored by the Moblin Your Move Developer's challenge Build the coolest Linux based applications with Moblin SDK & win great prizes Grand prize is a trip for two to an Open Source event anywhere in the world http://moblin-contest.org/redirect.php?banner_id=100&url=/ _______________________________________________ Flightgear-devel mailing list Flightgear-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/flightgear-devel
