Melchior FRANZ wrote:
* Curtis L. Olson -- Tuesday 04 October 2005 20:52:
For what it's worth, I don't like this patch.
I find the hole more annoying. Unfortunately, I can't fix what
you think is the real problem. Shall I revert for now?
I'm not saying the hole isn't annoying, I'm just saying that there is a
bug because for some reason, the sim thinks you are > 10 meters AGL when
you are sitting on the carrier deck. There is some ground intersection
problem going on there. If the ground interesection was computed
correctly, the system would think you are < 10 meters AGL and everything
would work the way it is intended.
I'd really like for this to get fixed the right way. When we slap on
bandaids without fixing the underlying problems, we end up with a system
that has a lot of bandaids on top of a rotting infrastructure.
Similarly whenever we see a stray crash or segfault we should pursue it
with our utmost agression and stamp those out right away. Anytime we
leave these sorts of crashes and problems for later, we end up with a
system full of unexpected, unexplained, impossible to debug crashes.
That kind of software is an incredible pain to operate.
In the past I had more time to defend against these things, right now I
don't. You've been granted CVS commit access so use your best
judgement. I'd just hate to have this slip through the cracks, and when
someone tries to land on an object that is 50.01 meters tall or more,
they are going to get a hole again. We could just remove that check and
leave the near clip plane in close all the time, but then our terrain
rendering will really stink for anyone with a 16bit depth buffer ...
It's not an easy problem, but slapping a bandaid ontop will probably
mask it long enough so that the person who introduced the orignal
problem will be long gone before we get bit again and no one will know
how to fix it ...
Regards,
Curt.
--
Curtis Olson http://www.flightgear.org/~curt
HumanFIRST Program http://www.humanfirst.umn.edu/
FlightGear Project http://www.flightgear.org
Unique text: 2f585eeea02e2c79d7b1d8c4963bae2d
_______________________________________________
Flightgear-devel mailing list
[email protected]
http://mail.flightgear.org/mailman/listinfo/flightgear-devel
2f585eeea02e2c79d7b1d8c4963bae2d
