Issue in btKinematicCharacterController::stepDown() needs review

[samhocevar]

The function btKinematicCharacterController::stepDown() in btKinematicCharacterController.cpp has the following code at line 567:

if (m_ghostObject->hasContactResponse() && (callback.hasHit() &&
        needsCollision(m_ghostObject, callback.m_hitCollisionObject)) || runonce == true)

There are parentheses around the second && but no parentheses to make it explicit that the branch will always be taken when runonce is true, which makes me suspect this code does not do what it should.

The code used to look like this:

if ((callback.hasHit() || runonce == true) && m_ghostObject->hasContactResponse() &&
         needsCollision(m_ghostObject, callback.m_hitCollisionObject))