raycast redundancy

[Jacky_J]

isn't it a little redundant to pass the start and end vector twice when you need to raycast the world?

i.e.

Code: Select all

btCollisionWorld::ClosestRayResultCallback Result(Start, End);
PhysicsWorld->getCollisionWorld()->rayTest(Start, End, Result);

I think rayTest() should only have one parameter, ClosestRayResultCallback.

[Erwin Coumans]

btCollisionWorld::rayTest accepts various RayResultCallback structures, and you can define your own that doesn't need the ray from/to.

We could remove from/to from the ClosestRayResultCallback, it is just a convenience thing.

We can add some examples for btCollisionWorld::rayTest to calculate _all_ hits, and to calculate the first (not necessary closest) hit, without the need to add from/to twice.

Hope this helps,
Erwin