From 3e155680309c0886d08e9804bebb130c5bc415c7 Mon Sep 17 00:00:00 2001
From: scott snyder <scott.snyder@cern.ch>
Date: Mon, 24 Apr 2017 13:53:32 +0200
Subject: [PATCH] CalibrationDataInterface: Fix apparent typo in conditions.

The conditional here doesn't make sense, and gets a warning from gcc7
because the value being tested is a constant.  Changing it to match
the way it was done other places in this source.


Former-commit-id: 334f95375d3f18e81a10e790e16e28737195ef50
---
 .../Root/CalibrationDataInterfaceROOT.cxx                       | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/PhysicsAnalysis/JetTagging/JetTagPerformanceCalibration/CalibrationDataInterface/Root/CalibrationDataInterfaceROOT.cxx b/PhysicsAnalysis/JetTagging/JetTagPerformanceCalibration/CalibrationDataInterface/Root/CalibrationDataInterfaceROOT.cxx
index 611d03a1392..ae31b6fbf89 100644
--- a/PhysicsAnalysis/JetTagging/JetTagPerformanceCalibration/CalibrationDataInterface/Root/CalibrationDataInterfaceROOT.cxx
+++ b/PhysicsAnalysis/JetTagging/JetTagPerformanceCalibration/CalibrationDataInterface/Root/CalibrationDataInterfaceROOT.cxx
@@ -1626,7 +1626,7 @@ Analysis::CalibrationDataInterfaceROOT::getWeightScaleFactor (const CalibrationD
       return Analysis::kError;
     }
     // the 'extrapolation' uncertainty (always a named one) needs a somewhat special treatment
-    bool extrapolate = SFNamed ? eigenVariation->isExtrapolationVariation(numVariation) : false;
+    bool extrapolate = (unc==SFNamed) ? eigenVariation->isExtrapolationVariation(numVariation) : false;
     
     double valueUp;
     double valueDown;
-- 
GitLab