On 23 Oct 2008, at 02:19, Csaba Halász wrote:

>
> Of course it isn't valid, inside the function it is only setting a
> local variable (pass by value), and it does not return anything.
> As the _runway argument is output only, there is no reason to pass it
> as argument. Rather, the function should have a return type of
> FGRunway*.

Grrr - kicking myself for not seeing this, this is all because  
FGRunway was on the stack previously, and the Mk-VIII didn't use a  
reference for the out parameter. Returning a pointer is of course  
cleaner, I even thought about doing that re-factoring, without  
spotting that the our parameter was broken.

Thanks for figuring this out.

James


-------------------------------------------------------------------------
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