From 31aa784d5c50d27e609a4b70ba690911a1567cec Mon Sep 17 00:00:00 2001
From: scott snyder <snyder@bnl.gov>
Date: Wed, 13 Sep 2017 04:37:58 +0200
Subject: [PATCH] DerivationFrameworkInDet: Pass large arguments by const
reference instead of by value.
Large objects should be passed by const reference instead of by value.
---
.../DerivationFrameworkInDet/EGammaTracksThinning.h | 4 ++--
.../DerivationFrameworkInDet/src/EGammaTracksThinning.cxx | 4 ++--
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/PhysicsAnalysis/DerivationFramework/DerivationFrameworkInDet/DerivationFrameworkInDet/EGammaTracksThinning.h b/PhysicsAnalysis/DerivationFramework/DerivationFrameworkInDet/DerivationFrameworkInDet/EGammaTracksThinning.h
index 56f541c3cd2..3a82873bc0d 100644
--- a/PhysicsAnalysis/DerivationFramework/DerivationFrameworkInDet/DerivationFrameworkInDet/EGammaTracksThinning.h
+++ b/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 ;
};
}
diff --git a/PhysicsAnalysis/DerivationFramework/DerivationFrameworkInDet/src/EGammaTracksThinning.cxx b/PhysicsAnalysis/DerivationFramework/DerivationFrameworkInDet/src/EGammaTracksThinning.cxx
index f99fa6529d8..028d79d029a 100644
--- a/PhysicsAnalysis/DerivationFramework/DerivationFrameworkInDet/src/EGammaTracksThinning.cxx
+++ b/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() << "..." );
--
GitLab