Skip to content

Commit 0643647

Browse files
committed
[PWGDQ] more linter fixes
1 parent 947bffc commit 0643647

1 file changed

Lines changed: 4 additions & 9 deletions

File tree

PWGDQ/Tasks/qaMatching.cxx

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1653,7 +1653,7 @@ struct QaMatching {
16531653
if (mchMcParticle.globalIndex() != mftMcParticle.globalIndex())
16541654
return false;
16551655

1656-
if (std::abs(mchMcParticle.pdgCode()) != 13)
1656+
if (std::abs(mchMcParticle.pdgCode()) != kMuonMinus)
16571657
return false;
16581658

16591659
return true;
@@ -1664,7 +1664,8 @@ struct QaMatching {
16641664
TMUONS const& /*muonTracks*/,
16651665
TMFTS const& /*mftTracks*/)
16661666
{
1667-
if (static_cast<int>(muonTrack.trackType()) >= 2)
1667+
static constexpr int maxGlobalFwdTrackType = 2;
1668+
if (static_cast<int>(muonTrack.trackType()) >= maxGlobalFwdTrackType)
16681669
return false;
16691670

16701671
auto const& mchTrack = muonTrack.template matchMCHTrack_as<TMUONS>();
@@ -1696,18 +1697,12 @@ struct QaMatching {
16961697
} else {
16971698
result = (ranking == 1) ? kMatchTypeWrongLeading : kMatchTypeWrongNonLeading;
16981699
}
1699-
} else if (decayRanking == 2) {
1700+
} else if (decayRanking == 1) {
17001701
result = (ranking == 1) ? kMatchTypeDecayLeading : kMatchTypeDecayNonLeading;
17011702
} else {
17021703
result = (ranking == 1) ? kMatchTypeFakeLeading : kMatchTypeFakeNonLeading;
17031704
}
17041705

1705-
if (result == kMatchTypeUndefined) {
1706-
std::cout << std::format("[GetMatchType] isPaired={} isMuon={} decayRanking={} result={}",
1707-
isPaired, isMuon, decayRanking, static_cast<int>(result))
1708-
<< std::endl;
1709-
}
1710-
17111706
return result;
17121707
}
17131708

0 commit comments

Comments
 (0)