From 137c224832e9dfef6beb7e3d4cbc1c176d6a2025 Mon Sep 17 00:00:00 2001 From: David Abdurachmanov Date: Wed, 6 Jan 2016 10:50:40 +0100 Subject: [PATCH] Add missing parentheses in DTTSCand::isLL() Logical not is applied to int value, which is a clear mistake. The patch applies logical not to the comparison. L1Trigger/DTTriggerServerPhi/interface/DTTSCand.h:119:77: warning: comparison of constant '8' with boolean expression is always false [-Wbool-compare] L1Trigger/DTTriggerServerPhi/interface/DTTSCand.h:120:77: warning: logical not is only applied to the left hand side of comparison [-Wlogical-not-parentheses] L1Trigger/DTTriggerServerPhi/interface/DTTSCand.h:120:77: warning: comparison of constant '80' with boolean expression is always false [-Wbool-compare] Signed-off-by: David Abdurachmanov (cherry picked from commit 168db818551634993a7db41e0f6cbd74789c17f5) --- L1Trigger/DTTriggerServerPhi/interface/DTTSCand.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/L1Trigger/DTTriggerServerPhi/interface/DTTSCand.h b/L1Trigger/DTTriggerServerPhi/interface/DTTSCand.h index 3862d32460350..d799177d4d68b 100644 --- a/L1Trigger/DTTriggerServerPhi/interface/DTTSCand.h +++ b/L1Trigger/DTTriggerServerPhi/interface/DTTSCand.h @@ -116,8 +116,8 @@ class DTTSCand { inline int isLH() const { return _tctrig->pvCorr() && _tctrig->pvCode()==8; } /// Return if LL - inline int isLL() const { return _tctrig->pvCorr() && !_tctrig->pvCode()==8 && - !_tctrig->pvCode()==80; } + inline int isLL() const { return _tctrig->pvCorr() && !(_tctrig->pvCode()==8) && + !(_tctrig->pvCode()==80); } /// Return if H inner inline int isH0() const { return !_tctrig->pvCorr() && _tctrig->pvCode()==80; }