Skip to content

Commit 1f78ebc

Browse files
committed
Fixed magic numbers
1 parent 3f8b58a commit 1f78ebc

1 file changed

Lines changed: 5 additions & 2 deletions

File tree

PWGCF/Flow/Tasks/flowSP.cxx

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -548,9 +548,12 @@ struct FlowSP {
548548

549549
std::vector<double> paramsMultPVCut = cfgEvSelsMultPv;
550550
std::vector<double> paramsMultCut = cfgEvSelsMult;
551+
552+
// number of parameters required in cfgEvSelsMultPv and cfgEvSelsMult. (5 Low + 5 High)
553+
int nParams = 10;
551554

552-
if(paramsMultPVCut.size() < 10) LOGF(fatal, "cfgEvSelsMultPv not set properly.. size = %d (should be 10) --> Check your config files!", paramsMultPVCut.size());
553-
else if(paramsMultCut.size() < 10) LOGF(fatal, "cfgEvSelsMult not set properly.. size = %d (should be 10) --> Check your config files!", paramsMultCut.size());
555+
if(paramsMultPVCut.size() < nParams) LOGF(fatal, "cfgEvSelsMultPv not set properly.. size = %d (should be 10) --> Check your config files!", paramsMultPVCut.size());
556+
else if(paramsMultCut.size() < nParams) LOGF(fatal, "cfgEvSelsMult not set properly.. size = %d (should be 10) --> Check your config files!", paramsMultCut.size());
554557
else {
555558
fMultPVCutLow->SetParameters(paramsMultPVCut[0], paramsMultPVCut[1], paramsMultPVCut[2], paramsMultPVCut[3], paramsMultPVCut[4]);
556559
fMultPVCutHigh->SetParameters(paramsMultPVCut[5], paramsMultPVCut[6], paramsMultPVCut[7], paramsMultPVCut[8], paramsMultPVCut[9]);

0 commit comments

Comments
 (0)