DerivationFrameworkInDet: Pass large arguments by const reference instead of by value.

Large objects should be passed by const reference instead of by value.

PhysicsAnalysis/DerivationFramework/DerivationFrameworkInDet/DerivationFrameworkInDet/EGammaTracksThinning.h

@@ -45,11 +45,11 @@ namespace DerivationFramework {
     double m_minEtEg;
 
     std::set<int> findGoodTracks(const TrackCollection* trackCont,
-                              TLorentzVector candHepLorentz, 
+                                 const TLorentzVector& candHepLorentz, 
                               double maxDeltaR) const ;
     
 
-    StatusCode thinTracks( const TrackCollection * trackCont , std::set<int> goodTracks ) const ;
+    StatusCode thinTracks( const TrackCollection * trackCont , const std::set<int>& goodTracks ) const ;
         
   };  
 }

PhysicsAnalysis/DerivationFramework/DerivationFrameworkInDet/src/EGammaTracksThinning.cxx

@@ -150,7 +150,7 @@ StatusCode DerivationFramework::EGammaTracksThinning::doThinning() const {
 // =======================================================
 
 std::set<int> DerivationFramework::EGammaTracksThinning::findGoodTracks(const TrackCollection* trackCont,
-                                                               TLorentzVector candHepLorentz, 
+                                                               const TLorentzVector& candHepLorentz, 
                                                                double maxDeltaR ) const 
 {
 
@@ -197,7 +197,7 @@ std::set<int> DerivationFramework::EGammaTracksThinning::findGoodTracks(const Tr
 // =======================================================
 
 StatusCode DerivationFramework::EGammaTracksThinning::thinTracks( const TrackCollection * trackCollection , 
-                                                            std::set<int> m_goodTrackIDs ) const
+                                                                  const std::set<int>& m_goodTrackIDs ) const
 {
   
   ATH_MSG_DEBUG( "==> thinTracks " << name() << "..." );