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

Reply via email to