23.0-cppcheck-ISF_FastCaloSimSD

[minor] cppcheck defect: prevent negative index for array in case of no events processed.

assigned to @sroe

changed the description

This merge request affects 1 package:

• Simulation/ISF/ISF_FastCaloSim/ISF_FastCaloSimSD

This merge request affects 1 file:

• Simulation/ISF/ISF_FastCaloSim/ISF_FastCaloSimSD/src/LArFCS_StepInfoSD.cxx

Adding @jchapman ,@pjacka ,@mfauccig ,@ahasib ,@schaarsc ,@tadej ,@xiaozhon as watchers

added Simulation master review-pending-level-1 labels

There were multiple CI triggers for this MR and commit. The system ignored duplicates but the GitLab pipeline status may incorrectly show the job as failed. Once the remaining job finished running, the CI results will be posted as usual.

CI Result SUCCESS (hash 9e3f2ed5)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
externals
cmake
make
tests

Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 57444]

Looks good, approving - L1

added review-approved label and removed review-pending-level-1 label

merged

mentioned in commit 7323793e

added sweep:ignore label