gazebosim / gz-sim

Open source robotics simulator. The latest version of Gazebo.
https://gazebosim.org
Apache License 2.0
685 stars 262 forks source link

Blast545/fix unused gcc warnings #2388

Closed Blast545 closed 5 months ago

Blast545 commented 5 months ago

🦟 Bug fix

Fixes gcc warnings appearing in CI: https://build.osrfoundation.org/job/gz_sim-ci-main-jammy-amd64/66/gcc/

Summary

Does not change at all the tests, just voids the variables so the compiler takes them as read and won't emit the warning

Checklist

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

codecov[bot] commented 5 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 65.91%. Comparing base (cffd297) to head (e986c4b).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## gz-sim8 #2388 +/- ## ======================================== Coverage 65.91% 65.91% ======================================== Files 327 327 Lines 31314 31314 ======================================== Hits 20641 20641 Misses 10673 10673 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Blast545 commented 5 months ago

:farmer: Green CI, except on windows, INTEGRATION_mesh_inertia_calculation.test_ran seems to be flaky. FYI @Crola1702

Merging!