PWGEM: Fix compilation warnings: unused-parameter (#5655)

* PWGEM: Fix compilation warnings: unused-parameter

* Please consider the following formatting changes (#55)

* Resolve conflicts

* Please consider the following formatting changes (#56)

---------

Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>

Author: vkucera

PWGEM/Dilepton/TableProducer/treeCreatorElectronML.cxx

@@ -170,7 +170,7 @@ struct TreeCreatorElectronML {
   int mRunNumber;
   float d_bz;
   Service<o2::ccdb::BasicCCDBManager> ccdb;
-  void init(InitContext& context)
+  void init(InitContext&)
   {
     mRunNumber = 0;
     d_bz = 0;
@@ -703,7 +703,7 @@ struct TreeCreatorElectronML {
   } // end of process
   PROCESS_SWITCH(TreeCreatorElectronML, processPair, "produce ML input for pair level", false);
 
-  void processDummy(soa::Join<aod::Collisions, aod::EvSels, aod::Mults, aod::McCollisionLabels> const& collisions) {}
+  void processDummy(soa::Join<aod::Collisions, aod::EvSels, aod::Mults, aod::McCollisionLabels> const&) {}
   PROCESS_SWITCH(TreeCreatorElectronML, processDummy, "process dummy", true);
 };
 

PWGEM/Dilepton/TableProducer/treeCreatorElectronMLDDA.cxx

@@ -212,7 +212,7 @@ struct TreeCreatorElectronMLDDA {
   std::mt19937 engine;
   std::uniform_real_distribution<float> dist01;
 
-  void init(InitContext& context)
+  void init(InitContext&)
   {
     mRunNumber = 0;
     d_bz = 0;
@@ -912,7 +912,7 @@ struct TreeCreatorElectronMLDDA {
   Partition<MyTracks> positrons = o2::aod::track::signed1Pt > 0.f && o2::aod::track::pt > minpt&& nabs(o2::aod::track::eta) < maxeta&& o2::aod::track::tpcChi2NCl < maxchi2tpc&& o2::aod::track::itsChi2NCl < maxchi2its&& nabs(o2::aod::track::dcaXY) < maxdcaXY&& nabs(o2::aod::track::dcaZ) < maxdcaZ&& ncheckbit(aod::track::v001::detectorMap, (uint8_t)o2::aod::track::ITS) == true && ncheckbit(aod::track::v001::detectorMap, (uint8_t)o2::aod::track::TPC) == true && (minTPCNsigmaEl < o2::aod::pidtpc::tpcNSigmaEl && o2::aod::pidtpc::tpcNSigmaEl < maxTPCNsigmaEl);
   Partition<MyTracks> electrons = o2::aod::track::signed1Pt < 0.f && o2::aod::track::pt > minpt&& nabs(o2::aod::track::eta) < maxeta&& o2::aod::track::tpcChi2NCl < maxchi2tpc&& o2::aod::track::itsChi2NCl < maxchi2its&& nabs(o2::aod::track::dcaXY) < maxdcaXY&& nabs(o2::aod::track::dcaZ) < maxdcaZ&& ncheckbit(aod::track::v001::detectorMap, (uint8_t)o2::aod::track::ITS) == true && ncheckbit(aod::track::v001::detectorMap, (uint8_t)o2::aod::track::TPC) == true && (minTPCNsigmaEl < o2::aod::pidtpc::tpcNSigmaEl && o2::aod::pidtpc::tpcNSigmaEl < maxTPCNsigmaEl);
   std::vector<uint64_t> stored_secondary_electronIds;
-  void processPrimary(filteredMyCollisions const& collisions, aod::BCsWithTimestamps const&, aod::V0s const& v0s, MyTracks const& tracks)
+  void processPrimary(filteredMyCollisions const& collisions, aod::BCsWithTimestamps const&, aod::V0s const&, MyTracks const&)
   {
     stored_trackIds.reserve(positrons.size() + electrons.size());
     stored_secondary_electronIds.reserve(positrons.size() + electrons.size());
@@ -1029,7 +1029,7 @@ struct TreeCreatorElectronMLDDA {
   } // end of process
 
   // please choose only 1 process function.
-  void processDummy(filteredMyCollisions const& collisions) {}
+  void processDummy(filteredMyCollisions const&) {}
 
   PROCESS_SWITCH(TreeCreatorElectronMLDDA, processPID, "produce ML input for single track level", false);     // this is for eID with ITSsa. e/pi/k/p are selected by TOF, and these can be used for ITS-TPC PID.
   PROCESS_SWITCH(TreeCreatorElectronMLDDA, processPrimary, "produce ML input for single track level", false); // this is for selecting electrons from primary or secondary.

PWGEM/Dilepton/Tasks/MCtemplates.cxx

@@ -130,7 +130,7 @@ struct AnalysisEventSelection {
   }
 
   template <uint32_t TEventFillMap, uint32_t TEventMCFillMap, typename TEvent, typename TEventsMC>
-  void runEventSelection(TEvent const& event, TEventsMC const& mcEvents)
+  void runEventSelection(TEvent const& event, TEventsMC const&)
   {
     // Reset the values array
     VarManager::ResetValues(0, VarManager::kNEventWiseVariables);
@@ -249,7 +249,7 @@ struct AnalysisTrackSelection {
   }
 
   template <uint32_t TTrackFillMap, uint32_t TTrackMCFillMap, typename TTracks, typename TTracksMC>
-  void runTrackSelection(TTracks const& tracks, TTracksMC const& tracksMC)
+  void runTrackSelection(TTracks const& tracks, TTracksMC const&)
   {
     uint32_t filterMap = 0;
     trackSel.reserve(tracks.size());
@@ -581,7 +581,7 @@ struct AnalysisSameEventPairing {
 
   void processDecayToEESkimmed(soa::Filtered<MyEventsSelected>::iterator const& event,
                                soa::Filtered<MyBarrelTracksSelected> const& tracks,
-                               ReducedMCEvents const& eventsMC, ReducedMCTracks const& tracksMC)
+                               ReducedMCEvents const&, ReducedMCTracks const& tracksMC)
   {
     // Reset the fValues array
     VarManager::ResetValues(0, VarManager::kNVars);
@@ -596,7 +596,7 @@ struct AnalysisSameEventPairing {
 
   void processDecayToEEVertexingSkimmed(soa::Filtered<MyEventsVtxCovSelected>::iterator const& event,
                                         soa::Filtered<MyBarrelTracksSelectedWithCov> const& tracks,
-                                        ReducedMCEvents const& eventsMC, ReducedMCTracks const& tracksMC)
+                                        ReducedMCEvents const&, ReducedMCTracks const& tracksMC)
   {
     // Reset the fValues array
     VarManager::ResetValues(0, VarManager::kNVars);
@@ -611,7 +611,7 @@ struct AnalysisSameEventPairing {
 
   void processDecayToEEAOD(soa::Filtered<MyEventsSelectedAOD>::iterator const& event,
                            soa::Filtered<MyBarrelTracksSelectedAOD> const& tracks,
-                           aod::McCollisions const& eventsMC, aod::McParticles const& tracksMC)
+                           aod::McCollisions const&, aod::McParticles const& tracksMC)
   {
     // Reset the fValues array
     VarManager::ResetValues(0, VarManager::kNVars);

PWGEM/Dilepton/Tasks/emEfficiencyEE.cxx

@@ -135,7 +135,7 @@ struct AnalysisEventSelection {
   }
 
   template <uint32_t TEventFillMap, uint32_t TEventMCFillMap, typename TEvent, typename TEventsMC>
-  void runSelection(TEvent const& event, TEventsMC const& mcEvents)
+  void runSelection(TEvent const& event, TEventsMC const& /*mcEvents*/)
   {
     // Reset the values array
     VarManager::ResetValues(0, VarManager::kNEventWiseVariables);
@@ -211,7 +211,7 @@ struct AnalysisEventQa {
   Preslice<aod::McParticles> perMcCollision = aod::mcparticle::mcCollisionId;
 
   template <uint32_t TEventFillMap, uint32_t TEventMCFillMap, uint32_t TTrackMCFillMap, typename TEvents, typename TEventsMC, typename TTracksMC>
-  void runSelection(TEvents const& events, TEventsMC const& eventsMC, TTracksMC const& tracksMC)
+  void runSelection(TEvents const& events, TEventsMC const& /*eventsMC*/, TTracksMC const& tracksMC)
   {
 
     uint8_t eventFilter = 0;
@@ -616,7 +616,7 @@ struct AnalysisTrackSelection {
   Preslice<MyBarrelTracksNoSkimmed> perCollisionTracks = aod::track::collisionId;
 
   template <uint32_t TEventFillMap, uint32_t TEventMCFillMap, uint32_t TTrackFillMap, uint32_t TTrackMCFillMap, typename TEvents, typename TTracks, typename TEventsMC, typename TTracksMC>
-  void runSelection(TEvents const& events, TTracks const& tracks, TEventsMC const& eventsMC, TTracksMC const& tracksMC, bool write)
+  void runSelection(TEvents const& events, TTracks const& tracks, TEventsMC const& /*eventsMC*/, TTracksMC const& tracksMC, bool write)
   {
 
     uint8_t eventFilter = 0;
@@ -985,7 +985,7 @@ struct AnalysisSameEventPairing {
 
   // QA: to be defined
 
-  void init(o2::framework::InitContext& context)
+  void init(o2::framework::InitContext&)
   {
 
     // Binning 2D histos
@@ -1063,7 +1063,7 @@ struct AnalysisSameEventPairing {
   Preslice<MyBarrelTracksNoSkimmed> perCollisionTracks = aod::track::collisionId;
 
   template <uint32_t TEventFillMap, uint32_t TEventMCFillMap, uint32_t TTrackFillMap, typename TEvents, typename TTracks, typename TEventsMC, typename TTracksMC>
-  void runPairing(TEvents const& events, TTracks const& tracks, TEventsMC const& eventsMC, TTracksMC const& tracksMC)
+  void runPairing(TEvents const& events, TTracks const& tracks, TEventsMC const& /*eventsMC*/, TTracksMC const& tracksMC)
   {
 
     std::map<uint64_t, int> fMCEventLabels;
@@ -1122,7 +1122,7 @@ struct AnalysisSameEventPairing {
   }   // end loop pairing function
 
   template <uint32_t TEventMCFillMap, uint32_t TTrackFillMap, typename TEventMC, typename TTracksMC>
-  void runMCPairing(TEventMC const& eventMC, TTracksMC const& tracksMC)
+  void runMCPairing(TEventMC const& /*eventMC*/, TTracksMC const& tracksMC)
   {
 
     runMCGenPair<true>(tracksMC);
@@ -1237,7 +1237,7 @@ struct AnalysisSameEventPairing {
   }   // end runMCGen
 
   template <uint32_t TTrackFillMap, typename TTracks, typename TTracksMC>
-  void runRecPair(TTracks const& tracks, TTracksMC const& tracksMC)
+  void runRecPair(TTracks const& tracks, TTracksMC const& /*tracksMC*/)
   {
     //
     Double_t masse = 0.00051099895; // 0.5 MeV/c2 -> 0.0005 GeV/c2 , to be taken differently

PWGEM/Dilepton/Tasks/lmeeLFCocktail.cxx

@@ -158,7 +158,7 @@ struct lmeelfcocktail {
 
   Configurable<std::vector<double>> fConfigDCATemplateEdges{"cfgDCATemplateEdges", {0., .3, .4, .6, 1., 2.}, "DCA template edges"};
 
-  void init(o2::framework::InitContext& ic)
+  void init(o2::framework::InitContext&)
   {
     if (fConfigWriteTTree) {
       SetTree();

PWGEM/Dilepton/Tasks/smearing.cxx

@@ -42,7 +42,7 @@ struct ApplySmearing {
 
   MomentumSmearer smearer;
 
-  void init(InitContext& context)
+  void init(InitContext&)
   {
     smearer.setResFileName(TString(fConfigResFileName));
     smearer.setResPtHistName(TString(fConfigResPtHistName));
@@ -87,9 +87,9 @@ struct ApplySmearing {
     applySmearing(tracksMC);
   }
 
-  void processDummyCocktail(aod::McParticles const& tracksMC) {}
+  void processDummyCocktail(aod::McParticles const&) {}
 
-  void processDummyMCanalysis(ReducedMCTracks const& tracksMC) {}
+  void processDummyMCanalysis(ReducedMCTracks const&) {}
 
   PROCESS_SWITCH(ApplySmearing, processMCanalysis, "Run for MC analysis", false);
   PROCESS_SWITCH(ApplySmearing, processCocktail, "Run for cocktail analysis", false);
@@ -175,8 +175,8 @@ struct CheckSmearing {
     Check(tracksMC);
   }
 
-  void processDummyMCanalysis(ReducedMCTracks const& tracksMC) {}
-  void processDummyCocktail(aod::McParticles const& tracksMC) {}
+  void processDummyMCanalysis(ReducedMCTracks const&) {}
+  void processDummyCocktail(aod::McParticles const&) {}
 
   PROCESS_SWITCH(CheckSmearing, processCheckMCanalysis, "Run for MC analysis", false);
   PROCESS_SWITCH(CheckSmearing, processCheckCocktail, "Run for cocktail analysis", false);

PWGEM/PhotonMeson/Core/CutsLibrary.cxx

@@ -667,10 +667,10 @@ EMCPhotonCut* o2::aod::pwgem::photon::emccuts::GetCut(const char* cutName)
     cut->SetM02Range(0.0f, 1000.f);
     cut->SetTimeRange(-500.f, 500.f);
 
-    cut->SetTrackMatchingEta([](float pT) {
+    cut->SetTrackMatchingEta([](float /*pT*/) {
       return -1.f;
     });
-    cut->SetTrackMatchingPhi([](float pT) {
+    cut->SetTrackMatchingPhi([](float /*pT*/) {
       return -1.f;
     });
     cut->SetMinEoverP(0.f);

PWGEM/PhotonMeson/Core/EMCPhotonCut.h

@@ -75,7 +75,7 @@ class EMCPhotonCut : public TNamed
   // Temporary function to check if cluster passes a given selection criteria. To be replaced by framework filters.
   // Returns true if a cluster survives the cuts!
   template <typename Cluster, typename Track>
-  bool IsSelectedEMCal(const EMCPhotonCuts& cut, Cluster const& cluster, Track const& track) const
+  bool IsSelectedEMCal(const EMCPhotonCuts& cut, Cluster const& cluster, Track const& /*track*/) const
   {
     switch (cut) {
       case EMCPhotonCuts::kEnergy:

PWGEM/PhotonMeson/Core/HistogramsLibrary.h

@@ -56,7 +56,7 @@ void DefineHistograms(THashList* list, const char* histClass, const char* subGro
 THashList* AddHistClass(THashList* list, const char* histClass);
 
 template <EMHistType htype1, typename T1>
-void FillHistClass(THashList* list, const char* subGroup, T1 const& obj1, const float weight = 1.f)
+void FillHistClass(THashList* list, const char* subGroup, T1 const& obj1 /*, const float weight = 1.f*/)
 {
   if constexpr (htype1 == EMHistType::kEvent || htype1 == EMHistType::kEvent_Cent || htype1 == EMHistType::kEvent_Cent_Qvec) {
     reinterpret_cast<TH1F*>(list->FindObject("hCollisionCounter"))->Fill("all", 1.f);

PWGEM/PhotonMeson/TableProducer/associateMCinfo.cxx

@@ -78,7 +78,7 @@ struct AssociateMCInfo {
   Preslice<MyEMCClusters> perCollision_emc = aod::skimmedcluster::collisionId;
 
   template <uint8_t system, typename TTracks, typename TPCMs, typename TPCMLegs, typename TPHOSs, typename TEMCs, typename TEMPrimaryElectrons, typename TEMPrimaryMuons>
-  void skimmingMC(MyCollisionsMC const& collisions, aod::BCs const&, aod::McCollisions const&, aod::McParticles const& mcTracks, TTracks const& o2tracks, TPCMs const& v0photons, TPCMLegs const& v0legs, TPHOSs const& phosclusters, TEMCs const& emcclusters, TEMPrimaryElectrons const& emprimaryelectrons, TEMPrimaryMuons const& emprimarymuons)
+  void skimmingMC(MyCollisionsMC const& collisions, aod::BCs const&, aod::McCollisions const&, aod::McParticles const& mcTracks, TTracks const& o2tracks, TPCMs const& v0photons, TPCMLegs const& /*v0legs*/, TPHOSs const& /*phosclusters*/, TEMCs const& emcclusters, TEMPrimaryElectrons const& emprimaryelectrons, TEMPrimaryMuons const& emprimarymuons)
   {
     // temporary variables used for the indexing of the skimmed MC stack
     std::map<uint64_t, int> fNewLabels;
@@ -526,7 +526,7 @@ struct AssociateMCInfo {
     skimmingMC<sysflag>(collisions, bcs, mccollisions, mcTracks, o2tracks, v0photons, v0legs, phosclusters, emcclusters, emprimaryelectrons, nullptr);
   }
 
-  void processDummy(MyCollisionsMC const& collisions) {}
+  void processDummy(MyCollisionsMC const&) {}
 
   PROCESS_SWITCH(AssociateMCInfo, processMC_PCM, "create em mc event table for PCM", false);
   PROCESS_SWITCH(AssociateMCInfo, processMC_PCM_DalitzEE, "create em mc event table for PCM, DalitzEE", false);