Bart/bullet3
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

set broadphase handle to zero when destructing a btCollisionWorld, just in case a collision object/ rigid body is re-used.

Browse Source 
Thanks to Ole for reporting

Fix a bug , causing some deep penetrations to be missed. Not likely to have impacted many users, as such deep penetrations should have prevented in the first place.
Thanks a lot to Andy O'Neil for reporting and reproduction case.
This commit is contained in:
erwin.coumans
2008-12-05 22:40:13 +00:00
parent b89b71b38d
commit 635e1a6aa0
2 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
src/BulletCollision/CollisionDispatch/btCollisionWorld.cpp
View File

@@ -69,6 +69,7 @@ btCollisionWorld::~btCollisionWorld()
// //
getBroadphase()->getOverlappingPairCache()->cleanProxyFromPairs(bp,m_dispatcher1); getBroadphase()->getOverlappingPairCache()->cleanProxyFromPairs(bp,m_dispatcher1);
getBroadphase()->destroyProxy(bp,m_dispatcher1); getBroadphase()->destroyProxy(bp,m_dispatcher1);
collisionObject->setBroadphaseHandle(0);
} }
} }

3
src/BulletCollision/NarrowPhaseCollision/btGjkPairDetector.cpp
View File

@@ -150,7 +150,8 @@ void btGjkPairDetector::getClosestPoints(const ClosestPointInput& input,Result&
// potential exit, they don't overlap // potential exit, they don't overlap
if ((delta > btScalar(0.0)) && (delta * delta > squaredDistance * input.m_maximumDistanceSquared)) if ((delta > btScalar(0.0)) && (delta * delta > squaredDistance * input.m_maximumDistanceSquared))
{ {
checkPenetration = false; checkSimplex=true;
//checkPenetration = false;
break; break;
} }