Skip to content

Commit 54cc232

Browse files
EloviyoShirajum Monira
andauthored
[PWGCF] fixed incorrect index and O2 linter warnings (AliceO2Group#11266)
Co-authored-by: Shirajum Monira <shirajum.monira@cernch>
1 parent 4653ef9 commit 54cc232

File tree

1 file changed

+13
-13
lines changed

1 file changed

+13
-13
lines changed

PWGCF/TableProducer/filter2Prong.cxx

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -81,15 +81,15 @@ struct Filter2Prong {
8181
return; // rejected collision
8282
if (cfgVerbosity > 0 && candidates.size() > 0)
8383
LOGF(info, "Candidates for collision: %lu, cfcollisions: %lu, CFTracks: %lu", candidates.size(), cfcollisions.size(), cftracks.size());
84-
for (auto& c : candidates) {
84+
for (const auto& c : candidates) {
8585
int prongCFId[2] = {-1, -1};
86-
for (auto& cftrack : cftracks) {
86+
for (const auto& cftrack : cftracks) {
8787
if (c.prong0Id() == cftrack.trackId()) {
8888
prongCFId[0] = cftrack.globalIndex();
8989
break;
9090
}
9191
}
92-
for (auto& cftrack : cftracks) {
92+
for (const auto& cftrack : cftracks) {
9393
if (c.prong1Id() == cftrack.trackId()) {
9494
prongCFId[1] = cftrack.globalIndex();
9595
break;
@@ -106,10 +106,10 @@ struct Filter2Prong {
106106
prongCFId[0], prongCFId[1], c.pt(), c.eta(), c.phi(), hfHelper.invMassD0ToPiK(c), aod::cf2prongtrack::D0ToPiK);
107107
if constexpr (std::experimental::is_detected<HasMLProb, typename HFCandidatesType::iterator>::value) {
108108
mlvecd.clear();
109-
for (float val : c.mlProbD0())
109+
for (const float val : c.mlProbD0())
110110
mlvecd.push_back(val);
111111
mlvecdbar.clear();
112-
for (float val : c.mlProbD0bar())
112+
for (const float val : c.mlProbD0bar())
113113
mlvecdbar.push_back(val);
114114
output2ProngTrackmls(cfcollisions.begin().globalIndex(), mlvecd, mlvecdbar);
115115
}
@@ -120,10 +120,10 @@ struct Filter2Prong {
120120
prongCFId[0], prongCFId[1], c.pt(), c.eta(), c.phi(), hfHelper.invMassD0barToKPi(c), aod::cf2prongtrack::D0barToKPi);
121121
if constexpr (std::experimental::is_detected<HasMLProb, typename HFCandidatesType::iterator>::value) {
122122
mlvecd.clear();
123-
for (float val : c.mlProbD0())
123+
for (const float val : c.mlProbD0())
124124
mlvecd.push_back(val);
125125
mlvecdbar.clear();
126-
for (float val : c.mlProbD0bar())
126+
for (const float val : c.mlProbD0bar())
127127
mlvecdbar.push_back(val);
128128
output2ProngTrackmls(cfcollisions.begin().globalIndex(), mlvecd, mlvecdbar);
129129
}
@@ -146,15 +146,15 @@ struct Filter2Prong {
146146
void processMC(aod::McCollisions::iterator const&, aod::CFMcParticleRefs const& cfmcparticles, aod::McParticles const& mcparticles)
147147
{
148148
// The main filter outputs the primary MC particles. Here we just resolve the daughter indices that are needed for the efficiency matching.
149-
for (auto& r : cfmcparticles) {
149+
for (const auto& r : cfmcparticles) {
150150
const auto& mcParticle = mcparticles.iteratorAt(r.mcParticleId());
151151
if (mcParticle.daughtersIds().size() != 2) {
152152
output2ProngMcParts(-1, -1);
153153
continue;
154154
}
155155
int prongCFId[2] = {-1, -1};
156156
for (uint i = 0; i < 2; ++i) {
157-
for (auto& cfmcpart : cfmcparticles) {
157+
for (const auto& cfmcpart : cfmcparticles) {
158158
if (mcParticle.daughtersIds()[i] == cfmcpart.mcParticleId()) {
159159
prongCFId[i] = cfmcpart.globalIndex();
160160
break;
@@ -172,16 +172,16 @@ struct Filter2Prong {
172172
{
173173
if (cfcollisions.size() <= 0 || cftracks.size() <= 0)
174174
return; // rejected collision
175-
for (auto& cftrack1 : cftracks) {
176-
auto p1 = tracks.iteratorAt(cftrack1.trackId());
175+
for (const auto& cftrack1 : cftracks) {
176+
const auto& p1 = tracks.iteratorAt(cftrack1.trackId() - tracks.begin().globalIndex());
177177
if (p1.sign() != 1)
178178
continue;
179179
if (sigmaFormula->Eval(o2::aod::pidutils::tpcNSigma(cfgImPart1PID, p1), o2::aod::pidutils::tofNSigma(cfgImPart1PID, p1)) <= 0.0f)
180180
continue;
181-
for (auto& cftrack2 : cftracks) {
181+
for (const auto& cftrack2 : cftracks) {
182182
if (cftrack2.globalIndex() == cftrack1.globalIndex())
183183
continue;
184-
auto p2 = tracks.iteratorAt(cftrack2.trackId());
184+
const auto& p2 = tracks.iteratorAt(cftrack2.trackId() - tracks.begin().globalIndex());
185185
if (p2.sign() != -1)
186186
continue;
187187
if (sigmaFormula->Eval(o2::aod::pidutils::tpcNSigma(cfgImPart2PID, p2), o2::aod::pidutils::tofNSigma(cfgImPart2PID, p2)) <= 0.0f)

0 commit comments

Comments
 (0)