fix(behavior_velocity_planner): fix cppcheck warnings of virtualCallInConstructor

[taisa1]

Description

This is a fix based on cppcheck warning virtualCallInConstructor.

planning/behavior_velocity_planner/autoware_behavior_velocity_blind_spot_module/src/manager.hpp:38:16: style: Virtual function 'getModuleName' is called from constructor 'BlindSpotModuleManager(rclcpp::Node&node)' at line 35. Dynamic binding is not used. [virtualCallInConstructor]
planning/behavior_velocity_planner/autoware_behavior_velocity_crosswalk_module/src/manager.hpp:45:16: style: Virtual function 'getModuleName' is called from constructor 'CrosswalkModuleManager(rclcpp::Node&node)' at line 37. Dynamic binding is not used. [virtualCallInConstructor]
planning/behavior_velocity_planner/autoware_behavior_velocity_detection_area_module/src/manager.hpp:37:16: style: Virtual function 'getModuleName' is called from constructor 'DetectionAreaModuleManager(rclcpp::Node&node)' at line 40. Dynamic binding is not used. [virtualCallInConstructor]
planning/behavior_velocity_planner/autoware_behavior_velocity_no_drivable_lane_module/src/manager.hpp:37:16: style: Virtual function 'getModuleName' is called from constructor 'NoDrivableLaneModuleManager(rclcpp::Node&node)' at line 29. Dynamic binding is not used. [virtualCallInConstructor]
planning/behavior_velocity_planner/autoware_behavior_velocity_no_stopping_area_module/src/manager.hpp:37:16: style: Virtual function 'getModuleName' is called from constructor 'NoStoppingAreaModuleManager(rclcpp::Node&node)' at line 40. Dynamic binding is not used. [virtualCallInConstructor]
planning/behavior_velocity_planner/autoware_behavior_velocity_occlusion_spot_module/src/manager.hpp:47:16: style: Virtual function 'getModuleName' is called from constructor 'OcclusionSpotModuleManager(rclcpp::Node&node)' at line 37. Dynamic binding is not used. [virtualCallInConstructor]
planning/behavior_velocity_planner/autoware_behavior_velocity_run_out_module/src/manager.hpp:33:16: style: Virtual function 'getModuleName' is called from constructor 'RunOutModuleManager(rclcpp::Node&node)' at line 41. Dynamic binding is not used. [virtualCallInConstructor]
planning/behavior_velocity_planner/autoware_behavior_velocity_speed_bump_module/src/manager.hpp:37:16: style: Virtual function 'getModuleName' is called from constructor 'SpeedBumpModuleManager(rclcpp::Node&node)' at line 39. Dynamic binding is not used. [virtualCallInConstructor]
planning/behavior_velocity_planner/autoware_behavior_velocity_stop_line_module/src/manager.hpp:42:16: style: Virtual function 'getModuleName' is called from constructor 'StopLineModuleManager(rclcpp::Node&node)' at line 32. Dynamic binding is not used. [virtualCallInConstructor]
planning/behavior_velocity_planner/autoware_behavior_velocity_template_module/src/manager.hpp:54:16: style: Virtual function 'getModuleName' is called from constructor 'TemplateModuleManager(rclcpp::Node&node)' at line 36. Dynamic binding is not used. [virtualCallInConstructor]
planning/behavior_velocity_planner/autoware_behavior_velocity_traffic_light_module/src/manager.hpp:38:16: style: Virtual function 'getModuleName' is called from constructor 'TrafficLightModuleManager(rclcpp::Node&node)' at line 36. Dynamic binding is not used. [virtualCallInConstructor]
planning/behavior_velocity_planner/autoware_behavior_velocity_virtual_traffic_light_module/src/manager.hpp:37:16: style: Virtual function 'getModuleName' is called from constructor 'VirtualTrafficLightModuleManager(rclcpp::Node&node)' at line 42. Dynamic binding is not used. [virtualCallInConstructor]
planning/behavior_velocity_planner/autoware_behavior_velocity_walkway_module/src/manager.hpp:43:16: style: Virtual function 'getModuleName' is called from constructor 'WalkwayModuleManager(rclcpp::Node&node)' at line 35. Dynamic binding is not used. [virtualCallInConstructor]

Related links

Private Links:

• TIER IV internal link: About virtualCallInConstructor

How was this PR tested?

Only checked if build succeeded.

Notes for reviewers

None.

Interface changes

None.

Effects on system behavior

None.

[github-actions]

Thank you for contributing to the Autoware project!

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

Please ensure:

• You've checked our contribution guidelines.
• Your PR follows our pull request guidelines.
• All required CI checks pass before marking the PR ready for review.

[shmpwk]

LGTM

[codecov]

Codecov Report

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

Project coverage is 29.21%. Comparing base (2c19f1e) to head (8095020).
Report is 55 commits behind head on main.

Files	Patch %	Lines
..._behavior_velocity_template_module/src/manager.cpp	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #8376      +/-   ##
==========================================
+ Coverage   23.90%   29.21%   +5.31%     
==========================================
  Files        1380     1615     +235     
  Lines      101750   119459   +17709     
  Branches    38714    51412   +12698     
==========================================
+ Hits        24324    34902   +10578     
- Misses      74920    75251     +331     
- Partials     2506     9306    +6800     

Flag	Coverage Δ		*Carryforward flag
differential	18.19% <92.85%> (?)
total	29.29% <ø> (+5.39%)	⬆️	Carriedforward from 9b7ae6b

*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.