Skip to content

Use CxxUtils::CachedUniquePtr<const Amg::Vector3D> for global position in Muon PRDs and RIOs

Susumu Oda requested to merge oda/athena:master-globalPos into master

Use CxxUtils::CachedUniquePtr for global position in Muon PRDs and RIOs.

http://goetz.web.cern.ch/goetz/mutable_2019-07-13.html

nightly/master/2019-07-13T2149:MuonSpectrometer/MuonReconstruction/MuonRecEvent/MuonTrigCoinData/MuonTrigCoinData/TgcCoinData.h +261

   259	      bool m_isPositiveDeltaR;
   260	
   261	      mutable const Amg::Vector3D* m_globalposIn;
   262	      mutable const Amg::Vector3D* m_globalposOut;
   263	  };
nightly/master/2019-07-13T2149:MuonSpectrometer/MuonReconstruction/MuonRecEvent/MuonTrigCoinData/MuonTrigCoinData/TgcCoinData.h +262

   260	
   261	      mutable const Amg::Vector3D* m_globalposIn;
   262	      mutable const Amg::Vector3D* m_globalposOut;
   263	  };
   264	  ////////////////////////////////////////////////

nightly/master/2019-07-13T2149:MuonSpectrometer/MuonReconstruction/MuonRecEvent/MuonRIO_OnTrack/MuonRIO_OnTrack/MdtDriftCircleOnTrack.h +251

   249	
   250	    /** global position of the measurement. */
   251	    mutable const Amg::Vector3D*          m_globalPosition;
   252	    
   253	    /** Surface at postion of sagged wire. This is owned by this object, and will be deleted when it 

nightly/master/2019-07-13T2149:MuonSpectrometer/MuonReconstruction/MuonRecEvent/MuonPrepRawData/MuonPrepRawData/MdtPrepData.h +137

   135	    /**@brief Global position of measurement.
   136	    Calculated on demand and cached (not deleted in destructor, not written to disk)*/
   137	    mutable const Amg::Vector3D*          m_globalPosition;
   138	
   139	};

nightly/master/2019-07-13T2149:MuonSpectrometer/MuonReconstruction/MuonRecEvent/MuonPrepRawData/MuonPrepRawData/CscStripPrepData.h +115

   113	      IdentifierHash m_collectionHash;
   114	      /** The global position is calculated 'on the fly' and is not written to disk*/
   115	      mutable const Amg::Vector3D* m_globalPosition;
   116	      const MuonGM::CscReadoutElement* m_detEl;
   117	      std::vector<float> m_sampleCharges;

nightly/master/2019-07-13T2149:MuonSpectrometer/MuonReconstruction/MuonRecEvent/MuonCompetingRIOsOnTrack/MuonCompetingRIOsOnTrack/CompetingMuonClustersOnTrack.h +134

   132	
   133	    /** The global Position */
   134	    mutable const Amg::Vector3D*        m_globalPosition;
   135	
   136	    /** The vector of contained Muon::MuonClusterOnTrack objects */

nightly/master/2019-07-13T2149:MuonSpectrometer/MuonReconstruction/MuonRecEvent/MuonPrepRawData/MuonPrepRawData/MuonCluster.h +87

    85	    /**@brief Global position of measurement.
    86	    Calculated on demand and cached */
    87	    mutable const Amg::Vector3D*          m_globalPosition;
    88	    
    89	  };
nightly/master/2019-07-13T2149:MuonSpectrometer/MuonReconstruction/MuonRecEvent/MuonRIO_OnTrack/MuonRIO_OnTrack/MuonClusterOnTrack.h +91

    89	  protected:
    90	    /** cache global position, the global position has to be calculated in the inheriting classes */
    91	    mutable const Amg::Vector3D*              m_globalPosition;
    92	    
    93	    /** The position along the strip - used to calculate the GlobalPosition*/

RunTier0Tests.py was fine. RunTier0Tests.log

Edited by Susumu Oda

Merge request reports