Skip to content
Snippets Groups Projects
Commit b9df69ca authored by Scott Snyder's avatar Scott Snyder Committed by Adam Edward Barton
Browse files

TauAnalysisTools: Fix cppcheck 2.14 warnings.

TauAnalysisTools: Fix cppcheck 2.14 warnings.

Prefer returning const references to members rather than copies.
Pass strings by const reference, not by value.
parent 3e5a023b
No related branches found
No related tags found
29 merge requests!78241Draft: FPGATrackSim: GenScan code refactor,!78236Draft: Switching Streams https://its.cern.ch/jira/browse/ATR-27417,!78056AFP monitoring: new synchronization and cleaning,!78041AFP monitoring: new synchronization and cleaning,!77990Updating TRT chip masks for L1TRT trigger simulation - ATR-28372,!77733Draft: add new HLT NN JVT, augmented with additional tracking information,!77731Draft: Updates to ZDC reconstruction,!77728Draft: updates to ZDC reconstruction,!77522Draft: sTGC Pad Trigger Emulator,!76725ZdcNtuple: Fix cppcheck warning.,!76611L1CaloFEXByteStream: Fix out-of-bounds array accesses.,!76475Punchthrough AF3 implementation in FastG4,!76474Punchthrough AF3 implementation in FastG4,!76343Draft: MooTrackBuilder: Recalibrate NSW hits in refine method,!75729New implementation of ZDC nonlinear FADC correction.,!75703Draft: Update to HI han config for HLT jets,!75184Draft: Update file heavyions_run.config,!74430Draft: Fixing upper bound for Delayed Jet Triggers,!73963Changing the path of the histograms to "Expert" area,!73875updating ID ART reference plots,!73874AtlasCLHEP_RandomGenerators: Fix cppcheck warnings.,!73449Add muon detectors to DarkJetPEBTLA partial event building,!73343Draft: [TrigEgamma] Add photon ringer chains on bootstrap mechanism,!72336Fixed TRT calibration crash,!72176Draft: Improving L1TopoOnline chain that now gets no-empty plots. Activating it by default,!72012Draft: Separate JiveXMLConfig.py into Config files,!71876Fix MET trigger name in MissingETMonitoring,!71820Draft: Adding new TLA End-Of-Fill (EOF) chains and removing obsolete DIPZ chains,!71585TauAnalysisTools: Fix cppcheck 2.14 warnings.
/*
Copyright (C) 2002-2022 CERN for the benefit of the ATLAS collaboration
Copyright (C) 2002-2024 CERN for the benefit of the ATLAS collaboration
*/
#include <fstream>
......@@ -276,7 +276,7 @@ int TauAnalysisTools::getNTauDecayParticles(const xAOD::TruthParticle& xTruthTau
}
//______________________________________________________________________________
bool TauAnalysisTools::testFileForEOFContainsCharacters(std::string sFileName)
bool TauAnalysisTools::testFileForEOFContainsCharacters(const std::string& sFileName)
{
// returns true if last line in file is empty or the line starts with the
// number sign #
......
/*
Copyright (C) 2002-2023 CERN for the benefit of the ATLAS collaboration
Copyright (C) 2002-2024 CERN for the benefit of the ATLAS collaboration
*/
#ifndef TAUANALYSISTOOLS_DITAUSELECTIONCUTS_H
......@@ -40,7 +40,7 @@ public:
asg::AcceptData& accept) = 0;
TH1F* CreateControlPlot(const char* sName, const char* sTitle, int iBins, double dXLow, double dXUp);
std::string getName()
const std::string& getName()
{
return m_sName;
};
......
......@@ -5,7 +5,7 @@
* @brief
* @date 2021-02-19
*
* @copyright Copyright (C) 2002-2022 CERN for the benefit of the ATLAS collaboration
* @copyright Copyright (C) 2002-2024 CERN for the benefit of the ATLAS collaboration
*
*/
......@@ -87,7 +87,7 @@ xAOD::TauJetParameters::DecayMode getTruthDecayMode(const xAOD::TauJet& xTau);
*/
int getNTauDecayParticles(const xAOD::TruthParticle& xTruthTau, int iPdgId, bool bCompareAbsoluteValues);
/** returns true if last line in file is empty or the line starts with the number sign */
bool testFileForEOFContainsCharacters(std::string sFileName);
bool testFileForEOFContainsCharacters(const std::string& sFileName);
void createPi0Vectors(const xAOD::TauJet* xTau, std::vector<TLorentzVector>& vPi0s);
void correctedPi0Vectors(const xAOD::TauJet* xTau, std::vector<TLorentzVector>& correctedPi0s, TLorentzVector& TauP4);
/** return charged and neutral daughters of truth tau */
......
/*
Copyright (C) 2002-2023 CERN for the benefit of the ATLAS collaboration
Copyright (C) 2002-2024 CERN for the benefit of the ATLAS collaboration
*/
#ifndef TAUANALYSISTOOLS_TAUSELECTIONCUTS_H
......@@ -39,7 +39,7 @@ public:
asg::AcceptData& accept) = 0;
TH1F* CreateControlPlot(const char* sName, const char* sTitle, int iBins, double dXLow, double dXUp);
std::string getName()
const std::string& getName()
{
return m_sName;
};
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment