Skip to content

Removed unused var warning#185

Closed
ahcorde wants to merge 1 commit intorollingfrom
ahcorde/rolling/unused_variables_warning
Closed

Removed unused var warning#185
ahcorde wants to merge 1 commit intorollingfrom
ahcorde/rolling/unused_variables_warning

Conversation

@ahcorde
Copy link
Contributor

@ahcorde ahcorde commented Apr 16, 2025

Related with this nightly clang build https://ci.ros2.org/view/nightly/job/nightly_linux_clang_libcxx/2219/clang/folder.-1798993873/

There are some unused variables.

Signed-off-by: Alejandro Hernandez Cordero <ahcorde@gmail.com>
@ahcorde ahcorde self-assigned this Apr 16, 2025
@ahcorde
Copy link
Contributor Author

ahcorde commented Apr 16, 2025

Pulls: #185
Gist: https://gist.githubusercontent.com/ahcorde/d9b31303884c48130ce84f2f49469842/raw/ea9faa55ab8b5657b8de02829292ad41a71dc8ba/ros2.repos
BUILD args: --packages-above-and-dependencies rmw_connextdds_common
TEST args: --packages-above rmw_connextdds_common
ROS Distro: rolling
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/15710

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
  • Windows Build Status

Copy link
Collaborator

@fujitatomoya fujitatomoya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ahcorde

actually i was trying to address the same issue with #184 a week ago, but holding it for branching.

you can pick either 😄

@ahcorde ahcorde closed this Apr 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants