Skip to content

Commit

Permalink
PWGLF: Fix compilation warnings (AliceO2Group#6479)
Browse files Browse the repository at this point in the history
* PWGLF: Fix compilation warnings

* Please consider the following formatting changes (#86)

---------

Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
  • Loading branch information
vkucera and alibuild authored Jun 11, 2024
1 parent ee24c9e commit 1e5174e
Show file tree
Hide file tree
Showing 18 changed files with 33 additions and 33 deletions.
4 changes: 2 additions & 2 deletions PWGLF/TableProducer/Common/mcCentrality.cxx
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ struct mcCentrality {

o2::pwglf::ParticleCounter<o2::framework::O2DatabasePDG> mCounter;

void init(o2::framework::InitContext& initContext)
void init(o2::framework::InitContext& /*initContext*/)
{
// Set up the CCDB
ccdb->setURL(url.value);
Expand Down Expand Up @@ -114,7 +114,7 @@ struct mcCentrality {
}

// Full tables (independent on central calibrations)
void process(aod::McCollision const& mcCollision,
void process(aod::McCollision const& /*mcCollision*/,
aod::McParticles const& mcParticles)
{
const float nFT0M = mCounter.countFT0A(mcParticles) + mCounter.countFT0C(mcParticles);
Expand Down
4 changes: 2 additions & 2 deletions PWGLF/TableProducer/Nuspex/ebyeMaker.cxx
Original file line number Diff line number Diff line change
Expand Up @@ -880,7 +880,7 @@ struct ebyeMaker {
}
PROCESS_SWITCH(ebyeMaker, processRun2, "process (Run 2)", false);

void processMcRun3(soa::Join<aod::Collisions, aod::McCollisionLabels, aod::EvSels, aod::CentFT0Cs> const& collisions, aod::McCollisions const& mcCollisions, TracksFullIU const& tracks, aod::V0s const& V0s, aod::McParticles const& mcParticles, aod::McTrackLabels const& mcLab, aod::BCsWithTimestamps const&)
void processMcRun3(soa::Join<aod::Collisions, aod::McCollisionLabels, aod::EvSels, aod::CentFT0Cs> const& collisions, aod::McCollisions const& /*mcCollisions*/, TracksFullIU const& tracks, aod::V0s const& V0s, aod::McParticles const& mcParticles, aod::McTrackLabels const& mcLab, aod::BCsWithTimestamps const&)
{
for (auto& collision : collisions) {
auto bc = collision.bc_as<aod::BCsWithTimestamps>();
Expand Down Expand Up @@ -948,7 +948,7 @@ struct ebyeMaker {
}
PROCESS_SWITCH(ebyeMaker, processMcRun3, "process MC (Run 3)", false);

void processMcRun2(soa::Join<aod::Collisions, aod::McCollisionLabels, aod::CentRun2V0Ms> const& collisions, aod::McCollisions const& mcCollisions, TracksFull const& tracks, aod::V0s const& V0s, aod::McParticles const& mcParticles, aod::McTrackLabels const& mcLab, BCsWithRun2Info const&)
void processMcRun2(soa::Join<aod::Collisions, aod::McCollisionLabels, aod::CentRun2V0Ms> const& collisions, aod::McCollisions const& /*mcCollisions*/, TracksFull const& tracks, aod::V0s const& V0s, aod::McParticles const& mcParticles, aod::McTrackLabels const& mcLab, BCsWithRun2Info const&)
{
for (auto& collision : collisions) {
auto bc = collision.bc_as<BCsWithRun2Info>();
Expand Down
2 changes: 1 addition & 1 deletion PWGLF/TableProducer/Resonances/LFResonanceInitializer.cxx
Original file line number Diff line number Diff line change
Expand Up @@ -329,7 +329,7 @@ struct reso2initializer {

// Check if the collision is INEL>0
template <typename MCColl, typename MCPart>
bool IsTrueINEL0(MCColl const& mccoll, MCPart const& mcparts)
bool IsTrueINEL0(MCColl const& /*mccoll*/, MCPart const& mcparts)
{
for (auto& mcparticle : mcparts) {
if (!mcparticle.isPhysicalPrimary())
Expand Down
2 changes: 1 addition & 1 deletion PWGLF/TableProducer/Strangeness/LFStrangeTreeCreator.cxx
Original file line number Diff line number Diff line change
Expand Up @@ -659,7 +659,7 @@ struct LFStrangeTreeCreator {
}
PROCESS_SWITCH(LFStrangeTreeCreator, processRun3, "process (Run 3)", false);

void processMcRun3(soa::Join<aod::Collisions, aod::McCollisionLabels, aod::EvSels, aod::CentFT0Cs> const& collisions, aod::McCollisions const& mcCollisions, TracksFullIU const& tracks, aod::V0s const& V0s, aod::Cascades const& cascades, aod::McParticles const& mcParticles, aod::McTrackLabels const& mcLab, aod::BCsWithTimestamps const&)
void processMcRun3(soa::Join<aod::Collisions, aod::McCollisionLabels, aod::EvSels, aod::CentFT0Cs> const& collisions, aod::McCollisions const& /*mcCollisions*/, TracksFullIU const& tracks, aod::V0s const& V0s, aod::Cascades const& cascades, aod::McParticles const& mcParticles, aod::McTrackLabels const& mcLab, aod::BCsWithTimestamps const&)
{
for (auto& collision : collisions) {
auto bc = collision.bc_as<aod::BCsWithTimestamps>();
Expand Down
2 changes: 1 addition & 1 deletion PWGLF/Tasks/Nuspex/AngularCorrelationsInJets.cxx
Original file line number Diff line number Diff line change
Expand Up @@ -166,7 +166,7 @@ struct AngularCorrelationsInJets {
//****************************************************************************************************

template <typename CollisionType, typename TracksType>
void angCorrData(const CollisionType& collision, const TracksType& tracks)
void angCorrData(const CollisionType& /*collision*/, const TracksType& tracks)
{
registryData.fill(HIST("hNumberOfEvents"), 0);
registryData.fill(HIST("hEventProtocol"), 0);
Expand Down
2 changes: 1 addition & 1 deletion PWGLF/Tasks/Nuspex/NucleiEfficiencyTask.cxx
Original file line number Diff line number Diff line change
Expand Up @@ -211,7 +211,7 @@ struct NucleiEfficiencyTask {
//****************************************************************************************************

template <typename CollisionType, typename TracksType, typename mcParticlesType>
void process_MC_reco(const CollisionType& collision, const TracksType& tracks, const mcParticlesType& mcParticles)
void process_MC_reco(const CollisionType& collision, const TracksType& tracks, const mcParticlesType& /*mcParticles*/)
{
if (event_selection_MC_sel8 && !collision.sel8())
return;
Expand Down
8 changes: 4 additions & 4 deletions PWGLF/Tasks/Nuspex/QCspectraTPC.cxx
Original file line number Diff line number Diff line change
Expand Up @@ -416,7 +416,7 @@ struct QCspectraTPC {
}
} // process_mc loop end
using CollisionCandidateMCRec = soa::Join<aod::Collisions, aod::McCollisionLabels, aod::EvSels, aod::CentFT0Cs>;
void processTrackHistograms_MC(CollisionCandidateMCRec::iterator const& collision, soa::Join<aod::Tracks, aod::TracksExtra, aod::TracksDCA, aod::McTrackLabels, aod::pidTOFFullPi, aod::pidTOFFullKa, aod::pidTOFFullPr, aod::TrackSelection> const& tracks, aod::McParticles const& mcParticles, aod::McCollisions const& mcCollisions)
void processTrackHistograms_MC(CollisionCandidateMCRec::iterator const& collision, soa::Join<aod::Tracks, aod::TracksExtra, aod::TracksDCA, aod::McTrackLabels, aod::pidTOFFullPi, aod::pidTOFFullKa, aod::pidTOFFullPr, aod::TrackSelection> const& tracks, aod::McParticles const& /*mcParticles*/, aod::McCollisions const& /*mcCollisions*/)
{
const float multiplicity = collision.centFT0C();
for (auto& track : tracks) {
Expand Down Expand Up @@ -600,8 +600,8 @@ struct QCspectraTPC {
PROCESS_SWITCH(QCspectraTPC, processTrackHistograms_MC, "process MC Reconstructed tracks", true);

void processParticleHistograms_MC(CollisionCandidateMCRec::iterator const& collision, soa::Join<aod::Tracks, aod::TracksExtra, aod::TracksDCA, aod::McTrackLabels, aod::pidTOFFullPi, aod::pidTOFFullKa, aod::pidTOFFullPr, aod::TrackSelection> const& tracks,
aod::McParticles const& mcParticles,
aod::McCollisions const& mcCollision)
aod::McParticles const& /*mcParticles*/,
aod::McCollisions const& /*mcCollision*/)
{

const float multiplicity = collision.centFT0C();
Expand Down Expand Up @@ -712,7 +712,7 @@ struct QCspectraTPC {
}
} // process_mc loop end
PROCESS_SWITCH(QCspectraTPC, processParticleHistograms_MC, "process MC Generated tracks", true);
void processParticleGen(aod::McCollision const& mcCollision, aod::McParticles& mcParticles)
void processParticleGen(aod::McCollision const& /*mcCollision*/, aod::McParticles& mcParticles)
{

for (auto& mcParticleGen : mcParticles) {
Expand Down
4 changes: 2 additions & 2 deletions PWGLF/Tasks/Nuspex/nucleiEbye.cxx
Original file line number Diff line number Diff line change
Expand Up @@ -321,7 +321,7 @@ struct nucleiEbye {
}

template <class C, class T, class V0>
int fillRecoEvent(C const& collision, T const& tracks, V0 const& V0s, float const& centrality)
int fillRecoEvent(C const& /*collision*/, T const& tracks, V0 const& V0s, float const& centrality)
{
candidateTracks[0].clear();
candidateTracks[1].clear();
Expand Down Expand Up @@ -607,7 +607,7 @@ struct nucleiEbye {
}

template <class C, class T, class V0>
void fillMcGen(C const& collision, T const& tracks, V0 const& v0s, float const& centrality)
void fillMcGen(C const& /*collision*/, T const& tracks, V0 const& v0s, float const& centrality)
{

tempTracks[0]->Reset();
Expand Down
6 changes: 3 additions & 3 deletions PWGLF/Tasks/QC/mcParticlePrediction.cxx
Original file line number Diff line number Diff line change
Expand Up @@ -296,9 +296,9 @@ struct mcParticlePrediction {
for (const auto& particle : mcParticles) {
particle.pdgCode();
const auto id = PIDExtended::pdgToId(particle);
if (id < 0) {
continue;
}
// if (id < 0) { // always false
// continue;
// }
if (!enabledParticlesArray[id]) {
continue;
}
Expand Down
2 changes: 1 addition & 1 deletion PWGLF/Tasks/QC/mcinelgt0.cxx
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ struct mcInelGt0 {
Service<o2::framework::O2DatabasePDG> pdgDB;
SliceCache cache;
Preslice<o2::aod::Tracks> perCollision = o2::aod::track::collisionId;
void process(o2::aod::McCollision const& collisionMC,
void process(o2::aod::McCollision const& /*collisionMC*/,
o2::soa::SmallGroups<o2::soa::Join<o2::aod::Collisions, o2::aod::McCollisionLabels>> const& collisions,
const soa::Join<o2::aod::TracksIU, o2::aod::TracksExtra, o2::aod::McTrackLabels>& tracks,
o2::aod::McParticles const& mcParticles)
Expand Down
2 changes: 1 addition & 1 deletion PWGLF/Tasks/QC/tracked_cascade_properties.cxx
Original file line number Diff line number Diff line change
Expand Up @@ -370,7 +370,7 @@ struct tracked_cascade_properties {
}

void processData(SelectedCollisions::iterator const& collision, aod::AssignedTrackedCascades const& trackedCascades,
aod::Cascades const&, FullTracks const& tracks)
aod::Cascades const&, FullTracks const& /*tracks*/)
{
registryQC.fill(HIST("number_of_events_data"), 0.5);
if (!collision.sel8())
Expand Down
6 changes: 3 additions & 3 deletions PWGLF/Tasks/Resonances/chargedkstaranalysis.cxx
Original file line number Diff line number Diff line change
Expand Up @@ -700,8 +700,8 @@ struct chargedkstaranalysis {

PROCESS_SWITCH(chargedkstaranalysis, processSE, "Process Same event", true);

void processME(EventCandidates const& collisions,
TrackCandidates const& tracks, V0TrackCandidate const& V0s)
void processME(EventCandidates const& /*collisions*/,
TrackCandidates const& /*tracks*/, V0TrackCandidate const& /*V0s*/)

/*
void processME(EventCandidates const& collisions,
Expand Down Expand Up @@ -883,7 +883,7 @@ struct chargedkstaranalysis {

void processRecMC(EventCandidatesMC::iterator const& collision,
TrackCandidatesMC const& tracks, V0TrackCandidatesMC const& V0s,
aod::McParticles const& mcParticles, aod::McCollisions const& mcCollisions)
aod::McParticles const& /*mcParticles*/, aod::McCollisions const& /*mcCollisions*/)

{

Expand Down
4 changes: 2 additions & 2 deletions PWGLF/Tasks/Resonances/lambdav2.cxx
Original file line number Diff line number Diff line change
Expand Up @@ -265,7 +265,7 @@ struct lambdav2 {

template <typename Collision, typename V0>
bool SelectionV0(Collision const& collision, V0 const& candidate,
float multiplicity)
float /*multiplicity*/)
{
if (fabs(candidate.dcav0topv()) > cMaxV0DCA) {
return false;
Expand Down Expand Up @@ -386,7 +386,7 @@ struct lambdav2 {
ROOT::Math::PxPyPzMVector Lambda;

void processSE(EventCandidates::iterator const& collision,
TrackCandidates const& tracks, aod::V0Datas const& V0s,
TrackCandidates const& /*tracks*/, aod::V0Datas const& V0s,
aod::BCs const&)

{
Expand Down
2 changes: 1 addition & 1 deletion PWGLF/Tasks/Resonances/phianalysisrun3_PbPb.cxx
Original file line number Diff line number Diff line change
Expand Up @@ -412,7 +412,7 @@ struct phianalysisrun3_PbPb {
}

PROCESS_SWITCH(phianalysisrun3_PbPb, processGen, "Process Generated", false);
void processRec(EventCandidatesMC::iterator const& collision, TrackCandidatesMC const& tracks, aod::McParticles const& mcParticles, aod::McCollisions const& mcCollisions)
void processRec(EventCandidatesMC::iterator const& collision, TrackCandidatesMC const& tracks, aod::McParticles const& /*mcParticles*/, aod::McCollisions const& /*mcCollisions*/)
{
if (!collision.has_mcCollision()) {
return;
Expand Down
2 changes: 1 addition & 1 deletion PWGLF/Tasks/Strangeness/derivedcascadeanalysis.cxx
Original file line number Diff line number Diff line change
Expand Up @@ -471,7 +471,7 @@ struct derivedCascadeAnalysis {
}

template <typename TCascade>
bool IsCascadeCandidateAccepted(TCascade casc, int counter, float centrality)
bool IsCascadeCandidateAccepted(TCascade casc, int counter, float /*centrality*/)
{
float cut = masswin;
histos.fill(HIST("hCutValue"), 2, cut);
Expand Down
2 changes: 1 addition & 1 deletion PWGLF/Tasks/Strangeness/lambdapolarization.cxx
Original file line number Diff line number Diff line change
Expand Up @@ -558,7 +558,7 @@ struct lambdapolarization {
}

void processData(EventCandidates::iterator const& collision,
TrackCandidates const& tracks, aod::V0Datas const& V0s,
TrackCandidates const& /*tracks*/, aod::V0Datas const& V0s,
aod::BCsWithTimestamps const&)
{
if (cfgCentEst == 1) {
Expand Down
4 changes: 2 additions & 2 deletions PWGLF/Tasks/Strangeness/strangeness_in_jets.cxx
Original file line number Diff line number Diff line change
Expand Up @@ -754,7 +754,7 @@ struct strangeness_in_jets {
return;
}

void processData(SelectedCollisions::iterator const& collision, aod::V0Datas const& fullV0s, aod::CascDataExt const& Cascades, aod::V0sLinked const& V0linked, FullTracks const& tracks)
void processData(SelectedCollisions::iterator const& collision, aod::V0Datas const& fullV0s, aod::CascDataExt const& Cascades, aod::V0sLinked const& /*V0linked*/, FullTracks const& tracks)
{
registryQC.fill(HIST("number_of_events_data"), 0.5);
if (!collision.sel8())
Expand Down Expand Up @@ -1094,7 +1094,7 @@ struct strangeness_in_jets {
Preslice<aod::CascDataExt> perCollisionCasc = o2::aod::cascade::collisionId;
Preslice<aod::McParticles> perMCCollision = o2::aod::mcparticle::mcCollisionId;

void processMCefficiency(SimCollisions const& collisions, MCTracks const& mcTracks, aod::V0Datas const& fullV0s, aod::CascDataExt const& Cascades, aod::McCollisions const& mcCollisions, const aod::McParticles& mcParticles)
void processMCefficiency(SimCollisions const& collisions, MCTracks const& /*mcTracks*/, aod::V0Datas const& fullV0s, aod::CascDataExt const& Cascades, aod::McCollisions const& /*mcCollisions*/, const aod::McParticles& mcParticles)
{
for (const auto& collision : collisions) {
registryQC.fill(HIST("number_of_events_mc"), 0.5);
Expand Down
8 changes: 4 additions & 4 deletions PWGLF/Tasks/Strangeness/vzero_cascade_absorption.cxx
Original file line number Diff line number Diff line change
Expand Up @@ -175,7 +175,7 @@ struct vzero_cascade_absorption {

// K0s Selections
template <typename V, typename T1, typename T2, typename C>
bool passedK0Selection(const V& v0, const T1& ntrack, const T2& ptrack, const C& collision)
bool passedK0Selection(const V& v0, const T1& ntrack, const T2& ptrack, const C& /*collision*/)
{
// Single-Track Selections
if (!passedSingleTrackSelection(ptrack))
Expand Down Expand Up @@ -224,7 +224,7 @@ struct vzero_cascade_absorption {

// Lambda Selections
template <typename V, typename T1, typename T2, typename C>
bool passedLambdaSelection(const V& v0, const T1& ntrack, const T2& ptrack, const C& collision)
bool passedLambdaSelection(const V& v0, const T1& ntrack, const T2& ptrack, const C& /*collision*/)
{
// Single-Track Selections
if (!passedSingleTrackSelection(ptrack))
Expand Down Expand Up @@ -273,7 +273,7 @@ struct vzero_cascade_absorption {

// AntiLambda Selections
template <typename V, typename T1, typename T2, typename C>
bool passedAntiLambdaSelection(const V& v0, const T1& ntrack, const T2& ptrack, const C& collision)
bool passedAntiLambdaSelection(const V& v0, const T1& ntrack, const T2& ptrack, const C& /*collision*/)
{
// Single-Track Selections
if (!passedSingleTrackSelection(ptrack))
Expand Down Expand Up @@ -418,7 +418,7 @@ struct vzero_cascade_absorption {
Preslice<aod::McParticles> perMCCollision = o2::aod::mcparticle::mcCollisionId;

// Process MC Rec
void processMCrec(SimCollisions const& collisions, MCTracks const& mcTracks, aod::V0Datas const& fullV0s, aod::McCollisions const& mcCollisions, const aod::McParticles& mcParticles)
void processMCrec(SimCollisions const& collisions, MCTracks const& /*mcTracks*/, aod::V0Datas const& fullV0s, aod::McCollisions const& /*mcCollisions*/, const aod::McParticles& /*mcParticles*/)
{

for (const auto& collision : collisions) {
Expand Down

0 comments on commit 1e5174e

Please sign in to comment.