Skip to content
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

fix(tier4_vehicle_rviz_plugin): fix cppcheck warning of virtualCallInConstructor #8379

Conversation

taisa1
Copy link
Contributor

@taisa1 taisa1 commented Aug 6, 2024

Description

This is a fix based on cppcheck warning virtualCallInConstructor.

common/tier4_vehicle_rviz_plugin/src/tools/jsk_overlay_utils.hpp:112:16: style: Virtual function 'hide' is called from destructor '~OverlayObject()' at line 116. Dynamic binding is not used. [virtualCallInConstructor]

Related links

Private Links:

How was this PR tested?

Only checked if build succeeded.

Notes for reviewers

None.

Interface changes

None.

Effects on system behavior

None.

@github-actions github-actions bot added the component:common Common packages from the autoware-common repository. (auto-assigned) label Aug 6, 2024
Copy link

github-actions bot commented Aug 6, 2024

Thank you for contributing to the Autoware project!

🚧 If your pull request is in progress, switch it to draft mode.

Please ensure:

@atsushi421 atsushi421 added the tag:run-build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci) label Aug 6, 2024
Copy link

codecov bot commented Aug 6, 2024

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 23.88%. Comparing base (9c95d21) to head (99467e1).

Files Patch % Lines
...ehicle_rviz_plugin/src/tools/jsk_overlay_utils.cpp 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8379      +/-   ##
==========================================
- Coverage   23.88%   23.88%   -0.01%     
==========================================
  Files        1380     1380              
  Lines      101584   101590       +6     
  Branches    38637    38640       +3     
==========================================
  Hits        24267    24267              
- Misses      74887    74893       +6     
  Partials     2430     2430              
Flag Coverage Δ *Carryforward flag
differential 0.00% <0.00%> (?)
total 23.88% <ø> (+<0.01%) ⬆️ Carriedforward from 9c95d21

*This pull request uses carry forward flags. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@taisa1 taisa1 marked this pull request as ready for review August 6, 2024 08:30
@taisa1 taisa1 requested a review from yukkysaito as a code owner August 6, 2024 08:30
@veqcc veqcc merged commit c28606b into autowarefoundation:main Aug 14, 2024
29 of 30 checks passed
xtk8532704 pushed a commit to tier4/autoware.universe that referenced this pull request Aug 15, 2024
…Constructor (autowarefoundation#8379)

fix: deal with virtualCallInConstructor warning

Signed-off-by: taisa1 <[email protected]>
Co-authored-by: Ryuta Kambe <[email protected]>
Signed-off-by: xtk8532704 <[email protected]>
@taisa1 taisa1 deleted the fix_virtual_call_in_constructor_tier4_vehicle_rviz_plugin branch August 27, 2024 05:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:common Common packages from the autoware-common repository. (auto-assigned) tag:run-build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants