Real-Time Physics Simulation Forum

Re: btDispatcher::dispatchAllCollisionPairs

2014-12-03T18:23:34+00:00

I know this is really old, but this discrepancy still hasn't been fixed. Why's that?

Statistics: Posted by Slight0 — Wed Dec 03, 2014 6:23 pm

Re: btDispatcher::dispatchAllCollisionPairs

2010-09-29T00:30:33+00:00

Good find, I don't recall any reason so it probably slipped in by accident during some refactoring a few years ago:

http://code.google.com/p/bullet/source/ ... spatcher.h
http://code.google.com/p/bullet/source/ ... atcher.cpp

Thanks,
Erwin

Statistics: Posted by Erwin Coumans — Wed Sep 29, 2010 12:30 am

btDispatcher::dispatchAllCollisionPairs

2010-09-28T23:52:31+00:00

Is there a subtle reason btDispatcher::dispatchAllCollisionPairs() takes a btDispatcher* parameter?

Code:

void btCollisionDispatcher::dispatchAllCollisionPairs(    btOverlappingPairCache* pairCache,    const btDispatcherInfo& dispatchInfo,    btDispatcher* dispatcher){    btCollisionPairCallback collisionCallback(dispatchInfo, this);    pairCache->processAllOverlappingPairs(&collisionCallback, dispatcher);}

"this" is used to create new algorithms; and "dispatcher" is used by the pair cache to free algorithms. So as far as I can tell, it would be an error if "this != dispatcher".

Statistics: Posted by paul.dubois — Tue Sep 28, 2010 11:52 pm