From d973debb69bc65d0bec9e0932a4aed6725a32a46 Mon Sep 17 00:00:00 2001
From: scott snyder <scott.snyder@cern.ch>
Date: Sat, 26 Aug 2017 05:31:27 +0200
Subject: [PATCH] TrigT1CaloTools: Fix clang warnings.
Fix operator precedence issue. Should not actually change results.
---
Trigger/TrigT1/TrigT1CaloTools/src/L1TriggerTowerTool.cxx | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/Trigger/TrigT1/TrigT1CaloTools/src/L1TriggerTowerTool.cxx b/Trigger/TrigT1/TrigT1CaloTools/src/L1TriggerTowerTool.cxx
index 6909e1811e4..a18188087f8 100644
--- a/Trigger/TrigT1/TrigT1CaloTools/src/L1TriggerTowerTool.cxx
+++ b/Trigger/TrigT1/TrigT1CaloTools/src/L1TriggerTowerTool.cxx
@@ -451,7 +451,7 @@ void L1TriggerTowerTool::bcid(const std::vector<int> &filter, const std::vector<
/// evaluate BCID decisions
std::vector<int> decisionRange;
- if (!decisionSource&0x1) etRange(digits, energyLow, energyHigh, decisionRange);
+ if (!(decisionSource&0x1)) etRange(digits, energyLow, energyHigh, decisionRange);
else etRange(lutInput, energyLow, energyHigh, decisionRange);
bcidDecision(result, decisionRange, decisionConditions, decision);
if (m_debug) {
@@ -682,7 +682,7 @@ void L1TriggerTowerTool::bcidDecisionRange(const std::vector<int>& lutInput, con
} else ATH_MSG_WARNING( "::bcidDecisionRange: No Conditions Container retrieved" );
- if (!decisionSource&0x1) etRange(digits, channelId, output);
+ if (!(decisionSource&0x1)) etRange(digits, channelId, output);
else etRange(lutInput, channelId, output);
if (m_debug) {
ATH_MSG_VERBOSE( "::bcidDecisionRange: decisionSource: " << decisionSource);
--
GitLab