egammaEnergyPositionAllSamples: Clarify / fix if structure

Refactor the code, so as to be more obvious what we try to do when there are cells in both endcap and barrel.

added 2 commits

• 0229e47b - try to re-organise the if
• b1f70d5e - try to re-organise the if

Compare with previous version

Mentioning @turra as he brought this up

This merge request affects 1 package:

• Reconstruction/egamma/egammaUtils

Adding @goetz as watcher

added Egamma Reconstruction master review-pending-level-1 labels

This merge request affects 1 package:

• Reconstruction/egamma/egammaUtils

Adding @goetz as watcher

added 1 commit

• 2ca4486a - indentation

Compare with previous version

This merge request affects 1 package:

• Reconstruction/egamma/egammaUtils

Adding @goetz as watcher

CI Result SUCCESS (hash b1f70d5e)

	Athena	AthSimulation
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 8462]

CI Result SUCCESS (hash b59eb2c7)

	Athena	AthSimulation
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 8461]

CI Result SUCCESS (hash 2ca4486a)

	Athena	AthSimulation
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 8466]

code changes look fine, CI tests passed, approving.

Roman (L1)

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

merged

mentioned in commit c6679ca5

added sweep:ignore label

mentioned in merge request !30475 (merged)

resolved all threads