MuonAlignmentData, etc: Muon correction containers hold payload by value.

Change the types in CorrContainer.h to hold their payloads by value rather than by pointer. Should head off memory management problems due to confusion as to what was owning what.

Also some additional const, etc. cleanups as needed.

q220, q221, q222, q223, q431 run successfully, and RunTier0Tests passes.

This merge request affects 6 packages:

• MuonSpectrometer/Amdcsimrec/AmdcAth
• MuonSpectrometer/MuonAlignment/MuonAlignmentData
• MuonSpectrometer/MuonConditions/MuonCondGeneral/MuonCondAlg
• MuonSpectrometer/MuonConditions/MuonCondGeneral/MuonCondTest
• MuonSpectrometer/MuonDetDescr/MuonReadoutGeometry
• MuonSpectrometer/MuonGeoModel

Adding @rosati ,@wleight ,@nkoehler as watchers

added MuonSpectrometer master review-pending-level-1 labels

CI Result SUCCESS (hash 03e9057c)

	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 5
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 11899]

I think this looks OK, the warnings are unrelated.

MLB (L1)

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

merged

mentioned in commit 891176aa

added sweep:ignore label