[PWGEM] Fix compilation warnings (#8383)

Author: vkucera

PWGEM/Dilepton/Tasks/eventQC.cxx

@@ -579,16 +579,6 @@ struct eventQC {
       return false;
     }
 
-    uint32_t itsClusterSizes = track.itsClusterSizes();
-    int total_cluster_size = 0, nl = 0;
-    for (unsigned int layer = 0; layer < 7; layer++) {
-      int cluster_size_per_layer = (itsClusterSizes >> (layer * 4)) & 0xf;
-      if (cluster_size_per_layer > 0) {
-        nl++;
-      }
-      total_cluster_size += cluster_size_per_layer;
-    }
-
     return true;
   }
 

PWGEM/Dilepton/Tasks/tableReaderBarrel.cxx

@@ -293,7 +293,7 @@ struct AnalysisTrackSelection {
   Configurable<std::string> fConfigRunPeriods{"cfgRunPeriods", "LHC22f", "run periods for used data"};
   Configurable<bool> fConfigDummyRunlist{"cfgDummyRunlist", false, "If true, use dummy runlist"};
   Configurable<int> fConfigInitRunNumber{"cfgInitRunNumber", 543215, "Initial run number used in run by run checks"};
-  Configurable<int> fConfigNbTrackCut{"cfgNbTrackCut", 1, "Number of cuts including prefilter cut, need to be below 30"};
+  Configurable<std::size_t> fConfigNbTrackCut{"cfgNbTrackCut", 1, "Number of cuts including prefilter cut, need to be below 30"};
 
   std::vector<AnalysisCompositeCut> fTrackCuts;
   struct : ConfigurableGroup {
@@ -345,10 +345,9 @@ struct AnalysisTrackSelection {
   {
     fCurrentRun = 0;
 
-    int nbofcuts = fConfigNbTrackCut;
-    if (nbofcuts > 0 && CheckSize()) {
-      for (unsigned int icut = 0; icut < nbofcuts; ++icut) {
-        AnalysisCompositeCut* cut = new AnalysisCompositeCut(Form("trackcut%d", icut), Form("trackcut%d", icut));
+    if (fConfigNbTrackCut > 0 && CheckSize()) {
+      for (std::size_t icut = 0; icut < fConfigNbTrackCut; ++icut) {
+        AnalysisCompositeCut* cut = new AnalysisCompositeCut(Form("trackcut%zu", icut), Form("trackcut%zu", icut));
         cut->AddCut(GetTrackCut(icut));
         cut->AddCut(GetPIDCut(icut));
         fTrackCuts.push_back(*cut);
@@ -795,7 +794,7 @@ struct AnalysisEventMixing {
   // single particle selection tasks to preserve the correspondence between the track cut name and its
   //  bit position in the cuts bitmap
   // TODO: Create a configurable to specify exactly on which of the bits one should run the event mixing
-  Configurable<int> fConfigNbTrackCut{"cfgNbTrackCut", 1, "Number of cuts without prefilter cut, need to be consistent with the track selection"};
+  Configurable<std::size_t> fConfigNbTrackCut{"cfgNbTrackCut", 1, "Number of cuts without prefilter cut, need to be consistent with the track selection"};
   Configurable<int> fConfigMixingDepth{"cfgMixingDepth", 100, "Number of Events stored for event mixing"};
   Configurable<std::string> fConfigAddEventMixingHistogram{"cfgAddEventMixingHistogram", "", "Comma separated list of histograms"};
   Configurable<std::string> ccdburl{"ccdburl", "http://alice-ccdb.cern.ch", "url of the ccdb repository"};
@@ -832,11 +831,11 @@ struct AnalysisEventMixing {
     // Keep track of all the histogram class names to avoid composing strings in the event mixing pairing
     TString histNames = "";
     if (fConfigNbTrackCut > 0 && fConfigNbTrackCut < 31) {
-      for (int icut = 0; icut < fConfigNbTrackCut; ++icut) {
+      for (std::size_t icut = 0; icut < fConfigNbTrackCut; ++icut) {
         std::vector<TString> names = {
-          Form("PairsBarrelMEPM_trackcut%d", icut),
-          Form("PairsBarrelMEPP_trackcut%d", icut),
-          Form("PairsBarrelMEMM_trackcut%d", icut)};
+          Form("PairsBarrelMEPM_trackcut%zu", icut),
+          Form("PairsBarrelMEPP_trackcut%zu", icut),
+          Form("PairsBarrelMEMM_trackcut%zu", icut)};
         histNames += Form("%s;%s;%s;", names[0].Data(), names[1].Data(), names[2].Data());
         fTrackHistNames.push_back(names);
         fTwoTrackFilterMask |= (static_cast<uint32_t>(1) << icut);
@@ -952,8 +951,8 @@ struct AnalysisSameEventPairing {
   int fCurrentRun; // needed to detect if the run changed and trigger update of calibrations etc.
 
   OutputObj<THashList> fOutputList{"output"};
-  Configurable<int> fConfigNbTrackCut{"cfgNbTrackCut", 1, "Number of track cuts without prefilter cut, need to be consistent with the track selection"};
-  Configurable<int> fConfigNbPairCut{"cfgNbPairCut", 1, "Number of pair cuts, need to be below 4 right now"};
+  Configurable<std::size_t> fConfigNbTrackCut{"cfgNbTrackCut", 1, "Number of track cuts without prefilter cut, need to be consistent with the track selection"};
+  Configurable<std::size_t> fConfigNbPairCut{"cfgNbPairCut", 1, "Number of pair cuts, need to be below 4 right now"};
   Configurable<string> url{"ccdb-url", "http://alice-ccdb.cern.ch", "url of the ccdb repository"};
   Configurable<string> ccdbPath{"ccdb-path", "Users/lm", "base path to the ccdb object"};
   Configurable<int64_t> nolaterthan{"ccdb-no-later-than", std::chrono::duration_cast<std::chrono::milliseconds>(std::chrono::system_clock::now().time_since_epoch()).count(), "latest acceptable timestamp of creation for the object"};
@@ -1073,28 +1072,27 @@ struct AnalysisSameEventPairing {
     std::vector<TString> names;
 
     if (fConfigNbPairCut > 0 && CheckSize()) {
-      for (int icut = 0; icut < fConfigNbPairCut; ++icut) {
+      for (std::size_t icut = 0; icut < fConfigNbPairCut; ++icut) {
         fPairCuts.push_back(*GetPairCut(icut));
       }
     }
 
     if (fConfigNbTrackCut > 0 && fConfigNbTrackCut < 31) {   // if track cuts
-      for (int icut = 0; icut < fConfigNbTrackCut; ++icut) { // loop over track cuts
+      for (std::size_t icut = 0; icut < fConfigNbTrackCut; ++icut) { // loop over track cuts
         fTwoTrackFilterMask |= (static_cast<uint32_t>(1) << icut);
         // no pair cuts
         names = {
-          Form("PairsBarrelSEPM_trackcut%d", icut),
-          Form("PairsBarrelSEPP_trackcut%d", icut),
-          Form("PairsBarrelSEMM_trackcut%d", icut)};
+          Form("PairsBarrelSEPM_trackcut%zu", icut),
+          Form("PairsBarrelSEPP_trackcut%zu", icut),
+          Form("PairsBarrelSEMM_trackcut%zu", icut)};
         histNames += Form("%s;%s;%s;", names[0].Data(), names[1].Data(), names[2].Data());
         fTrackHistNames.push_back(names);
 
-        unsigned int npaircuts = fPairCuts.size();
-        for (int iPairCut = 0; iPairCut < npaircuts; ++iPairCut) { // loop over pair cuts
+        for (std::size_t iPairCut = 0; iPairCut < fPairCuts.size(); ++iPairCut) { // loop over pair cuts
           names = {
-            Form("PairsBarrelSEPM_trackcut%d_paircut%d", icut, iPairCut),
-            Form("PairsBarrelSEPP_trackcut%d_paircut%d", icut, iPairCut),
-            Form("PairsBarrelSEMM_trackcut%d_paircut%d", icut, iPairCut)};
+            Form("PairsBarrelSEPM_trackcut%zu_paircut%zu", icut, iPairCut),
+            Form("PairsBarrelSEPP_trackcut%zu_paircut%zu", icut, iPairCut),
+            Form("PairsBarrelSEMM_trackcut%zu_paircut%zu", icut, iPairCut)};
 
           histNames += Form("%s;%s;%s;", names[0].Data(), names[1].Data(), names[2].Data());
 
@@ -1147,7 +1145,7 @@ struct AnalysisSameEventPairing {
       }
 
       int iCut = 0;
-      for (int icut = 0; icut < fConfigNbTrackCut; icut++) {
+      for (std::size_t icut = 0; icut < fConfigNbTrackCut; icut++) {
         if (twoTrackFilter & (static_cast<uint32_t>(1) << icut)) {
           if (t1.sign() * t2.sign() < 0) {
             fHistMan->FillHistClass(histNames[iCut][0].Data(), VarManager::fgValues);