refactor(bpp): remove virtual qualifier

[satoshi-ota]

Description

Hide updateCurrentState() as private function. Inherit class only can override state transit conditions, and interface decides next state based on them.

This PR contains following modifications.

• remove virtual qualifier from updateState()
• remove virtual qualifier from updateCurrentState()
• move current_state_ to private

Tests performed

Psim

Effects on system behavior

Nothing.

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

• I've confirmed the contribution guidelines.
• The PR follows the pull request guidelines.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

• The PR follows the pull request guidelines.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

• There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

[zulfaqar-azmi-t4]

LGTM

[codecov]

Codecov Report

Attention: 172 lines in your changes are missing coverage. Please review.

Comparison is base (1d6e7e8) 15.28% compared to head (828141d) 15.28%.
Report is 7 commits behind head on main.

Files	Patch %	Lines
...nning/behavior_path_avoidance_module/src/utils.cpp	0.00%	82 Missing ⚠️
...behavior_path_lane_change_module/src/interface.cpp	0.00%	70 Missing and 1 partial ⚠️
...ing/behavior_path_lane_change_module/src/scene.cpp	0.00%	7 Missing ⚠️
...nning/behavior_path_avoidance_module/src/scene.cpp	0.00%	6 Missing ⚠️
...path_avoidance_by_lane_change_module/src/scene.cpp	0.00%	4 Missing ⚠️
...ude/behavior_path_lane_change_module/interface.hpp	0.00%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #5968      +/-   ##
==========================================
- Coverage   15.28%   15.28%   -0.01%     
==========================================
  Files        1750     1750              
  Lines      120568   120599      +31     
  Branches    36781    36776       -5     
==========================================
  Hits        18434    18434              
- Misses      81469    81499      +30     
- Partials    20665    20666       +1     

Flag	Coverage Δ		*Carryforward flag
differential	11.78% <0.00%> (?)
total	15.30% <ø> (+0.01%)	⬆️	Carriedforward from 1d6e7e8

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