Skip to content

Fix clang/gcc warnings #178

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

Closed
christophebedard opened this issue May 21, 2025 · 2 comments · Fixed by #179
Closed

Fix clang/gcc warnings #178

christophebedard opened this issue May 21, 2025 · 2 comments · Fixed by #179
Assignees
Labels
bug Something isn't working

Comments

@christophebedard
Copy link
Member

Generated by Generative AI

No response

Operating System:

Ubuntu

ROS version or commit hash:

Rolling (latest)

RMW implementation (if applicable):

No response

RMW Configuration (if applicable):

No response

Client library (if applicable):

No response

'ros2 doctor --report' output

No response

Steps to reproduce issue

  1. Build tracetools using the current rolling branch with clang and libc++:
    $ colcon build --packages-up-to tracetools --mixin clang-libcxx --cmake-args -DCMAKE_BUILD_TYPE=Debug
    
  2. Check the build output for tracetools and see if warnings are reported

Expected behavior

No issues/warnings

Actual behavior

There are some issues reported by clang, see #115.

Some of those issues may have been fixed since that PR was opened over a year ago.

Additional information

After confirming that there are warnings, try to fix them. Look at the changes in #115.

After fixing the warnings, re-build and confirm that the warnings disappeared.

@christophebedard christophebedard added the bug Something isn't working label May 21, 2025
@christophebedard
Copy link
Member Author

@ShravanDeva5327 I created this issue and included instructions on how to build with clang so that you can reproduce the build warnings and try to fix them.

Let me know if you have any questions.

@ShravanDeva5327
Copy link
Contributor

@christophebedard Thankyou for these instructions, I'll start looking into it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants