Trk::Surface Move inline methods to icc, add final where applicable/possible

• Move inline methods to separate .icc as per relevant ATLAS style guideline
• Add final where possible applicable

This merge request affects 1 package:

• Tracking/TrkDetDescr/TrkSurfaces

This merge request affects 10 files:

• Tracking/TrkDetDescr/TrkSurfaces/TrkSurfaces/AnnulusBounds.h
• Tracking/TrkDetDescr/TrkSurfaces/TrkSurfaces/DiscTrapezoidalBounds.h
• Tracking/TrkDetDescr/TrkSurfaces/TrkSurfaces/NoBounds.h
• Tracking/TrkDetDescr/TrkSurfaces/TrkSurfaces/NoBounds.icc
• Tracking/TrkDetDescr/TrkSurfaces/TrkSurfaces/RectangleBounds.h
• Tracking/TrkDetDescr/TrkSurfaces/TrkSurfaces/RectangleBounds.icc
• Tracking/TrkDetDescr/TrkSurfaces/TrkSurfaces/RotatedDiamondBounds.h
• Tracking/TrkDetDescr/TrkSurfaces/TrkSurfaces/RotatedDiamondBounds.icc
• Tracking/TrkDetDescr/TrkSurfaces/TrkSurfaces/RotatedTrapezoidBounds.h
• Tracking/TrkDetDescr/TrkSurfaces/TrkSurfaces/RotatedTrapezoidBounds.icc

Adding @amorley as watcher

added Tracking master review-pending-level-1 labels

CI Result SUCCESS (hash 71646de6)

	Athena	AthSimulation	AthGeneration	AnalysisBase
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
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 24139]

MR looks good, but many files have been changed. Therefore marking for L2 (L1)

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

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

Looks good. Karolos (L2)

merged

mentioned in commit 04b20abc

added sweep:ignore label