Skip to content

Commit 8d5bca4

Browse files
committed
[PWGCF] Lambda-Cascade correlation: fix cppcheck shadowing + unused member
1 parent f9557f4 commit 8d5bca4

1 file changed

Lines changed: 11 additions & 18 deletions

File tree

PWGCF/TwoParticleCorrelations/Tasks/Lambdacascadecorrelation.cxx

Lines changed: 11 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1451,14 +1451,14 @@ struct LambdaCascadeProducer {
14511451
histos.fill(HIST("Tracks/h1f_tracks_info"), kPassV0DauTrackSel);
14521452

14531453
// Apply Kinematic Selection
1454-
float rap = 0.;
1454+
float rapV0 = 0.;
14551455
if (!cDoEtaAnalysis) {
1456-
rap = std::abs(v0.yLambda());
1456+
rapV0 = std::abs(v0.yLambda());
14571457
} else {
1458-
rap = std::abs(v0.eta());
1458+
rapV0 = std::abs(v0.eta());
14591459
}
14601460

1461-
if (!kinCutSelection(v0.pt(), rap, cMinV0Pt, cMaxV0Pt, cMaxV0Rap)) {
1461+
if (!kinCutSelection(v0.pt(), rapV0, cMinV0Pt, cMaxV0Pt, cMaxV0Rap)) {
14621462
return false;
14631463
}
14641464

@@ -1646,7 +1646,7 @@ struct LambdaCascadeProducer {
16461646

16471647
// initialize efficiency factor and primary fraction values
16481648
float effCorrFact = 1., primFrac = 1.;
1649-
float rap = (cDoEtaAnalysis) ? v0.eta() : v0.yLambda();
1649+
float rapV0 = (cDoEtaAnalysis) ? v0.eta() : v0.yLambda();
16501650

16511651
// Get Efficiency Factor
16521652
if (cGetEffFact) {
@@ -1665,7 +1665,7 @@ struct LambdaCascadeProducer {
16651665
effCorrFact = histEff->GetBinContent(histEff->FindBin(cent, v0.pt()));
16661666
} else if (histEff->GetDimension() == ThreeDimCorr) {
16671667
histos.fill(HIST("Tracks/h1f_tracks_info"), kEffCorrPtRapCent);
1668-
effCorrFact = histEff->GetBinContent(histEff->FindBin(cent, v0.pt(), rap));
1668+
effCorrFact = histEff->GetBinContent(histEff->FindBin(cent, v0.pt(), rapV0));
16691669
} else {
16701670
histos.fill(HIST("Tracks/h1f_tracks_info"), kNoEffCorr);
16711671
LOGF(warning, "CCDB OBJECT IS NOT A HISTOGRAM !!!");
@@ -1691,7 +1691,7 @@ struct LambdaCascadeProducer {
16911691
primFrac = histPrm->GetBinContent(histPrm->FindBin(cent, v0.pt()));
16921692
} else if (histPrm->GetDimension() == ThreeDimCorr) {
16931693
histos.fill(HIST("Tracks/h1f_tracks_info"), kPFCorrPtRapCent);
1694-
primFrac = histPrm->GetBinContent(histPrm->FindBin(cent, v0.pt(), rap));
1694+
primFrac = histPrm->GetBinContent(histPrm->FindBin(cent, v0.pt(), rapV0));
16951695
} else {
16961696
histos.fill(HIST("Tracks/h1f_tracks_info"), kNoPFCorr);
16971697
LOGF(warning, "CCDB OBJECT IS NOT A HISTOGRAM !!!");
@@ -2030,7 +2030,7 @@ struct LambdaCascadeProducer {
20302030
// initialize track objects
20312031
ParticleType v0Type = kLambda;
20322032
PrmScdType v0PrmScdType = kPrimary;
2033-
float rap = 0.;
2033+
float rapGen = 0.;
20342034

20352035
for (auto const& mcpart : mcParticles) {
20362036
// check for Lambda first
@@ -2051,13 +2051,13 @@ struct LambdaCascadeProducer {
20512051

20522052
// Decide Eta/Rap
20532053
if (!cDoEtaAnalysis) {
2054-
rap = mcpart.y();
2054+
rapGen = mcpart.y();
20552055
} else {
2056-
rap = mcpart.eta();
2056+
rapGen = mcpart.eta();
20572057
}
20582058

20592059
// Apply Kinematic Acceptance
2060-
if (!kinCutSelection(mcpart.pt(), std::abs(rap), cMinV0Pt, cMaxV0Pt, cMaxV0Rap)) {
2060+
if (!kinCutSelection(mcpart.pt(), std::abs(rapGen), cMinV0Pt, cMaxV0Pt, cMaxV0Rap)) {
20612061
continue;
20622062
}
20632063

@@ -3254,13 +3254,6 @@ struct Lambdacascadecorrelation {
32543254
int64_t mLastEventCountRecoCollIdx{-1};
32553255
int64_t mLastEventCountGenCollIdx{-1};
32563256

3257-
// Gen-Λ singles guard. processMCGenXi and processMCGenOmega
3258-
// both iterate the same LambdaMcGenTracks table and fill the same
3259-
// McGen/Singles/Lambda/* histograms. When both flags are on (typical for
3260-
// a full Λ-Ξ + Λ-Ω closure run) every gen Λ would be counted twice.
3261-
// The guard makes the singles fills idempotent per gen event.
3262-
int64_t mLastGenLamSinglesCollIdx{-1};
3263-
32643257
// Helper: fill the per-event counter exactly once per reco collision.
32653258
// Reco-side guard. The histogram itself is filled here so
32663259
// hEventCount reports unique reco collisions (the canonical event count

0 commit comments

Comments
 (0)