From 3fde7fe72bb36cac7a0301201aed2d23a6721e10 Mon Sep 17 00:00:00 2001
From: Scott Snyder <scott.snyder@cern.ch>
Date: Mon, 30 Jan 2023 13:57:45 +0100
Subject: [PATCH] CscSegmentMakers: Fix typo in
 CscSegmentUtilTool::matchLikelihood

CscSegmentMakers: Fix typo in CscSegmentUtilTool::matchLikelihood

phi/eta mixup.
Was causing this method not to do anything.
Fixes a cppcheck warning.
---
 .../CscSegmentMakers/src/CscSegmentUtilTool.cxx               | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/MuonSpectrometer/MuonReconstruction/MuonSegmentMakers/MuonSegmentMakerAlgs/CscSegmentMakers/src/CscSegmentUtilTool.cxx b/MuonSpectrometer/MuonReconstruction/MuonSegmentMakers/MuonSegmentMakerAlgs/CscSegmentMakers/src/CscSegmentUtilTool.cxx
index 60bd745ee85..848bdcb0eb3 100644
--- a/MuonSpectrometer/MuonReconstruction/MuonSegmentMakers/MuonSegmentMakerAlgs/CscSegmentMakers/src/CscSegmentUtilTool.cxx
+++ b/MuonSpectrometer/MuonReconstruction/MuonSegmentMakers/MuonSegmentMakerAlgs/CscSegmentMakers/src/CscSegmentUtilTool.cxx
@@ -1,5 +1,5 @@
 /*
-  Copyright (C) 2002-2022 CERN for the benefit of the ATLAS collaboration
+  Copyright (C) 2002-2023 CERN for the benefit of the ATLAS collaboration
 */
 
 #include "CscSegmentUtilTool.h"
@@ -2183,7 +2183,7 @@ double CscSegmentUtilTool::matchLikelihood(const MuonSegment& rsg, const MuonSeg
     ICscSegmentFinder::RioList etarios;
     for (unsigned int irot = 0; irot < rsg.numberOfContainedROTs(); irot++) etarios.push_back(rsg.rioOnTrack(irot));
     ICscSegmentFinder::RioList phirios;
-    for (unsigned int irot = 0; irot < psg.numberOfContainedROTs(); irot++) etarios.push_back(psg.rioOnTrack(irot));
+    for (unsigned int irot = 0; irot < psg.numberOfContainedROTs(); irot++) phirios.push_back(psg.rioOnTrack(irot)); // CHECK THIS
     int maxeta = etarios.size();
     int maxphi = phirios.size();
 
-- 
GitLab