From 24ca04586409cf94bb50bea89c8dfedfbecd1d64 Mon Sep 17 00:00:00 2001
From: scott snyder <scott.snyder@cern.ch>
Date: Thu, 6 Apr 2017 15:58:48 +0200
Subject: [PATCH] TriggerMatchingTool: Fix compilation warnings.
Remove unused variable.
Fix signed/unsigned comparison.
Former-commit-id: f47febc0c8bd7c0d17ae55ff561959f4435010a6
---
Trigger/TrigAnalysis/TriggerMatchingTool/Root/MatchResult.cxx | 1 -
Trigger/TrigAnalysis/TriggerMatchingTool/Root/MatchingTool.cxx | 2 +-
2 files changed, 1 insertion(+), 2 deletions(-)
diff --git a/Trigger/TrigAnalysis/TriggerMatchingTool/Root/MatchResult.cxx b/Trigger/TrigAnalysis/TriggerMatchingTool/Root/MatchResult.cxx
index fe31f29d49d9..4bfbee38fc55 100644
--- a/Trigger/TrigAnalysis/TriggerMatchingTool/Root/MatchResult.cxx
+++ b/Trigger/TrigAnalysis/TriggerMatchingTool/Root/MatchResult.cxx
@@ -41,7 +41,6 @@ namespace Trig{
std::vector<const xAOD::IParticle*> MatchResult::bestMatchedObjects(){
double bestcost = 0;
- const ComboMatch* bestcombo = 0;
int i = 0;
int bestindex = -1;
for (const auto combresult : m_combresults){
diff --git a/Trigger/TrigAnalysis/TriggerMatchingTool/Root/MatchingTool.cxx b/Trigger/TrigAnalysis/TriggerMatchingTool/Root/MatchingTool.cxx
index 91ab50641113..4a1c658e8862 100644
--- a/Trigger/TrigAnalysis/TriggerMatchingTool/Root/MatchingTool.cxx
+++ b/Trigger/TrigAnalysis/TriggerMatchingTool/Root/MatchingTool.cxx
@@ -140,7 +140,7 @@ namespace Trig {
//record sum of costs
- for(int i = 0; i < result.assignment.size(); ++i){
+ for(size_t i = 0; i < result.assignment.size(); ++i){
result.total_cost += distance_matrix[i][result.assignment[i]];
}
--
GitLab