From 1a1c85e9b70b3b0d49c4a92cc3a87d4d61924452 Mon Sep 17 00:00:00 2001
From: scott snyder <sss@karma>
Date: Mon, 22 Jun 2020 12:07:00 -0400
Subject: [PATCH] TileMonitoring: Fix clang 10 warning.
Avoid spurious object copy in range for.
---
.../TileMonitoring/src/TileRawChannelTimeMonTool.cxx | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/TileCalorimeter/TileMonitoring/src/TileRawChannelTimeMonTool.cxx b/TileCalorimeter/TileMonitoring/src/TileRawChannelTimeMonTool.cxx
index e2c0d12cff4..362be27c053 100644
--- a/TileCalorimeter/TileMonitoring/src/TileRawChannelTimeMonTool.cxx
+++ b/TileCalorimeter/TileMonitoring/src/TileRawChannelTimeMonTool.cxx
@@ -1,5 +1,5 @@
/*
- Copyright (C) 2002-2018 CERN for the benefit of the ATLAS collaboration
+ Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration
*/
// ********************************************************************
@@ -141,7 +141,7 @@ StatusCode TileRawChannelTimeMonTool::bookHists()
}
}
- for (const std::pair<int, int> roses : m_timeDifferenceBetweenROS) {
+ for (const std::pair<int, int>& roses : m_timeDifferenceBetweenROS) {
int ros1 = roses.first;
int ros2 = roses.second;
@@ -367,7 +367,7 @@ StatusCode TileRawChannelTimeMonTool::fillHists()
}
int rosesPairIndex(0);
- for (const std::pair<int, int> roses : m_timeDifferenceBetweenROS) {
+ for (const std::pair<int, int>& roses : m_timeDifferenceBetweenROS) {
int ros1 = roses.first;
int ros2 = roses.second;
@@ -377,7 +377,7 @@ StatusCode TileRawChannelTimeMonTool::fillHists()
} else {
int rosesPairIndex(0);
- for (const std::pair<int, int> roses : m_timeDifferenceBetweenROS) {
+ for (const std::pair<int, int>& roses : m_timeDifferenceBetweenROS) {
int ros1 = roses.first;
int ros2 = roses.second;
--
GitLab