refactor(behavior_path_planner): refactor pull_over

[kosuke55]

Description

refactor pull_over

• remove unnecessary functions and variables, args.
• refactor each function

Related links

Tests performed

psim

Notes for reviewers

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.
• The PR has been properly tested.
• The PR has been reviewed by the code owners.

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.
• The PR is ready for merge.

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

[codecov]

Codecov Report

Merging #1842 (3f35ba0) into main (cb2552b) will decrease coverage by 0.12%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #1842      +/-   ##
==========================================
- Coverage   10.44%   10.32%   -0.13%     
==========================================
  Files        1221     1221              
  Lines       87728    89055    +1327     
  Branches    20493    21431     +938     
==========================================
+ Hits         9160     9191      +31     
- Misses      69091    70299    +1208     
- Partials     9477     9565      +88     

Flag	Coverage Δ		*Carryforward flag
differential	2.49% <0.00%> (?)
total	10.42% <0.00%> (ø)		Carriedforward from cb2552b

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

Impacted Files	Coverage Δ
...lanner/scene_module/pull_over/pull_over_module.hpp	0.00% <ø> (ø)
...lanner/include/behavior_path_planner/utilities.hpp	23.52% <ø> (-1.48%)	⬇️
...or_path_planner/src/behavior_path_planner_node.cpp	0.43% <0.00%> (+0.21%)	⬆️
...er/src/scene_module/pull_over/pull_over_module.cpp	0.00% <0.00%> (ø)
...r_path_planner/src/scene_module/pull_over/util.cpp	0.00% <0.00%> (ø)
planning/route_handler/src/route_handler.cpp	0.00% <0.00%> (ø)
...nning/behavior_path_planner/src/path_utilities.cpp	0.00% <0.00%> (ø)
... and 5 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[takayuki5168]

L