diff --git a/InnerDetector/InDetDetDescr/InDetGeoModelUtils/InDetGeoModelUtils/IInDetServMatBuilderTool.h b/InnerDetector/InDetDetDescr/InDetGeoModelUtils/InDetGeoModelUtils/IInDetServMatBuilderTool.h index 5cd2eeb8e3d64c66744d99f1ef4954d0586626da..d3ac8a4d069d5612b64f503b7faca2b8b0fa5aa1 100644 --- a/InnerDetector/InDetDetDescr/InDetGeoModelUtils/InDetGeoModelUtils/IInDetServMatBuilderTool.h +++ b/InnerDetector/InDetDetDescr/InDetGeoModelUtils/InDetGeoModelUtils/IInDetServMatBuilderTool.h @@ -1,5 +1,5 @@ /* - Copyright (C) 2002-2018 CERN for the benefit of the ATLAS collaboration + Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration */ /////////////////////////////////////////////////////////////////// @@ -24,7 +24,7 @@ class IInDetServMatBuilderTool : virtual public IAlgTool { public: static const InterfaceID& interfaceID( ) ; - virtual const std::vector<const InDetDD::ServiceVolume *> & getServices() = 0; + virtual const std::vector<const InDetDD::ServiceVolume *> & getServices() const = 0; }; inline const InterfaceID& IInDetServMatBuilderTool::interfaceID() diff --git a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/CMakeLists.txt b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/CMakeLists.txt index 2f42a3c03c2b2b44473be9411013013e38ba89c0..f8de9f5d402145029c0b46012d6371d2001c638e 100644 --- a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/CMakeLists.txt +++ b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/CMakeLists.txt @@ -9,6 +9,7 @@ atlas_subdir( InDetServMatGeoModel ) atlas_depends_on_subdirs( PUBLIC Control/AthenaBaseComps Control/AthenaKernel + Control/CxxUtils Database/RDBAccessSvc DetectorDescription/GeoPrimitives DetectorDescription/GeoModel/GeoModelUtilities @@ -31,7 +32,7 @@ atlas_add_component( InDetServMatGeoModel src/*.cxx src/components/*.cxx INCLUDE_DIRS ${CORAL_INCLUDE_DIRS} - LINK_LIBRARIES ${CORAL_LIBRARIES} ${GEOMODELCORE_LIBRARIES} AthenaBaseComps AthenaKernel GeoModelUtilities GaudiKernel InDetGeoModelUtils SGTools StoreGateLib SGtests GeometryDBSvcLib ) + LINK_LIBRARIES ${CORAL_LIBRARIES} ${GEOMODELCORE_LIBRARIES} AthenaBaseComps AthenaKernel CxxUtils GeoModelUtilities GaudiKernel InDetGeoModelUtils SGTools StoreGateLib SGtests GeometryDBSvcLib ) # Install files from the package: atlas_install_headers( InDetServMatGeoModel ) diff --git a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/ComputeStaveServices.h b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/ComputeStaveServices.h index 4868d8df5ddd12d8d7760f23ccdd6438110a0878..7ef824a5c21b997d18526ffed52fd7ab002ad92c 100644 --- a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/ComputeStaveServices.h +++ b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/ComputeStaveServices.h @@ -1,5 +1,5 @@ /* - Copyright (C) 2002-2017 CERN for the benefit of the ATLAS collaboration + Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration */ #ifndef ComputeStaveServices_H @@ -7,6 +7,7 @@ #include "InDetServMatGeoModel/StaveServices.h" #include "AthenaKernel/MsgStreamMember.h" +#include "CxxUtils/checker_macros.h" class ComputeStaveServices { public: @@ -21,7 +22,7 @@ public: private: // the message stream (same for all derived classes) MsgStream& msg (MSG::Level lvl) const { return m_msg << lvl; } - mutable Athena::MsgStreamMember m_msg; + mutable Athena::MsgStreamMember m_msg ATLAS_THREAD_SAFE; }; diff --git a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/EndPlateFactoryFS.h b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/EndPlateFactoryFS.h index 5dcd212f8a150b0a3c9c33bf7fb08d5816fda675..62323975ae37ee873ee55a4f8f84f1cea35185f3 100755 --- a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/EndPlateFactoryFS.h +++ b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/EndPlateFactoryFS.h @@ -1,11 +1,12 @@ /* - Copyright (C) 2002-2017 CERN for the benefit of the ATLAS collaboration + Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration */ #ifndef INDETSERVMATGEOMODEL_ENDPLATEFACTORYFS_H #define INDETSERVMATGEOMODEL_ENDPLATEFACTORYFS_H #include "AthenaKernel/MsgStreamMember.h" +#include "CxxUtils/checker_macros.h" #include "GaudiKernel/ServiceHandle.h" class StoreGateSvc; @@ -39,7 +40,7 @@ class EndPlateFactoryFS { // private data StoreGateSvc *m_detStore; ServiceHandle<IRDBAccessSvc> m_rdbAccess; - mutable Athena::MsgStreamMember m_msg; + mutable Athena::MsgStreamMember m_msg ATLAS_THREAD_SAFE; }; #endif diff --git a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/HRoute.h b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/HRoute.h index 3ac1effba67781a18d68ed6dda11459347a307ae..75d2646365f0ea1793760e095109abe7a777e8df 100644 --- a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/HRoute.h +++ b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/HRoute.h @@ -1,5 +1,5 @@ /* - Copyright (C) 2002-2017 CERN for the benefit of the ATLAS collaboration + Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration */ #ifndef HRoute_H @@ -29,7 +29,7 @@ public: virtual double exit() const {return zExit();} - virtual Route* nextRoute() const {return m_next;} + virtual Route* nextRoute() {return m_next;} const VolumeContainer& volumes() const {return m_volumes;} diff --git a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/InDetServMatAthenaComps.h b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/InDetServMatAthenaComps.h index 28076e6b370cb0e931e8e590c26caa4f9d76bdd5..5b4da5326f1c3f2314c082d9b25cdfb9e5ce9729 100644 --- a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/InDetServMatAthenaComps.h +++ b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/InDetServMatAthenaComps.h @@ -1,5 +1,5 @@ /* - Copyright (C) 2002-2017 CERN for the benefit of the ATLAS collaboration + Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration */ #ifndef InDetServMatGeoModel_InDetServMatAthenaComps_H @@ -18,7 +18,7 @@ public: //Add Builder Tool void setBuilderTool(IInDetServMatBuilderTool * builderTool); - IInDetServMatBuilderTool *builderTool() const; + const IInDetServMatBuilderTool *builderTool() const; private: diff --git a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/InDetServMatBuilderToolSLHC.h b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/InDetServMatBuilderToolSLHC.h index 1e961306b3a17ae7a3fb4c9c6e577c4e37a561d8..3ee2ecd54af7d7f37d08291900e27ed1e59f0a0b 100644 --- a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/InDetServMatBuilderToolSLHC.h +++ b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/InDetServMatBuilderToolSLHC.h @@ -1,5 +1,5 @@ /* - Copyright (C) 2002-2017 CERN for the benefit of the ATLAS collaboration + Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration */ #ifndef InDetServMatBuilderToolSLHC_H @@ -62,7 +62,7 @@ public: virtual StatusCode finalize(); /// Get the services - virtual const std::vector<const InDetDD::ServiceVolume *> & getServices(); + virtual const std::vector<const InDetDD::ServiceVolume *> & getServices() const; /// General service adding method. void addService(InDetDD::ServiceVolume * param); diff --git a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/InDetServMatFactoryDC2.h b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/InDetServMatFactoryDC2.h index a75d268b25e50ada7fa747c66b87cd457bc91943..0d243aed661c6208ab04c8fb00e84c1845b431f4 100755 --- a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/InDetServMatFactoryDC2.h +++ b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/InDetServMatFactoryDC2.h @@ -1,5 +1,5 @@ /* - Copyright (C) 2002-2017 CERN for the benefit of the ATLAS collaboration + Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration */ #ifndef INDETSERVMATGEOMODEL_INDETSERVMATFACTORYDC2_H @@ -7,12 +7,13 @@ #include "AthenaKernel/MsgStreamMember.h" -#include "GaudiKernel/ServiceHandle.h" +#include "CxxUtils/checker_macros.h" #include "GeoModelKernel/GeoVDetectorFactory.h" //the following needed because the return type of getDetectorManager() is not //the same as the method return type as specified in the baseclass #include "InDetServMatGeoModel/InDetServMatManager.h" +#include "GaudiKernel/ServiceHandle.h" class StoreGateSvc; class IRDBAccessSvc; @@ -45,7 +46,7 @@ class InDetServMatFactoryDC2 : public GeoVDetectorFactory { StoreGateSvc *m_detStore; ServiceHandle<IRDBAccessSvc> m_rdbAccess; InDetDD::InDetServMatManager *m_manager; - mutable Athena::MsgStreamMember m_msg; + mutable Athena::MsgStreamMember m_msg ATLAS_THREAD_SAFE; }; #endif // INDETSERVMATGEOMODEL_INDETSERVMATFACTORYDC2_H diff --git a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/InDetServMatFactoryFS.h b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/InDetServMatFactoryFS.h index 73ef0ddd0469037ecba4941ed5e68fcf0f4fbd35..a2342a598d723a2fe497ab2aab0ae79d55299c6c 100755 --- a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/InDetServMatFactoryFS.h +++ b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/InDetServMatFactoryFS.h @@ -1,5 +1,5 @@ /* - Copyright (C) 2002-2017 CERN for the benefit of the ATLAS collaboration + Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration */ #ifndef INDETSERVMATGEOMODEL_INDETSERVMATFACTORYFS_H @@ -7,10 +7,12 @@ #include "AthenaKernel/MsgStreamMember.h" -#include "GaudiKernel/ServiceHandle.h" +#include "CxxUtils/checker_macros.h" #include "GeoModelKernel/GeoVDetectorFactory.h" #include "InDetServMatGeoModel/InDetServMatManager.h" +#include "GaudiKernel/ServiceHandle.h" + class StoreGateSvc; class IRDBAccessSvc; @@ -43,9 +45,7 @@ class InDetServMatFactoryFS : public GeoVDetectorFactory { StoreGateSvc *m_detStore; ServiceHandle<IRDBAccessSvc> m_rdbAccess; InDetDD::InDetServMatManager *m_manager; - mutable Athena::MsgStreamMember m_msg; + mutable Athena::MsgStreamMember m_msg ATLAS_THREAD_SAFE; }; #endif - - diff --git a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/InDetServMatGeometryManager.h b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/InDetServMatGeometryManager.h index 0492c528c4602f16d79b8a1b27815b910ccc9109..100f00b8847a7380e37a3ef6f5f45ae6847f4e6a 100644 --- a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/InDetServMatGeometryManager.h +++ b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/InDetServMatGeometryManager.h @@ -28,7 +28,7 @@ public: const IGeometryDBSvc * db() const {return m_athenaComps->geomDB();} // Access to material manager - InDetMaterialManager * matMgr() const {return m_matMgr;} + InDetMaterialManager * matMgr() {return m_matMgr;} // Access to message stream diff --git a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/PixelServMatFactoryDC2.h b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/PixelServMatFactoryDC2.h index e7128af38533b53dd3b910211da9fb503fddc02d..03de32713da33d3feaf9b593b0f2feb190a4d3e1 100755 --- a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/PixelServMatFactoryDC2.h +++ b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/PixelServMatFactoryDC2.h @@ -1,11 +1,12 @@ /* - Copyright (C) 2002-2017 CERN for the benefit of the ATLAS collaboration + Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration */ #ifndef INDETSERVMATGEOMODEL_PIXELSERVMATFACTORYDC2_H #define INDETSERVMATGEOMODEL_PIXELSERVMATFACTORYDC2_H #include "AthenaKernel/MsgStreamMember.h" +#include "CxxUtils/checker_macros.h" #include "GaudiKernel/ServiceHandle.h" class StoreGateSvc; @@ -37,7 +38,7 @@ class PixelServMatFactoryDC2 { // private data StoreGateSvc *m_detStore; ServiceHandle<IRDBAccessSvc> m_rdbAccess; - mutable Athena::MsgStreamMember m_msg; + mutable Athena::MsgStreamMember m_msg ATLAS_THREAD_SAFE; }; #endif // INDETSERVMATGEOMODEL_PIXELSERVMATFACTORYDC2_H diff --git a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/PixelServMatFactoryFS.h b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/PixelServMatFactoryFS.h index 79f91d19cdcaba1a0fa03f3dba5bb8636b6bd877..dc8abbbc8c4560909e408064eab645bfcd83cb8c 100755 --- a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/PixelServMatFactoryFS.h +++ b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/PixelServMatFactoryFS.h @@ -6,6 +6,7 @@ #define INDETSERVMATGEOMODEL_PIXELSERVMATFACTORYFS_H #include "AthenaKernel/MsgStreamMember.h" +#include "CxxUtils/checker_macros.h" #include "GaudiKernel/ServiceHandle.h" class StoreGateSvc; @@ -40,7 +41,7 @@ class PixelServMatFactoryFS { StoreGateSvc *m_detStore; ServiceHandle<IRDBAccessSvc> m_rdbAccess; std::unique_ptr<InDetMaterialManager> m_materialManager; - mutable Athena::MsgStreamMember m_msg; + mutable Athena::MsgStreamMember m_msg ATLAS_THREAD_SAFE; }; diff --git a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/Route.h b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/Route.h index 4477275fbe9041cd13e3f29b7a39e57eaf3b80e4..cfc92db7cadad94d1b2b4bcc4a1aa64cf288dffc 100644 --- a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/Route.h +++ b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/Route.h @@ -1,5 +1,5 @@ /* - Copyright (C) 2002-2017 CERN for the benefit of the ATLAS collaboration + Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration */ #ifndef Route_H @@ -33,7 +33,7 @@ public: //virtual int direction() = 0; - virtual Route* nextRoute() const = 0; + virtual Route* nextRoute() = 0; virtual const VolumeContainer& volumes() const = 0; diff --git a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/SCT_ServMatFactoryDC2.h b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/SCT_ServMatFactoryDC2.h index beb1a0949c8150d764b5fde7104bba6a206e5349..abbdab708e9c6ca2e285fcc3c2bafbce19537875 100755 --- a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/SCT_ServMatFactoryDC2.h +++ b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/SCT_ServMatFactoryDC2.h @@ -1,11 +1,12 @@ /* - Copyright (C) 2002-2019 CERN for the benefit of the ATLAS collaboration + Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration */ #ifndef INDETSERVMATGEOMODEL_SCT_SERVMATFACTORYDC2_H #define INDETSERVMATGEOMODEL_SCT_SERVMATFACTORYDC2_H #include "AthenaKernel/MsgStreamMember.h" +#include "CxxUtils/checker_macros.h" #include "GaudiKernel/ServiceHandle.h" class StoreGateSvc; @@ -57,7 +58,7 @@ class SCT_ServMatFactoryDC2 { StoreGateSvc *m_detStore; ServiceHandle<IRDBAccessSvc> m_rdbAccess; const StoredMaterialManager *m_materialManager; - mutable Athena::MsgStreamMember m_msg; + mutable Athena::MsgStreamMember m_msg ATLAS_THREAD_SAFE; }; #endif // INDETSERVMATGEOMODEL_SCT_SERVMATFACTORYDC2_H diff --git a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/SCT_ServMatFactoryFS.h b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/SCT_ServMatFactoryFS.h index 3a4c3a0a493d2e9f0132650afaca78f53f4c070a..f5f519f807769be7ad64bfe26c65a36e57c0089b 100755 --- a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/SCT_ServMatFactoryFS.h +++ b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/SCT_ServMatFactoryFS.h @@ -6,6 +6,7 @@ #define INDETSERVMATGEOMODEL_SCT_SERVMATFACTORYFS_H #include "AthenaKernel/MsgStreamMember.h" +#include "CxxUtils/checker_macros.h" #include "GaudiKernel/ServiceHandle.h" class StoreGateSvc; @@ -42,7 +43,7 @@ class SCT_ServMatFactoryFS { StoreGateSvc *m_detStore; ServiceHandle<IRDBAccessSvc> m_rdbAccess; std::unique_ptr<InDetMaterialManager> m_materialManager; - mutable Athena::MsgStreamMember m_msg; + mutable Athena::MsgStreamMember m_msg ATLAS_THREAD_SAFE; }; diff --git a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/ServiceVolume.h b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/ServiceVolume.h index 68b66819a6e3ce4c0c0962e0d22be0627efaad00..f5f5d14461d8706f2d2b96d5b7c1337cdab26e13 100644 --- a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/ServiceVolume.h +++ b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/ServiceVolume.h @@ -1,5 +1,5 @@ /* - Copyright (C) 2002-2017 CERN for the benefit of the ATLAS collaboration + Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration */ #ifndef ServiceVolume_H @@ -77,7 +77,7 @@ public: LayerContainer layers() const {return m_layers;} - ServiceVolume* next() const {return m_next;} + ServiceVolume* next() {return m_next;} void addPrevious( ServiceVolume* prev) { m_previous.push_back(prev);} diff --git a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/SquirrelCageFactoryFS.h b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/SquirrelCageFactoryFS.h index ddaa209b7ed47f9e94c5f32c32a6d477fac55aa0..991d968647ffe2fe33392f2746f2d84a31b12916 100755 --- a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/SquirrelCageFactoryFS.h +++ b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/SquirrelCageFactoryFS.h @@ -1,11 +1,12 @@ /* - Copyright (C) 2002-2017 CERN for the benefit of the ATLAS collaboration + Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration */ #ifndef INDETSERVMATGEOMODEL_SQUIRRELCAGEFACTORYFS_H #define INDETSERVMATGEOMODEL_SQUIRRELCAGEFACTORYFS_H #include "AthenaKernel/MsgStreamMember.h" +#include "CxxUtils/checker_macros.h" #include "GaudiKernel/ServiceHandle.h" class StoreGateSvc; @@ -37,7 +38,7 @@ class SquirrelCageFactoryFS { // private data StoreGateSvc *m_detStore; ServiceHandle<IRDBAccessSvc> m_rdbAccess; - mutable Athena::MsgStreamMember m_msg; + mutable Athena::MsgStreamMember m_msg ATLAS_THREAD_SAFE; }; #endif diff --git a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/SupportRailFactoryFS.h b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/SupportRailFactoryFS.h index 74278e2230e8c74fb413be6a7f63125fb2b8be73..bc8f7ffb0abad50941fac21799f31945d0ee4764 100755 --- a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/SupportRailFactoryFS.h +++ b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/SupportRailFactoryFS.h @@ -1,11 +1,12 @@ /* - Copyright (C) 2002-2017 CERN for the benefit of the ATLAS collaboration + Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration */ #ifndef INDETSERVMATGEOMODEL_SUPPORTRAILFACTORYFS_H #define INDETSERVMATGEOMODEL_SUPPORTRAILFACTORYFS_H #include "AthenaKernel/MsgStreamMember.h" +#include "CxxUtils/checker_macros.h" #include "GaudiKernel/ServiceHandle.h" class StoreGateSvc; @@ -37,7 +38,7 @@ class SupportRailFactoryFS { // private data StoreGateSvc *m_detStore; ServiceHandle<IRDBAccessSvc> m_rdbAccess; - mutable Athena::MsgStreamMember m_msg; + mutable Athena::MsgStreamMember m_msg ATLAS_THREAD_SAFE; }; diff --git a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/TRT_ServMatFactoryDC2.h b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/TRT_ServMatFactoryDC2.h index 4fddb6f6baec5def8807afa7966d4a5dda94112a..3daf7df92fa91e7c75649a4aa2c8ede56296d838 100755 --- a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/TRT_ServMatFactoryDC2.h +++ b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/TRT_ServMatFactoryDC2.h @@ -1,10 +1,11 @@ /* - Copyright (C) 2002-2017 CERN for the benefit of the ATLAS collaboration + Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration */ #ifndef INDETSERVMATGEOMODEL_TRT_SERVMATFACTORYDC2_H #define INDETSERVMATGEOMODEL_TRT_SERVMATFACTORYDC2_H #include "AthenaKernel/MsgStreamMember.h" +#include "CxxUtils/checker_macros.h" #include "GaudiKernel/ServiceHandle.h" class StoreGateSvc; @@ -56,7 +57,7 @@ class TRT_ServMatFactoryDC2 { StoreGateSvc *m_detStore; ServiceHandle<IRDBAccessSvc> m_rdbAccess; const StoredMaterialManager *m_materialManager; - mutable Athena::MsgStreamMember m_msg; + mutable Athena::MsgStreamMember m_msg ATLAS_THREAD_SAFE; }; #endif // INDETSERVMATGEOMODEL_TRT_SERVMATFACTORYDC2_H diff --git a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/TRT_ServMatFactoryFS.h b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/TRT_ServMatFactoryFS.h index 34ba811ead0a883d7bdee7d8f96b85d1af34f92c..4f7e28e66fe49abe9dcfcb8e981dc771642703a1 100755 --- a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/TRT_ServMatFactoryFS.h +++ b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/TRT_ServMatFactoryFS.h @@ -6,6 +6,7 @@ #define INDETSERVMATGEOMODEL_TRT_SERVMATFACTORYFS_H #include "AthenaKernel/MsgStreamMember.h" +#include "CxxUtils/checker_macros.h" #include "GaudiKernel/ServiceHandle.h" class StoreGateSvc; @@ -42,7 +43,7 @@ class TRT_ServMatFactoryFS { StoreGateSvc *m_detStore; ServiceHandle<IRDBAccessSvc> m_rdbAccess; std::unique_ptr<InDetMaterialManager> m_materialManager; - mutable Athena::MsgStreamMember m_msg; + mutable Athena::MsgStreamMember m_msg ATLAS_THREAD_SAFE; }; #endif diff --git a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/VRoute.h b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/VRoute.h index 67d1cb6d18e93c03601eeb6b61791017e937f0fc..c10afdbb6cfad5fa85979e2443deee826f71f0b5 100644 --- a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/VRoute.h +++ b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/InDetServMatGeoModel/VRoute.h @@ -1,5 +1,5 @@ /* - Copyright (C) 2002-2017 CERN for the benefit of the ATLAS collaboration + Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration */ #ifndef VRoute_H @@ -29,7 +29,7 @@ public: virtual double exit() const {return rExit();} - virtual Route* nextRoute() const {return m_next;} + virtual Route* nextRoute() {return m_next;} virtual const VolumeContainer& volumes() const {return m_volumes;} diff --git a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/src/InDetServMatAthenaComps.cxx b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/src/InDetServMatAthenaComps.cxx index a1e85356a64785e00ea3a576f47f5708c96db013..e37a4958a677c53661a88968783ce0e3376b8305 100644 --- a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/src/InDetServMatAthenaComps.cxx +++ b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/src/InDetServMatAthenaComps.cxx @@ -1,5 +1,5 @@ /* - Copyright (C) 2002-2017 CERN for the benefit of the ATLAS collaboration + Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration */ #include "InDetServMatGeoModel/InDetServMatAthenaComps.h" @@ -17,8 +17,8 @@ InDetServMatAthenaComps::setBuilderTool(IInDetServMatBuilderTool * builderTool) m_builderTool = builderTool; } -IInDetServMatBuilderTool * -InDetServMatAthenaComps::builderTool() const +const IInDetServMatBuilderTool * +InDetServMatAthenaComps::builderTool() const { return m_builderTool; } diff --git a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/src/InDetServMatBuilderToolSLHC.cxx b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/src/InDetServMatBuilderToolSLHC.cxx index c7e100403a0045865804892e6983ef0f522d31a7..c30a08e87c44f08c4253f55d0dc5d1b1714ef843 100644 --- a/InnerDetector/InDetDetDescr/InDetServMatGeoModel/src/InDetServMatBuilderToolSLHC.cxx +++ b/InnerDetector/InDetDetDescr/InDetServMatGeoModel/src/InDetServMatBuilderToolSLHC.cxx @@ -86,6 +86,8 @@ StatusCode InDetServMatBuilderToolSLHC::initialize() m_athenaComps->setRDBAccessSvc(&*m_rdbAccessSvc); m_athenaComps->setGeometryDBSvc(&*m_geometryDBSvc); + build(); + msg(MSG::INFO) << "initialize() successful in " << name() << endmsg; return StatusCode::SUCCESS; } @@ -97,9 +99,8 @@ StatusCode InDetServMatBuilderToolSLHC::finalize() return sc; } -const std::vector<const InDetDD::ServiceVolume *> & InDetServMatBuilderToolSLHC::getServices() +const std::vector<const InDetDD::ServiceVolume *> & InDetServMatBuilderToolSLHC::getServices() const { - if (!m_init) build(); return m_services; }