Norman Vine writes:
> Oh yea - there is an indexing bug in the FAST TRISTRIP logic that 
> somehow never got changed even though the fix was in several of 
> my submissions

For what it's worth, separating out individual concepts into separate
patch submissions makes things a lot easier and simpler for the patch
applier.  Mixing a bunch of stuff into a single submission can make it
really difficult to sort out what's going on, and without blindly
applying everything (which I don't like to do) it's really easy to
miss something.  Sometimes it's impossible to just blindly apply a
whole patch when the submission isn't against the latest cvs.  If you
take the view that the project coordinator is a complete dummy and
make things as simple as possible for him, that can be very
helpful. :-)

> maybe this has finally reared it's head if the scenery has hit a 
> tristrip but I didn't think we were actualy using any yet
> 
> easiest to check by just falling back to the 'slightly' slower version
> by changing the lines in 
> 
> void FGHitList::IntersectBranch( ssgBranch *branch, sgdMat4 m,
>                                  sgdVec3 orig, sgdVec3 dir )
> ....
> 
> -       GLenum primType = ((ssgLeaf *)kid)->getPrimitiveType();
> -       IntersectLeaf( (ssgLeaf *)kid, m, orig_leaf, dir_leaf, primType );
> 
> to
> 
> +         IntersectLeaf( (ssgLeaf *)kid, m, orig_leaf, dir_leaf );

I tried this change and it didn't seem to have any affect on the
problem. :-(

Curt.
-- 
Curtis Olson   IVLab / HumanFIRST Program       FlightGear Project
Twin Cities    [EMAIL PROTECTED]                  [EMAIL PROTECTED]
Minnesota      http://www.menet.umn.edu/~curt   http://www.flightgear.org

_______________________________________________
Flightgear-devel mailing list
[EMAIL PROTECTED]
http://mail.flightgear.org/mailman/listinfo/flightgear-devel

Reply via email to