fix(autoware_tracking_object_merger): fix unusedFunction

[kobayu858]

Description

This is a fix based on cppcheck unusedFunction warnings.

perception/autoware_tracking_object_merger/src/utils/utils.cpp:301:0: style: The function 'objectsYawIsReverted' is never used. [unusedFunction]
bool objectsYawIsReverted(const TrackedObject & main_obj, const TrackedObject & sub_obj)
^

perception/autoware_tracking_object_merger/src/utils/utils.cpp:413:0: style: The function 'probabilityMerger' is never used. [unusedFunction]
float probabilityMerger(const float main_prob, const float sub_prob, const MergePolicy policy)
^

perception/autoware_tracking_object_merger/src/utils/utils.cpp:428:0: style: The function 'shapeMerger' is never used. [unusedFunction]
autoware_perception_msgs::msg::Shape shapeMerger(
^

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 24.01%. Comparing base (60cc9c7) to head (162a41c).
Report is 54 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #8578   +/-   ##
=======================================
  Coverage   24.01%   24.01%           
=======================================
  Files        1384     1384           
  Lines      102173   102161   -12     
  Branches    38947    38939    -8     
=======================================
  Hits        24536    24536           
+ Misses      75138    75126   -12     
  Partials     2499     2499           

Flag	Coverage Δ		*Carryforward flag
differential	0.00% <ø> (?)
total	24.01% <ø> (+<0.01%)	⬆️	Carriedforward from 60cc9c7

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

[technolojin]

For my point of view, those functions provide additional functionalities for Autoware users. Even it is not used in current Autoware, someone can use those.

@YoshiRi @yukkysaito
Do you think those unused functions should be removed?

[YoshiRi]

@technolojin
As you say I intended to use those function in the future development.
But this is trivial, so I think we can remove this.
Ref: https://star4.slack.com/archives/C4P0NSMB5/p1724040335846959

[YoshiRi]

LGTM. Go ahead.