Skip to content

Commit 281de79

Browse files
removed unusued variable and solved linter issues
1 parent 03e7aff commit 281de79

1 file changed

Lines changed: 20 additions & 12 deletions

File tree

PWGLF/TableProducer/Nuspex/LFTreeCreatorClusterStudies.cxx

Lines changed: 20 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -603,8 +603,9 @@ struct LfTreeCreatorClusterStudies {
603603
mHistograms.fill(HIST(cNames[partID]) + HIST("/nSigmaTPC"), track.p() * track.sign(), nsigmaTpc);
604604
mHistograms.fill(HIST(cNames[partID]) + HIST("/nSigmaITS"), track.p() * track.sign(), nsigmaIts);
605605
mHistograms.fill(HIST(cNames[partID]) + HIST("/nSigmaTOF"), track.p() * track.sign(), nsigmaTof);
606-
if (partID == static_cast<int>(PartID::de) || partID == static_cast<int>(PartID::he))
606+
if (partID == static_cast<int>(PartID::de) || partID == static_cast<int>(PartID::he)) {
607607
mHistograms.fill(HIST(cNames[partID]) + HIST("/TOFmass"), track.p() * track.sign(), massTof);
608+
}
608609
mHistograms.fill(HIST(cNames[partID]) + HIST("/pmatching"), correctedTpcInnerParam * track.sign(), (correctedTpcInnerParam - track.p()) / correctedTpcInnerParam);
609610
}
610611

@@ -758,7 +759,7 @@ struct LfTreeCreatorClusterStudies {
758759
}
759760

760761
const auto& timestamp = bc.timestamp();
761-
o2::parameters::GRPMagField* grpmag = 0x0;
762+
o2::parameters::GRPMagField* grpmag = nullptr;
762763

763764
auto grpmagPath{"GLO/Config/GRPMagField"};
764765
grpmag = mccdb->getForTimeStamp<o2::parameters::GRPMagField>("GLO/Config/GRPMagField", timestamp);
@@ -823,24 +824,29 @@ struct LfTreeCreatorClusterStudies {
823824
LOG(info) << "resolution: " << mBBparamsDe[5];
824825

825826
std::vector<std::string> collisionSelectionLabels = {"All", "sel8", "z_{VTX} < 10 cm"};
826-
for (int i = 0; i < Selections::kAll; i++)
827+
for (int i = 0; i < Selections::kAll; i++) {
827828
mHistograms.get<TH1>(HIST("collision_selections"))->GetXaxis()->SetBinLabel(i + 1, collisionSelectionLabels[i].c_str());
829+
}
828830

829831
std::vector<std::string> V0selectionLabels = {"All", "daughter track quality", "V0 topology", "V0 mass selection"};
830-
for (int i = 0; i < V0Selections::kV0All; i++)
832+
for (int i = 0; i < V0Selections::kV0All; i++) {
831833
mHistograms.get<TH1>(HIST("v0_selections"))->GetXaxis()->SetBinLabel(i + 1, V0selectionLabels[i].c_str());
834+
}
832835

833836
std::vector<std::string> CascSelectionLabels = {"All", "Topology", "n#sigma_{TPC} V0 daughters", "Veto Xi", "n#sigma_{TPC} K", "Accepted Omega"};
834-
for (int i = 0; i < CascSelections::kCascAll; i++)
837+
for (int i = 0; i < CascSelections::kCascAll; i++) {
835838
mHistograms.get<TH1>(HIST("casc_selections"))->GetXaxis()->SetBinLabel(i + 1, CascSelectionLabels[i].c_str());
839+
}
836840

837841
std::vector<std::string> ESelectionLabels = {"All", "Track quality", "Primary", "Pid", "#pi^{0}"};
838-
for (int i = 0; i < ESelections::kEAll; i++)
842+
for (int i = 0; i < ESelections::kEAll; i++) {
839843
mHistograms.get<TH1>(HIST("e_selections"))->GetXaxis()->SetBinLabel(i + 1, ESelectionLabels[i].c_str());
844+
}
840845

841846
std::vector<std::string> V0TypeLabels = {"K0s", "#Lambda", "#bar{#Lambda}", "Photon"};
842-
for (int i = 0; i < V0Type::V0TypeAll; i++)
847+
for (int i = 0; i < V0Type::V0TypeAll; i++) {
843848
mHistograms.get<TH1>(HIST("v0_type"))->GetXaxis()->SetBinLabel(i + 1, V0TypeLabels[i].c_str());
849+
}
844850
}
845851

846852
template <bool isMC = false, typename Tracks>
@@ -911,7 +917,7 @@ struct LfTreeCreatorClusterStudies {
911917
}
912918

913919
template <bool isMC = false, typename Track>
914-
void fillKCand(const std::array<float, 3>& pv, aod::CascDatas::iterator const& cascade, const aod::V0Datas& v0s, const Track&)
920+
void fillKCand(const std::array<float, 3>& pv, aod::CascDatas::iterator const& cascade, const Track&)
915921
{
916922
mHistograms.fill(HIST("casc_selections"), CascSelections::kCascNoCut);
917923
const auto& bachelorTrack = cascade.template bachelor_as<Track>();
@@ -999,8 +1005,9 @@ struct LfTreeCreatorClusterStudies {
9991005
}
10001006
mHistograms.fill(HIST(cNames[kPartID]) + HIST("/trackSelections"), NucleiSelections::kNucleiNoCut);
10011007

1002-
if (track.itsNCls() < desettingNclsIts)
1008+
if (track.itsNCls() < desettingNclsIts) {
10031009
return;
1010+
}
10041011
mHistograms.fill(HIST(cNames[kPartID]) + HIST("/trackSelections"), NucleiSelections::kNucleiNClsIts);
10051012

10061013
const float tpcNsigma = kPartID == static_cast<int>(PartID::de) ? track.tpcNSigmaDe() : computeNSigmaTPCHe3(track);
@@ -1288,7 +1295,7 @@ struct LfTreeCreatorClusterStudies {
12881295
}
12891296

12901297
for (const auto& cascade : cascades) {
1291-
fillKCand</*isMC*/ false>(PV, cascade, v0s, tracks);
1298+
fillKCand</*isMC*/ false>(PV, cascade, tracks);
12921299
}
12931300
}
12941301
PROCESS_SWITCH(LfTreeCreatorClusterStudies, processDataV0Casc, "process Data V0 and cascade", false);
@@ -1396,7 +1403,7 @@ struct LfTreeCreatorClusterStudies {
13961403
}
13971404

13981405
for (const auto& cascade : cascades) {
1399-
fillKCand</*isMC*/ true>(PV, cascade, v0s, tracks);
1406+
fillKCand</*isMC*/ true>(PV, cascade, tracks);
14001407
}
14011408
}
14021409
PROCESS_SWITCH(LfTreeCreatorClusterStudies, processMcV0Casc, "process Mc V0 and cascade", false);
@@ -1441,8 +1448,9 @@ struct LfTreeCreatorClusterStudies {
14411448
const auto& negTracks_thisCollision = negTracksMc.sliceByCached(o2::aod::track::collisionId, collision.globalIndex(), mcache);
14421449

14431450
for (const auto& [posTrack, negTrack] : o2::soa::combinations(o2::soa::CombinationsFullIndexPolicy(posTracks_thisCollision, negTracks_thisCollision))) {
1444-
if (!posTrack.has_mcParticle() || !negTrack.has_mcParticle())
1451+
if (!posTrack.has_mcParticle() || !negTrack.has_mcParticle()) {
14451452
continue;
1453+
}
14461454

14471455
fillElectronTableFromPi0Dalitz</*isMC*/ true>(posTrack, negTrack);
14481456
}

0 commit comments

Comments
 (0)