Skip to content

Commit 278d0aa

Browse files
vkuceraalibuild
andauthored
[PWGJE] Fix compilation warnings (#8385)
Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
1 parent f4ad6ef commit 278d0aa

File tree

8 files changed

+8
-11
lines changed

8 files changed

+8
-11
lines changed

PWGJE/Core/JetMatchingUtilities.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -473,7 +473,7 @@ auto getConstituents(T const& jet, U const& /*constituents*/)
473473
if constexpr (jetfindingutilities::isEMCALClusterTable<U>()) {
474474
return jet.template clusters_as<U>();
475475
} else if constexpr (jetfindingutilities::isDummyTable<U>()) { // this is for the case where EMCal clusters are tested but no clusters exist, like in the case of charged jet analyses
476-
return NULL;
476+
return nullptr;
477477
} else {
478478
return jet.template tracks_as<U>();
479479
}

PWGJE/TableProducer/rhoEstimator.cxx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ struct RhoEstimatorTask {
7171

7272
Filter trackCuts = (aod::jtrack::pt >= trackPtMin && aod::jtrack::pt < trackPtMax && aod::jtrack::eta > trackEtaMin && aod::jtrack::eta < trackEtaMax && aod::jtrack::phi >= trackPhiMin && aod::jtrack::phi <= trackPhiMax);
7373

74-
void processChargedCollisions(aod::JetCollision const& collision, soa::Filtered<aod::JetTracks> const& tracks)
74+
void processChargedCollisions(aod::JetCollision const& /*collision*/, soa::Filtered<aod::JetTracks> const& tracks)
7575
{
7676
inputParticles.clear();
7777
jetfindingutilities::analyseTracks<soa::Filtered<aod::JetTracks>, soa::Filtered<aod::JetTracks>::iterator>(inputParticles, tracks, trackSelection, trackingEfficiency);

PWGJE/Tasks/emcclustermonitor.cxx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,6 @@ struct ClusterMonitor {
109109
mHistManager.add("numberOfClustersSMBC", "number of clusters per supermodule per bunch crossing (ambiguous BCs)", o2HistType::kTH2F, {numberClustersAxis, {20, -0.5, 19.5, "SupermoduleID"}});
110110

111111
// cluster properties (matched clusters)
112-
int MaxMatched = 20; // maximum number of matched tracks, hardcoded in emcalCorrectionTask.cxx!
113112
mHistManager.add("clusterE", "Energy of cluster", o2HistType::kTH1F, {energyAxis});
114113
mHistManager.add("clusterEMatched", "Energy of cluster (with match)", o2HistType::kTH1F, {energyAxis});
115114
mHistManager.add("clusterESupermodule", "Energy of the cluster vs. supermoduleID", o2HistType::kTH2F, {energyAxis, supermoduleAxis});

PWGJE/Tasks/jetTutorial.cxx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -313,7 +313,7 @@ struct JetTutorialTask {
313313
return;
314314
}
315315
for (auto& jet : jets) {
316-
if (std::abs(RecoDecay::constrainAngle(jet.phi() - leadingTrackPhi, -o2::constants::math::PIHalf) > 0.6)) {
316+
if (std::abs(RecoDecay::constrainAngle(jet.phi() - leadingTrackPhi, -o2::constants::math::PIHalf)) > 0.6) {
317317
registry.fill(HIST("h_recoil_jet_pt"), jet.pt());
318318
registry.fill(HIST("h_recoil_jet_eta"), jet.eta());
319319
registry.fill(HIST("h_recoil_jet_phi"), jet.phi());

PWGJE/Tasks/nucleiInJets.cxx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1202,7 +1202,7 @@ struct nucleiInJets {
12021202
// add pid later
12031203

12041204
bool jetFlag = false;
1205-
for (int iDJet = 0; iDJet < mcdJetPt.size(); iDJet++) {
1205+
for (std::size_t iDJet = 0; iDJet < mcdJetPt.size(); iDJet++) {
12061206
double delPhi = TVector2::Phi_mpi_pi(mcdJetPhi[iDJet] - track.phi());
12071207
double delEta = mcdJetEta[iDJet] - track.eta();
12081208
double R = TMath::Sqrt((delEta * delEta) + (delPhi * delPhi));
@@ -1282,7 +1282,7 @@ struct nucleiInJets {
12821282
// add pid later
12831283

12841284
bool jetFlag = false;
1285-
for (int iDJet = 0; iDJet < mcpJetPt.size(); iDJet++) {
1285+
for (std::size_t iDJet = 0; iDJet < mcpJetPt.size(); iDJet++) {
12861286
double delPhi = TVector2::Phi_mpi_pi(mcpJetPhi[iDJet] - mcParticle.phi());
12871287
double delEta = mcpJetEta[iDJet] - mcParticle.eta();
12881288
double R = TMath::Sqrt((delEta * delEta) + (delPhi * delPhi));

PWGJE/Tasks/phiInJets.cxx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -821,7 +821,7 @@ struct phiInJets {
821821
int goodjets = 0;
822822
double jetpt = 0;
823823

824-
for (int i = 0; i < mcd_pt.size(); i++) {
824+
for (std::size_t i = 0; i < mcd_pt.size(); i++) {
825825
if (i == 0) {
826826
if (lResonance.M() > 1.005 && lResonance.M() < 1.035) {
827827
RealPhiCandWithJet++;

PWGJE/Tasks/statPromptPhoton.cxx

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -405,7 +405,7 @@ struct statPromptPhoton {
405405
using jfilteredMCClusters = soa::Filtered<jMCClusters>;
406406

407407
int nEventsRecMC_JE = 0;
408-
void processMCRec_JE(jfilteredCollisions::iterator const& collision, jfilteredMCClusters const& mcclusters, jTrackCandidates const& tracks, soa::Join<aod::JTracks, aod::JTrackExtras, aod::JTrackPIs> const& caltracks, aod::JMcParticles const&, TrackCandidates const&)
408+
void processMCRec_JE(jfilteredCollisions::iterator const& collision, jfilteredMCClusters const& mcclusters, jTrackCandidates const& tracks, soa::Join<aod::JTracks, aod::JTrackExtras, aod::JTrackPIs> const& /*caltracks*/, aod::JMcParticles const&, TrackCandidates const&)
409409
{
410410

411411
nEventsRecMC_JE++;
@@ -437,7 +437,6 @@ struct statPromptPhoton {
437437
histos.fill(HIST("REC_M02_AC"), mccluster.energy());
438438
histos.fill(HIST("REC_All_Energy"), mccluster.energy());
439439
bool photontrigger = false; // is a neutral cluster
440-
bool nonvetotrigger = false; // probably is not a neutral cluster
441440
bool vetotrigger = false; // might be a neutral cluster
442441
bool chargetrigger = false; // is definitely not a neutral cluster
443442
double photonPt = 0.0;
@@ -471,7 +470,6 @@ struct statPromptPhoton {
471470
histos.fill(HIST("REC_Cluster_QA"), 2.5);
472471
vetotrigger = true;
473472
} else {
474-
nonvetotrigger = true;
475473
photontrigger = true;
476474
histos.fill(HIST("REC_Cluster_QA"), 1.5);
477475
}

PWGJE/Tasks/v0qa.cxx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1258,7 +1258,7 @@ struct V0QA {
12581258

12591259
using DaughterJTracks = soa::Join<aod::JetTracks, aod::JTrackPIs>;
12601260
using DaughterTracks = soa::Join<aod::FullTracks, aod::TracksDCA, aod::TrackSelection, aod::TracksCov>;
1261-
void processV0TrackQA(aod::JetCollision const& jcoll, soa::Join<aod::CandidatesV0Data, aod::V0SignalFlags> const& v0s, DaughterJTracks const&, DaughterTracks const&)
1261+
void processV0TrackQA(aod::JetCollision const& /*jcoll*/, soa::Join<aod::CandidatesV0Data, aod::V0SignalFlags> const& v0s, DaughterJTracks const&, DaughterTracks const&)
12621262
{
12631263
// if (!jetderiveddatautilities::selectCollision(jcoll, eventSelection)) {
12641264
// return;

0 commit comments

Comments
 (0)