diff --git a/MuonSpectrometer/MuonValidation/MuonRecValidation/MuonTrackPerformance/src/MuonSegmentPerformanceAlg.cxx b/MuonSpectrometer/MuonValidation/MuonRecValidation/MuonTrackPerformance/src/MuonSegmentPerformanceAlg.cxx
index ea05793b3312430a63d3007aec7e80d17954c172..576e21d18d26751046d70c0ef3cbeb22b2f69912 100644
--- a/MuonSpectrometer/MuonValidation/MuonRecValidation/MuonTrackPerformance/src/MuonSegmentPerformanceAlg.cxx
+++ b/MuonSpectrometer/MuonValidation/MuonRecValidation/MuonTrackPerformance/src/MuonSegmentPerformanceAlg.cxx
@@ -1,5 +1,5 @@
/*
- Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration
+ Copyright (C) 2002-2021 CERN for the benefit of the ATLAS collaboration
*/
#include "MuonSegmentPerformanceAlg.h"
@@ -56,7 +56,7 @@ StatusCode MuonSegmentPerformanceAlg::execute()
++m_nevents;
bool missedSegment = false;
- for( const auto& seg : *truthSegments ){
+ for( const auto seg : *truthSegments ){
int chIndex = seg->chamberIndex();
if( chIndex < 0 || chIndex >= Muon::MuonStationIndex::ChIndexMax ){
@@ -86,7 +86,7 @@ StatusCode MuonSegmentPerformanceAlg::execute()
}
if(missedSegment) ATH_MSG_DEBUG(" Dump Fake segments " );
- for( const auto& seg : *segments ){
+ for( const auto seg : *segments ){
if( matchedSegments.count(seg) ) continue;
int chIndex = seg->chamberIndex();
if( chIndex < 0 || chIndex >= Muon::MuonStationIndex::ChIndexMax ){
diff --git a/Reconstruction/MuonIdentification/MuonCombinedTrackFindingTools/src/MuonInsideOutRecoTool.cxx b/Reconstruction/MuonIdentification/MuonCombinedTrackFindingTools/src/MuonInsideOutRecoTool.cxx
index 4de4cd9499febf0663d7fb8977c2cdc723b6f8a3..e895b015634041bf5a97e52900689be3232d7a0b 100644
--- a/Reconstruction/MuonIdentification/MuonCombinedTrackFindingTools/src/MuonInsideOutRecoTool.cxx
+++ b/Reconstruction/MuonIdentification/MuonCombinedTrackFindingTools/src/MuonInsideOutRecoTool.cxx
@@ -1,5 +1,5 @@
/*
- Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration
+ Copyright (C) 2002-2021 CERN for the benefit of the ATLAS collaboration
*/
#include "MuonInsideOutRecoTool.h"
@@ -211,7 +211,7 @@ namespace MuonCombined {
}
else
{
- for ( const auto& vx : *vertices )
+ for ( const auto vx : *vertices )
{
for ( const auto& tpLink : vx->trackParticleLinks() )
{
diff --git a/Reconstruction/eflowRec/src/PFChargedFlowElementCreatorAlgorithm.cxx b/Reconstruction/eflowRec/src/PFChargedFlowElementCreatorAlgorithm.cxx
index 899a06a9256e868abf1c518262f7610568a66432..b61eeb7851b792c492e75740ff160dba81fe50c1 100644
--- a/Reconstruction/eflowRec/src/PFChargedFlowElementCreatorAlgorithm.cxx
+++ b/Reconstruction/eflowRec/src/PFChargedFlowElementCreatorAlgorithm.cxx
@@ -29,7 +29,7 @@ StatusCode PFChargedFlowElementCreatorAlgorithm::execute(const EventContext& ctx
/* Create Charged FlowElements from all eflowCaloObjects */
SG::ReadHandle<eflowCaloObjectContainer> eflowCaloObjectContainerReadHandle(m_eflowCaloObjectContainerReadHandleKey,ctx);
- for (const auto& thisEflowCaloObject : *eflowCaloObjectContainerReadHandle) createChargedFlowElements(*thisEflowCaloObject,true,chargedFlowElementContainerWriteHandle);
+ for (const auto thisEflowCaloObject : *eflowCaloObjectContainerReadHandle) createChargedFlowElements(*thisEflowCaloObject,true,chargedFlowElementContainerWriteHandle);
std::sort(chargedFlowElementContainerWriteHandle->begin(), chargedFlowElementContainerWriteHandle->end(), [] (const xAOD::FlowElement* flowElement1, const xAOD::FlowElement* flowElement2) {return flowElement1->pt()>flowElement2->pt();});
diff --git a/Trigger/TrigT1/TrigT1CTP/src/CTPSimulation.cxx b/Trigger/TrigT1/TrigT1CTP/src/CTPSimulation.cxx
index 23c408107df9307e092dbe70ab5945ad5e4d4c4a..3f25d40b9864d8fe977624dfb706b898bbabf189 100644
--- a/Trigger/TrigT1/TrigT1CTP/src/CTPSimulation.cxx
+++ b/Trigger/TrigT1/TrigT1CTP/src/CTPSimulation.cxx
@@ -1,5 +1,5 @@
/*
- Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration
+ Copyright (C) 2002-2021 CERN for the benefit of the ATLAS collaboration
*/
#include "./CTPSimulation.h"
@@ -470,7 +470,7 @@ LVL1CTP::CTPSimulation::fillInputHistograms(const EventContext& context) const {
auto h0 = *get1DHist("/input/jets/jJetPt"); // calling operator* to get the Guard outside the filling loop
auto h1 = *get1DHist("/input/jets/jJetEta");
auto h2 = *get1DHist("/input/jets/jJetPhi");
- for( const auto & jet : *jFexJets ) {
+ for( const auto jet : *jFexJets ) {
h0->Fill(fabs(jet->et8x8()/1000.));
h1->Fill(jet->eta());
h2->Fill(jet->phi());
@@ -486,7 +486,7 @@ LVL1CTP::CTPSimulation::fillInputHistograms(const EventContext& context) const {
auto h0 = *get1DHist("/input/jets/jLJetPt");
auto h1 = *get1DHist("/input/jets/jLJetEta");
auto h2 = *get1DHist("/input/jets/jLJetPhi");
- for( const auto & jet : *jFexLJets ) {
+ for( const auto jet : *jFexLJets ) {
h0->Fill(fabs(jet->et8x8()/1000.));
h1->Fill(jet->eta());
h2->Fill(jet->phi());
@@ -502,7 +502,7 @@ LVL1CTP::CTPSimulation::fillInputHistograms(const EventContext& context) const {
auto h0 = *get1DHist("/input/jets/gJetPt");
auto h1 = *get1DHist("/input/jets/gJetEta");
auto h2 = *get1DHist("/input/jets/gJetPhi");
- for( const auto & jet : *gFexJets ) {
+ for( const auto jet : *gFexJets ) {
h0->Fill(fabs(jet->et8x8()/1000.));
h1->Fill(jet->eta());
h2->Fill(jet->phi());
@@ -541,7 +541,7 @@ LVL1CTP::CTPSimulation::fillInputHistograms(const EventContext& context) const {
auto h0 = *get1DHist("/input/em/et");
auto h1 = *get1DHist("/input/em/eta");
auto h2 = *get1DHist("/input/em/phi");
- for( const auto & cl : *eFexCluster ) {
+ for( const auto cl : *eFexCluster ) {
h0->Fill(cl->et());
h1->Fill(cl->eta());
h2->Fill(cl->phi());
@@ -563,7 +563,7 @@ LVL1CTP::CTPSimulation::fillInputHistograms(const EventContext& context) const {
auto h6 = *get1DHist("/input/tau/R3ClusterIso");
const static SG::AuxElement::ConstAccessor<float> accR3ClET ("R3ClusterET");
const static SG::AuxElement::ConstAccessor<float> accR3ClIso ("R3ClusterIso");
- for( const auto & tau : *eFexTau ) {
+ for( const auto tau : *eFexTau ) {
h0->Fill(tau->eT());
h1->Fill(tau->eta());
h2->Fill(tau->phi());
@@ -770,7 +770,7 @@ LVL1CTP::CTPSimulation::calculateJetMultiplicity( const TrigConf::L1Threshold &
if ( jets.isValid() ) {
auto pt = confThr.getAttribute<unsigned int>("pt");
auto ranges = confThr.getList("ranges");
- for ( const auto & jet : *jets ) {
+ for ( const auto jet : *jets ) {
if( (unsigned int) (jet->et8x8()/1000.) < pt ) continue;
// calculate eta index from eta
float eta = jet->eta();
@@ -833,7 +833,7 @@ LVL1CTP::CTPSimulation::calculateJetMultiplicity( const TrigConf::TriggerThresho
}
auto jets = SG::makeHandle( *rhk, context );
if ( jets.isValid() ) {
- for ( const auto & jet : *jets ) {
+ for ( const auto jet : *jets ) {
float eta = jet->eta();
float phi = jet->phi();
if ( phi < 0 ) phi += 2*M_PI;
@@ -871,7 +871,7 @@ LVL1CTP::CTPSimulation::calculateEMMultiplicity( const TrigConf::L1Threshold & c
// new EM threshold from eFEX
float scale = l1menu->getObject("thresholds.legacyCalo.EM.emscale").getValue<float>();
auto eFexCluster = SG::makeHandle( m_iKeyEFexCluster, context );
- for ( const auto & cl : *eFexCluster ) {
+ for ( const auto cl : *eFexCluster ) {
float eta = cl->eta();
int ieta = int((eta + (eta>0 ? 0.005 : -0.005))/0.1);
unsigned int thrV = confThr.thrValue( ieta );
@@ -903,7 +903,7 @@ LVL1CTP::CTPSimulation::calculateEMMultiplicity( const TrigConf::TriggerThreshol
if ( confThr->name()[0]=='e' ) {
// new EM threshold from eFEX
auto eFexCluster = SG::makeHandle( m_iKeyEFexCluster, context );
- for ( const auto & cl : *eFexCluster ) {
+ for ( const auto cl : *eFexCluster ) {
float eta = cl->eta();
int ieta = int((eta + (eta>0 ? 0.005 : -0.005))/0.1);
int iphi = 0;
@@ -939,7 +939,7 @@ LVL1CTP::CTPSimulation::calculateTauMultiplicity( const TrigConf::L1Threshold &
const static SG::AuxElement::ConstAccessor<float> accR3ClET ("R3ClusterET");
const static SG::AuxElement::ConstAccessor<float> accR3ClIso ("R3ClusterIso");
if( eFexTaus.isValid() ) {
- for ( const auto & tau : *eFexTaus ) {
+ for ( const auto tau : *eFexTaus ) {
unsigned int eT = (unsigned int) (accR3ClET(*tau)/1000.); // tau eT is in MeV while the cut is in GeV - this is only temporary and needs to be made consistent for all L1Calo
//float iso = accR3ClIso(*tau);
unsigned int etCut = confThr.data().get_child("et").get_value<unsigned int>();
@@ -975,7 +975,7 @@ LVL1CTP::CTPSimulation::calculateTauMultiplicity( const TrigConf::TriggerThresho
const static SG::AuxElement::ConstAccessor<float> accR3ClET ("R3ClusterET");
const static SG::AuxElement::ConstAccessor<float> accR3ClIso ("R3ClusterIso");
if( eFexTaus.isValid() ) {
- for ( const auto & tau : *eFexTaus ) {
+ for ( const auto tau : *eFexTaus ) {
unsigned int eT = (unsigned int) (accR3ClET(*tau)/1000.); // tau eT is in MeV while the cut is in GeV - this is only temporary and needs to be made consistent for all L1Calo
//float iso = accR3ClIso(*tau);
float eta = tau->eta();