Trk::Track Use the new type functionality from surfaces to avoid dynamic_casts

• There are now methods to deduce directly the surfaceType (without inspecting the associated ptr)
• So we can use the type (we have specific bounded number of types)
• Also make sure we return the ptr (from a DataVector so not owned ) in the cachedValue only if it is Valid, otherwise nullptr (this seems to never happen ...)

RunTier0 seems fine RunTier0Tests.log

This merge request affects 1 package:

• Tracking/TrkEvent/TrkTrack

Adding @amorley as watcher

added Tracking master review-pending-level-1 labels

CI Result SUCCESS (hash f5c0f848)

	Athena	AthSimulation	AnalysisBase	AthGeneration
externals
cmake
make
required tests
optional tests

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

Clear, approving.

Pavol [as L2 MR shifter]

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

merged

mentioned in commit fbe73129

added sweep:ignore label