New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Part of #9473: Create Vehicle::UpdateMotionCollisionDetection #11949
Part of #9473: Create Vehicle::UpdateMotionCollisionDetection #11949
Conversation
Question: should I change the xyz parameters to use Coords xyz? They're 16 bits, while Coords are 32 bits |
I guess yeah, ultimately we'll have to do it anyway, might as well do it now, the larger types shouldn't make a difference |
Yes it should be a coordsxyz. The size of the types doesn't matter for this. We have standardised towards 32bit as gives us room to expand in the future potentially. |
519e0fc
to
4764bf7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just small notes on readability
78a7f2a
to
38ce2d0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just some small simplifications
Converted from vehicle_update_motion_collision_detection
38ce2d0
to
95ec63f
Compare
Converted from vehicle_update_motion_collision_detection