From 1f309c1ce96181ec7923bcd9d276e489d20b19a9 Mon Sep 17 00:00:00 2001 From: Marco Clemencic <marco.clemencic@cern.ch> Date: Fri, 13 Oct 2023 12:54:55 +0200 Subject: [PATCH] Remove unused variables --- GaudiAlg/src/lib/GaudiTool.cpp | 2 -- GaudiKernel/src/Lib/ParsersVct.cpp | 5 ----- GaudiKernel/tests/src/test_GaudiTiming.cpp | 5 ++++- 3 files changed, 4 insertions(+), 8 deletions(-) diff --git a/GaudiAlg/src/lib/GaudiTool.cpp b/GaudiAlg/src/lib/GaudiTool.cpp index 9298a4386b..f0e0fafa95 100644 --- a/GaudiAlg/src/lib/GaudiTool.cpp +++ b/GaudiAlg/src/lib/GaudiTool.cpp @@ -49,8 +49,6 @@ template class GaudiCommon<AlgTool>; */ // ============================================================================ namespace GaudiToolServices { - /// the default name for Event Data Service - const std::string s_EventDataSvc = "EventDataSvc"; /// the default name for Detector Data Service const std::string s_DetectorDataSvc = "DetectorDataSvc"; /// the default name for Chrono & Stat Service diff --git a/GaudiKernel/src/Lib/ParsersVct.cpp b/GaudiKernel/src/Lib/ParsersVct.cpp index 1a22a34146..364234839c 100644 --- a/GaudiKernel/src/Lib/ParsersVct.cpp +++ b/GaudiKernel/src/Lib/ParsersVct.cpp @@ -29,11 +29,6 @@ namespace { return m.end(); } // ========================================================================== - const std::string s_X[] = { "x", "X", "px", "pX", "Px", "PX" }; - const std::string s_Y[] = { "y", "Y", "py", "pY", "Py", "PY" }; - const std::string s_Z[] = { "z", "Z", "pz", "pZ", "Pz", "PZ" }; - const std::string s_E[] = { "t", "T", "e", "E" }; - // ========================================================================== } // namespace // ============================================================================ namespace Gaudi { diff --git a/GaudiKernel/tests/src/test_GaudiTiming.cpp b/GaudiKernel/tests/src/test_GaudiTiming.cpp index b66e5a1401..31c4cb4a22 100644 --- a/GaudiKernel/tests/src/test_GaudiTiming.cpp +++ b/GaudiKernel/tests/src/test_GaudiTiming.cpp @@ -110,7 +110,10 @@ namespace GaudiKernelTest { // This should result in non-zero user and kernel times t0 = System::getProcessTime(); float x = 1.5; - long m = 0; + + // this variable is only needed to ensure that the timed loop + // takes a measurable time + [[maybe_unused]] long m = 0; for ( int i = 0; i < 10000; i++ ) { x *= sin( x ) / atan( x ) * tanh( x ) * sqrt( x ); m += System::virtualMemory(); -- GitLab