From 9039851bb0d62b923f852f17546b6647d8b3758a Mon Sep 17 00:00:00 2001 From: Frank Winklmeier <frank.winklmeier@cern.ch> Date: Thu, 28 Nov 2024 16:12:38 +0100 Subject: [PATCH] TrigConfL1Data: remove unused setPriorities method Remove unused method taking an array (we have the same method taking a std::vector). Fixes cppcheck defect. --- .../TrigConfL1Data/Root/PrioritySet.cxx | 9 +-------- .../TrigConfL1Data/TrigConfL1Data/PrioritySet.h | 3 +-- 2 files changed, 2 insertions(+), 10 deletions(-) diff --git a/Trigger/TrigConfiguration/TrigConfL1Data/Root/PrioritySet.cxx b/Trigger/TrigConfiguration/TrigConfL1Data/Root/PrioritySet.cxx index abfbdd20015a..63f402b474ef 100644 --- a/Trigger/TrigConfiguration/TrigConfL1Data/Root/PrioritySet.cxx +++ b/Trigger/TrigConfiguration/TrigConfL1Data/Root/PrioritySet.cxx @@ -1,5 +1,5 @@ /* - Copyright (C) 2002-2018 CERN for the benefit of the ATLAS collaboration + Copyright (C) 2002-2024 CERN for the benefit of the ATLAS collaboration */ #include "TrigConfL1Data/PrioritySet.h" @@ -12,13 +12,6 @@ TrigConf::PrioritySet::PrioritySet() : L1DataBaseclass(),m_Priorities(N_PRIORITIES,"LOW") {} -void -TrigConf::PrioritySet::setPriorities(const std::string p[], const unsigned int size) { - if (size > N_PRIORITIES) std::cerr - << "setPriorities >> number of priorities too high: " << size << std::endl; - for (unsigned int i=0; i<size; i++) m_Priorities.at(i) = p[i]; -} - void TrigConf::PrioritySet::setPriorities(const std::vector<std::string>& vec) { if (vec.size() > N_PRIORITIES) std::cerr diff --git a/Trigger/TrigConfiguration/TrigConfL1Data/TrigConfL1Data/PrioritySet.h b/Trigger/TrigConfiguration/TrigConfL1Data/TrigConfL1Data/PrioritySet.h index d1efdf7c34e7..13d1cbcaf01d 100644 --- a/Trigger/TrigConfiguration/TrigConfL1Data/TrigConfL1Data/PrioritySet.h +++ b/Trigger/TrigConfiguration/TrigConfL1Data/TrigConfL1Data/PrioritySet.h @@ -1,5 +1,5 @@ /* - Copyright (C) 2002-2018 CERN for the benefit of the ATLAS collaboration + Copyright (C) 2002-2024 CERN for the benefit of the ATLAS collaboration */ #ifndef TrigConf_PrioritySet @@ -22,7 +22,6 @@ namespace TrigConf { // Accessors const std::vector<std::string>& priorities() const { return m_Priorities; } - void setPriorities( const std::string p[], const unsigned int size); void setPriorities(const std::vector<std::string>& vec); virtual void print(const std::string& indent="", unsigned int detail=1) const override; -- GitLab