fix(reaction_analyzer): fix unreadVariable

[kobayu858]

Description

This is a fix based on cppcheck unreadVariable warnings.

tools/reaction_analyzer/src/utils.cpp:225:27: style: Variable 'current_distance' is assigned a value that is never used. [unreadVariable]
  double current_distance = 0.0;
                          ^

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

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

Please upload report for BASE (main@e0fa943). Learn more about missing BASE report.
Report is 104 commits behind head on main.

Files	Patch %	Lines
tools/reaction_analyzer/src/utils.cpp	0.00%	1 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #8373   +/-   ##
=======================================
  Coverage        ?   23.72%           
=======================================
  Files           ?     1379           
  Lines           ?   101368           
  Branches        ?    38515           
=======================================
  Hits            ?    24047           
  Misses          ?    74866           
  Partials        ?     2455           

Flag	Coverage Δ		*Carryforward flag
differential	0.00% <0.00%> (?)
total	23.72% <ø> (?)		Carriedforward from ba34878

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

[kobayu858]

@brkay54

Please confirm the PR. Thank you very much.

[brkay54]

No need to define current_distance before the loop. LGTM. Thanks.

[brkay54]

Sorry for the inconvenience, it seems there is another PR which proposed almost same changes:

• fix(reaction_analyzer): fix variableScope #8450

making const double is better so IMO we can close this PR.

[veqcc]

@brkay54
Thank you for the reply!! I will colse this PR instead of @kobayu858 ans merge #8450