fix(autoware_motion_velocity_obstacle_velocity_limiter_module): fix functionStatic

[kobayu858]

Description

This is a fix based on cppcheck functionStatic warnings.

Since each function clearly changes its member variables, it is assumed to be a false positive for cppcheck.
I believe the reason for the false positive is that the function mechanism itself does not depend on the member variables.

planning/motion_velocity_planner/autoware_motion_velocity_obstacle_velocity_limiter_module/src/parameters.hpp:73:8: performance: inconclusive: Technically the member function 'autoware::motion_velocity_planner::obstacle_velocity_limiter::ObstacleParameters::updateType' can be static (but you may consider moving to unnamed namespace). [functionStatic]
  bool updateType(const rclcpp::Logger & logger, const std::string & type)
       ^

planning/motion_velocity_planner/autoware_motion_velocity_obstacle_velocity_limiter_module/src/parameters.hpp:143:8: performance: inconclusive: Technically the member function 'autoware::motion_velocity_planner::obstacle_velocity_limiter::ProjectionParameters::updateModel' can be static (but you may consider moving to unnamed namespace). [functionStatic]
  bool updateModel(const rclcpp::Logger & logger, const std::string & model_str)
       ^

planning/motion_velocity_planner/autoware_motion_velocity_obstacle_velocity_limiter_module/src/parameters.hpp:157:8: performance: inconclusive: Technically the member function 'autoware::motion_velocity_planner::obstacle_velocity_limiter::ProjectionParameters::updateDistanceMethod' can be static (but you may consider moving to unnamed namespace). [functionStatic]
  bool updateDistanceMethod(const rclcpp::Logger & logger, const std::string & method_str)
       ^

Related links

Parent Issue:

• Link

How was this PR tested?

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.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 23.89%. Comparing base (1f32972) to head (83d41dc).
Report is 7 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #8483      +/-   ##
==========================================
+ Coverage   23.88%   23.89%   +0.01%     
==========================================
  Files        1380     1385       +5     
  Lines      101833   101847      +14     
  Branches    38776    38764      -12     
==========================================
+ Hits        24320    24334      +14     
- Misses      75009    75050      +41     
+ Partials     2504     2463      -41     

Flag	Coverage Δ		*Carryforward flag
differential	18.16% <ø> (?)
total	23.89% <ø> (+0.01%)	⬆️	Carriedforward from 9d522ea

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