main-cleanup-GeoModelSvc

Reduce verbosity reported in ATLASRECTS-4877. Also tidied headers a bit and moved some repetitive logic to a lambda.

This merge request affects 1 package:

• DetectorDescription/GeoModel/GeoModelSvc

This merge request affects 4 files:

• DetectorDescription/GeoModel/GeoModelSvc/src/GeoModelSvc.cxx
• DetectorDescription/GeoModel/GeoModelSvc/src/GeoModelSvc.h
• DetectorDescription/GeoModel/GeoModelSvc/src/RDBMaterialManager.cxx
• DetectorDescription/GeoModel/GeoModelSvc/src/RDBMaterialManager.h

Adding @apsallid ,@rbianchi ,@jojungge as watchers

added Geometry main review-pending-level-1 labels

CI Result SUCCESS (hash d47bcf4e)

	Athena	AthSimulation
externals
cmake
make
tests

Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 10266] (remote access info)

MR looks fine from L1 perspective, CI shows no problems - approving

Cheers, Qiyu, L1

removed review-pending-level-1 label

added review-approved label

merged

mentioned in commit c5a74833

mentioned in commit fwinkl/athena@7aee53e6

mentioned in merge request !71770 (merged)

This broke a good junk of unit tests. Fixed in !71770 (merged). For the future please use full-unit-tests for printout changes in services.