Commit 7db3871f authored by scott snyder's avatar scott snyder Committed by scott snyder
Browse files

L1TopoEvent: Fix cppcheck warnings.

Pass std::string by const reference, not by value.
parent 79ec0f26
......@@ -19,13 +19,13 @@ namespace TCS {
public:
// default constructor
jLargeRJetTOB(uint32_t roiWord = 0, std::string tobName = "jLargeRJetTOB");
jLargeRJetTOB(uint32_t roiWord = 0, const std::string& tobName = "jLargeRJetTOB");
// copy constructor
jLargeRJetTOB(const jLargeRJetTOB & jet);
// constructor with initial values
jLargeRJetTOB(unsigned int Et, int eta, unsigned phi, uint32_t roiWord = 0, std::string tobName = "jLargeRJetTOB" );
jLargeRJetTOB(unsigned int Et, int eta, unsigned phi, uint32_t roiWord = 0, const std::string& tobName = "jLargeRJetTOB" );
// destructor
virtual ~jLargeRJetTOB();
......
......@@ -19,13 +19,13 @@ namespace TCS {
public:
// default constructor
jTauTOB(uint32_t roiWord = 0, std::string tobName = "jTauTOB");
jTauTOB(uint32_t roiWord = 0, const std::string& tobName = "jTauTOB");
// copy constructor
jTauTOB(const jTauTOB & jet);
// constructor with initial values
jTauTOB(unsigned int Et, int eta, unsigned phi, uint32_t roiWord = 0, std::string tobName = "jTauTOB" );
jTauTOB(unsigned int Et, int eta, unsigned phi, uint32_t roiWord = 0, const std::string& tobName = "jTauTOB" );
// destructor
virtual ~jTauTOB();
......
......@@ -6,12 +6,12 @@ thread_local TCS::Heap<TCS::jLargeRJetTOB> TCS::jLargeRJetTOB::fg_heap("Jet");
// constructors
// default constructor
TCS::jLargeRJetTOB::jLargeRJetTOB(uint32_t roiWord, std::string tobName) :
TCS::jLargeRJetTOB::jLargeRJetTOB(uint32_t roiWord, const std::string& tobName) :
BaseTOB( roiWord,tobName )
{}
// constructor with initial values
TCS::jLargeRJetTOB::jLargeRJetTOB(unsigned int Et, int eta, unsigned phi, uint32_t roiWord, std::string tobName) :
TCS::jLargeRJetTOB::jLargeRJetTOB(unsigned int Et, int eta, unsigned phi, uint32_t roiWord, const std::string& tobName) :
BaseTOB( roiWord,tobName )
, m_Et(Et)
, m_eta(eta)
......
......@@ -6,12 +6,12 @@ thread_local TCS::Heap<TCS::jTauTOB> TCS::jTauTOB::fg_heap("Jet");
// constructors
// default constructor
TCS::jTauTOB::jTauTOB(uint32_t roiWord, std::string tobName) :
TCS::jTauTOB::jTauTOB(uint32_t roiWord, const std::string& tobName) :
BaseTOB( roiWord,tobName )
{}
// constructor with initial values
TCS::jTauTOB::jTauTOB(unsigned int Et, int eta, unsigned phi, uint32_t roiWord, std::string tobName) :
TCS::jTauTOB::jTauTOB(unsigned int Et, int eta, unsigned phi, uint32_t roiWord, const std::string& tobName) :
BaseTOB( roiWord,tobName )
, m_Et(Et)
, m_eta(eta)
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment