main-coverity-LArL1Sim

solve coverity defects: Integer overflow and possible resource leak.

This merge request affects 1 package:

• LArCalorimeter/LArL1Sim

This merge request affects 2 files:

• LArCalorimeter/LArL1Sim/LArL1Sim/LArTTL1Maker.h
• LArCalorimeter/LArL1Sim/src/LArTTL1Maker.cxx

Adding @pavol as watcher

added LAr main review-pending-level-1 labels

CI Result SUCCESS (hash 9d2730f9)

	Athena
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
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 9116] (remote access info)

The changes are good to me. Approved from L1.

Something indirectly related to this MR: the indentations in LArCalorimeter/LArL1Sim/src/LArTTL1Maker.cxx mix tabs and spaces and show some disagreements from the view of git website. It's better to clean them up.

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

merged

mentioned in commit abefb474

resolved all threads