On Jan 5, 2008 8:18 PM, Stuart Buchanan <[EMAIL PROTECTED]> wrote:
>
> I think that rather than working around the AI property reuse issue,
> we should simply remove all the children of the AI node after it is marked as
> valid=false. Probably something around line 152 of AIManager.cxx.
>
> However, I'm puzzled as to why it doesn't do that already, so I suspect I'm 
> missing
> something.
>
> Does anyone have any idea why the subtree under an AI model marked as 
> valid=false
> isn't deleted?

Because the reuse was introduced for some animations that depended on
the nodes remaining the same.
That was actually for the old "xml" radar. Looks like it causes all
sort of trouble so maybe we should indeed
revert back to deleting the subtree.

Also, I think that my nasal fix introduces a race condition: if the
message isn't displayed because the pilot disconnects then it might be
shown with the wrong source after the node gets reused. So it would be
better to fix it from c++.

-- 
Csaba/Jester

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Flightgear-devel mailing list
Flightgear-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/flightgear-devel

Reply via email to