Make TimedExtrapolator const correct.

• dropped unused mutable members
• moved mutable members into cache object which is passed through.

This should allow to call the TimedExtrapolator from multiple threads in parallel.

"Addresses" ATLASSIM-4429.

This fixes the segmentation fault in the TimedExtrapolator discussed in ATLASSIM-4429. But the job still crashes now in G4TransportTool::genEvent (full stack traces: crash_g4.log )

WARNING: big files (>100K) are found in the changeset

128K in file Tracking/TrkExtrapolation/TrkExTools/src/TimedExtrapolator.cxx

This merge request affects 1 package:

• Tracking/TrkExtrapolation/TrkExTools

Adding @amorley as watcher

added Tracking master review-pending-level-1 labels

I did not perform many tests I only checked that together with MR !31282 (merged) the verbose output of the TimedExtrapolator and FastCaloSimV2Tool are identical when running the command from ATLASSIM-4429 (limited to 9 events) with and without this patch in serial athena. When running in MT with multiple threads the job now crashes in G4TransportTool::genEvent (see preceding comment).

CI Result SUCCESS (hash edd52224)

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

Looks good, although given the amount and complexity of changes I will scale it to L2

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

Looks good, approving.

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

merged

mentioned in commit 93232011

added sweep:ignore label

mentioned in merge request !31644 (merged)