Skip to content
Snippets Groups Projects
Commit 5fa6be2d authored by scott snyder's avatar scott snyder Committed by scott snyder
Browse files

LArReadoutGeometry: Fix cppcheck warnings.

Pass objects by const reference.
Suppress false positve where cppcheck doesn't handle CachedValue properly.
parent f8651c6e
No related branches found
No related tags found
No related merge requests found
/*
Copyright (C) 2002-2021 CERN for the benefit of the ATLAS collaboration
Copyright (C) 2002-2024 CERN for the benefit of the ATLAS collaboration
*/
// ***************************************************************************
......@@ -47,7 +47,7 @@ public:
class TubePosition { //
public: //
TubePosition(); //
TubePosition(tileName_t name, float x, float y, std::string hvFT);//
TubePosition(tileName_t name, float x, float y, const std::string& hvFT);//
tileName_t get_tileName() const; //
float x() const; //
float y() const; //
......@@ -186,7 +186,7 @@ FCAL_ChannelMap::TubePosition::TubePosition()
{}
inline
FCAL_ChannelMap::TubePosition::TubePosition(tileName_t name, float x, float y, std::string hvFT)
FCAL_ChannelMap::TubePosition::TubePosition(tileName_t name, float x, float y, const std::string& hvFT)
:
m_tileName(name),
m_x(x),
......
/*
Copyright (C) 2002-2022 CERN for the benefit of the ATLAS collaboration
Copyright (C) 2002-2024 CERN for the benefit of the ATLAS collaboration
*/
#include "GaudiKernel/MsgStream.h"
......@@ -146,6 +146,7 @@ const EMECHVManager& EMECDetectorManager::getHVManager (EMECHVManager::IOType io
m_HVManager[io].set (&(manager->getEMECHVManager(io)));
}
}
// cppcheck-suppress nullPointerRedundantCheck; false positive
return *(m_HVManager[io].get());
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment