Skip to content
Snippets Groups Projects
Commit 7949baaa authored by Scott Snyder's avatar Scott Snyder Committed by Johannes Elmsheuser
Browse files

G4UserActions: Fix cppcheck warnings.

G4UserActions: Fix cppcheck warning.

Pass container by const reference, not by value.
Delete copy/assignment for class managing memory.
parent bd30992f
No related branches found
No related tags found
30 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,!71279Draft: ATR-29330: Move L1_4J15 and the HLT chains seeded by it in the MC Menu,!71230G4UserActions: Fix cppcheck warnings.
/*
Copyright (C) 2002-2021 CERN for the benefit of the ATLAS collaboration
Copyright (C) 2002-2024 CERN for the benefit of the ATLAS collaboration
*/
#ifndef G4UserActions_LengthIntegrator_H
......@@ -45,7 +45,10 @@ namespace G4UA
public:
/// Constructor takes the name of the histogram service as argument.
LengthIntegrator(const std::string& histSvcName, bool doHistos);
LengthIntegrator(const std::string& histSvcName, bool doHistos);
LengthIntegrator(const LengthIntegrator&) = delete;
LengthIntegrator& operator=(const LengthIntegrator&) = delete;
/// Called at beginning of G4 event to cache some details about the
/// current primary vertex and particle. Also resets some measurements.
......
/*
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 "RadLengthAction.h"
......@@ -281,7 +281,7 @@ namespace G4UA
}
// how variables should be dumped in variables map
void RadLengthAction::fillVariables(std::vector<double> varvec, const std::string& name){
void RadLengthAction::fillVariables(const std::vector<double>& varvec, const std::string& name){
// first three components should be added (deltaenergy, radlength, intlength)
for(unsigned int i = 0; i<3; i++) variables[name].at(i) += varvec.at(i);
// other components should be overwritten
......
/*
Copyright (C) 2002-2022 CERN for the benefit of the ATLAS collaboration
Copyright (C) 2002-2024 CERN for the benefit of the ATLAS collaboration
*/
#ifndef G4UserActions_RadLengthAction_H
......@@ -78,7 +78,7 @@ namespace G4UA
ServiceHandle<ITHistSvc> m_hSvc;
// methode to fill vector stored in variables map (index volume name)
void fillVariables(std::vector<double> varvec, const std::string& name);
void fillVariables(const std::vector<double>& varvec, const std::string& name);
}; //class RadLengthAction
} //namespace G4UA
......
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