Merge branch 'clang.TrigCostMonitorMT-20210209' into 'master'

TrigCostMonitorMT+MuonRPC_CnvTools: Fix clang warnings.

See merge request atlas/athena!40502

MuonSpectrometer/MuonCnv/MuonRPC_CnvTools/src/RpcRdoToPrepDataToolCore.h

@@ -37,7 +37,7 @@ public:
   RpcRdoToPrepDataToolCore( const std::string&, const std::string&, const IInterface* );
 
   // to allow access to the IMuonRdoToPrepDataTool interface
-  StatusCode queryInterface( const InterfaceID& riid, void** ppvIf );
+  virtual StatusCode queryInterface( const InterfaceID& riid, void** ppvIf ) override;
 
   // setup/teardown functions, similar like those for Algorithm/Service
   virtual StatusCode initialize() override;
@@ -47,8 +47,8 @@ public:
   virtual StatusCode decode( const std::vector<uint32_t>& robIds ) override;
 
   // debugging 
-  void printInputRdo();
-  void printPrepData();
+  virtual void printInputRdo() override;
+  virtual void printPrepData() override;
   void printCoinData();
 
   // to resolve possible conflicts with IProperty::interfaceID()

Trigger/TrigMonitoring/TrigCostMonitorMT/src/TrigCostMTSvc.cxx

@@ -459,7 +459,7 @@ StatusCode TrigCostMTSvc::generateTimeoutReport(const EventContext& context, std
   // Save top 5 times to the report
   report = "Timeout detected with the following algorithms consuming the most time: ";
   int algCounter = 0;
-  for(const std::pair<uint64_t, std::string>& p : timeToAlgMap){
+  for(const std::pair<const uint64_t, std::string>& p : timeToAlgMap){
     // Save time in miliseconds instead of microseconds
     report += p.second + " (" + std::round(p.first/3.) + " ms)";
     ++algCounter;