TrigT1CaloTools: VarHandle migration.

Use a ReadHandle to access xAOD::EventInfo rather than directly retrieving EventInfo from SG.

This merge request affects 1 package:

• Trigger/TrigT1/TrigT1CaloTools

added Trigger master review-pending-level-1 labels

CI Result SUCCESS

	Athena	AthSimulation
externals
cmake
make
required tests
optional tests

Full details available at NICOS MR-20808-2019-02-01-11-14
Athena: number of compilation errors 0, warnings 1
AthSimulation: number of compilation errors 0, warnings 0
CI Jenkins server is switched to https://atlas-sit-ci.cern.ch. It is accessible world-wide (behind CERN SSO). In old links to Jenkins server aibuild080.cern.ch:8080 should be replaced with atlas-sit-ci.cern.ch For experts only: Jenkins output [CI-MERGE-REQUEST 33115]

Hi @ssnyder.

It all looks fine to me, but I see you've removed the ASG warnings if it can't determine the run. Do we want those back in, or it is not necessary?

Clara (L1)

added review-user-action-required label and removed review-pending-level-1 label

Not necessary. Getting the EventContext can't fail, and having a configured input missing is a hard error in MT.

Hi @ssnyder,

Ok, thank you for the explanation. I'm happy to approve.

Cheers, Clara

added review-approved label and removed review-user-action-required label

merged

mentioned in commit c10ca40d

added sweep:ignore label