-
Notifications
You must be signed in to change notification settings - Fork 274
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
TF causing casting compiler warnings in strict mode #126
Comments
This is doing byte ordering swapping for endianness. Its a valid warning in general but in this case it's actually correct. If you'd like to submit a PR to suppress the warnings that would be considered. https://github.com/ros/geometry/blob/indigo-devel/tf/include/tf/LinearMath/Vector3.h#L626 Though this is a fork of the bullet linear math library and it would be better to submit the patch upstream and we can port back the changes. https://github.com/bulletphysics/bullet3/blob/master/src/LinearMath/btVector3.h#L1231 We've had trouble in the past due to diverging from upstream when we applied patches locally that weren't accepted upstream. |
Adding to my random ROS package's CMake:
TF starts having lots of warnings, e.g. in
/opt/ros/jade/include/tf/LinearMath/Vector3.h
The text was updated successfully, but these errors were encountered: