Fix for codecheck

Author: zeptotera

PWGHF/HFC/TableProducer/correlatorLcHadrons.cxx

@@ -912,8 +912,8 @@ struct HfCorrelatorLcHadrons {
           }
         }
 
-        int8_t const chargeLc = pdg->GetParticle(particle.pdgCode())->Charge();         // Retrieve charge
-        int8_t const chargeAssoc = pdg->GetParticle(particleAssoc.pdgCode())->Charge(); // Retrieve charge
+        int8_t const chargeLc = static_cast<int8_t>(pdg->GetParticle(particle.pdgCode())->Charge());         // Retrieve charge
+        int8_t const chargeAssoc = static_cast<int8_t>(pdg->GetParticle(particleAssoc.pdgCode())->Charge()); // Retrieve charge
 
         int trackOrigin = RecoDecay::getCharmHadronOrigin(mcParticles, particleAssoc, true);
         registry.fill(HIST("hPtParticleAssocMcGen"), particleAssoc.pt());
@@ -1157,9 +1157,9 @@ struct HfCorrelatorLcHadrons {
         if (pidTrkApplied && (std::abs(particleAssoc.pdgCode()) != kProton)) {
           continue; // proton PID
         }
-        int8_t const chargeLc = pdg->GetParticle(candidate.pdgCode())->Charge();        // Retrieve charge
-        int8_t const chargeAssoc = pdg->GetParticle(particleAssoc.pdgCode())->Charge(); // Retrieve charge
-        float cent = 100.0;                                                             // will be updated later
+        int8_t const chargeLc = static_cast<int8_t>(pdg->GetParticle(candidate.pdgCode())->Charge());        // Retrieve charge
+        int8_t const chargeAssoc = static_cast<int8_t>(pdg->GetParticle(particleAssoc.pdgCode())->Charge()); // Retrieve charge
+        float cent = 100.0;                                                                                  // will be updated later
 
         int trackOrigin = RecoDecay::getCharmHadronOrigin(mcParticles, particleAssoc, true);
         bool isLcPrompt = candidate.originMcGen() == RecoDecay::OriginType::Prompt;

PWGHF/HFC/Tasks/taskCorrelationLcHadrons.cxx

@@ -52,6 +52,7 @@
 #include <THnSparse.h>
 #include <TString.h>
 
+#include <array>
 #include <chrono>
 #include <cstddef>
 #include <cstdint>
@@ -87,20 +88,37 @@ const TString stringMcGenLcFd = "MC gen, non-prompt #Lambda_c;";
 
 // definition of vectors for standard ptbin and invariant mass configurables
 const int nPtBinsCorrelations = 8;
-const double pTBinsCorrelations[nPtBinsCorrelations + 1] = {0., 2., 4., 6., 8., 12., 16., 24., 99.};
-const auto vecBinsPtCorrelations = std::vector<double>{pTBinsCorrelations, pTBinsCorrelations + nPtBinsCorrelations + 1};
-const double signalRegionInnerDefault[nPtBinsCorrelations] = {2.269, 2.269, 2.269, 2.269, 2.269, 2.269, 2.269, 2.269};
-const double signalRegionOuterDefault[nPtBinsCorrelations] = {2.309, 2.309, 2.309, 2.309, 2.309, 2.309, 2.309, 2.309};
-const double sidebandLeftOuterDefault[nPtBinsCorrelations] = {2.209, 2.209, 2.209, 2.209, 2.209, 2.209, 2.209, 2.209};
-const double sidebandLeftInnerDefault[nPtBinsCorrelations] = {2.249, 2.249, 2.249, 2.249, 2.249, 2.249, 2.249, 2.249};
-const double sidebandRightInnerDefault[nPtBinsCorrelations] = {2.329, 2.329, 2.329, 2.329, 2.329, 2.329, 2.329, 2.329};
-const double sidebandRightOuterDefault[nPtBinsCorrelations] = {2.369, 2.369, 2.369, 2.369, 2.369, 2.369, 2.369, 2.369};
-const auto vecSignalRegionInner = std::vector<double>{signalRegionInnerDefault, signalRegionInnerDefault + nPtBinsCorrelations};
-const auto vecSignalRegionOuter = std::vector<double>{signalRegionOuterDefault, signalRegionOuterDefault + nPtBinsCorrelations};
-const auto vecSidebandLeftInner = std::vector<double>{sidebandLeftInnerDefault, sidebandLeftInnerDefault + nPtBinsCorrelations};
-const auto vecSidebandLeftOuter = std::vector<double>{sidebandLeftOuterDefault, sidebandLeftOuterDefault + nPtBinsCorrelations};
-const auto vecSidebandRightInner = std::vector<double>{sidebandRightInnerDefault, sidebandRightInnerDefault + nPtBinsCorrelations};
-const auto vecSidebandRightOuter = std::vector<double>{sidebandRightOuterDefault, sidebandRightOuterDefault + nPtBinsCorrelations};
+
+const std::array<double, nPtBinsCorrelations + 1> pTBinsCorrelations = {
+  0., 2., 4., 6., 8., 12., 16., 24., 99.};
+const auto vecBinsPtCorrelations =
+  std::vector<double>{pTBinsCorrelations.begin(), pTBinsCorrelations.end()};
+
+const std::array<double, nPtBinsCorrelations> signalRegionInnerDefault = {
+  2.269, 2.269, 2.269, 2.269, 2.269, 2.269, 2.269, 2.269};
+const std::array<double, nPtBinsCorrelations> signalRegionOuterDefault = {
+  2.309, 2.309, 2.309, 2.309, 2.309, 2.309, 2.309, 2.309};
+const std::array<double, nPtBinsCorrelations> sidebandLeftOuterDefault = {
+  2.209, 2.209, 2.209, 2.209, 2.209, 2.209, 2.209, 2.209};
+const std::array<double, nPtBinsCorrelations> sidebandLeftInnerDefault = {
+  2.249, 2.249, 2.249, 2.249, 2.249, 2.249, 2.249, 2.249};
+const std::array<double, nPtBinsCorrelations> sidebandRightInnerDefault = {
+  2.329, 2.329, 2.329, 2.329, 2.329, 2.329, 2.329, 2.329};
+const std::array<double, nPtBinsCorrelations> sidebandRightOuterDefault = {
+  2.369, 2.369, 2.369, 2.369, 2.369, 2.369, 2.369, 2.369};
+
+const auto vecSignalRegionInner =
+  std::vector<double>{signalRegionInnerDefault.begin(), signalRegionInnerDefault.end()};
+const auto vecSignalRegionOuter =
+  std::vector<double>{signalRegionOuterDefault.begin(), signalRegionOuterDefault.end()};
+const auto vecSidebandLeftInner =
+  std::vector<double>{sidebandLeftInnerDefault.begin(), sidebandLeftInnerDefault.end()};
+const auto vecSidebandLeftOuter =
+  std::vector<double>{sidebandLeftOuterDefault.begin(), sidebandLeftOuterDefault.end()};
+const auto vecSidebandRightInner =
+  std::vector<double>{sidebandRightInnerDefault.begin(), sidebandRightInnerDefault.end()};
+const auto vecSidebandRightOuter =
+  std::vector<double>{sidebandRightOuterDefault.begin(), sidebandRightOuterDefault.end()};
 
 /// Lc-Hadron correlation pair filling task, from pair tables - for real data and data-like analysis (i.e. reco-level w/o matching request via Mc truth)
 struct HfTaskCorrelationLcHadrons {
@@ -405,7 +423,13 @@ struct HfTaskCorrelationLcHadrons {
 
     if (activateQA) {
       const int regionLimits = 6;
-      std::string labels[regionLimits] = {"SigReg Left", "SigReg Right", "Left SB Low", "Left SB Up", "Right SB Low", "Right SB Up"};
+      const std::array<std::string, regionLimits> labels = {
+        "SigReg Left",
+        "SigReg Right",
+        "Left SB Low",
+        "Left SB Up",
+        "Right SB Low",
+        "Right SB Up"};
       static const AxisSpec axisSidebandLimits = {regionLimits, 0.5, 6.5, ""};
       auto hSigSidebandLimits = registry.add<TH2>("Inputs/hSigSidebandLimits", "Signal and Sideband Limits;;#it{p}_{T} (GeV/#it{c})", {HistType::kTH2F, {axisSidebandLimits, {(std::vector<double>)binsPtCorrelations, "#it{p}_{T} (GeV/#it{c})"}}});
       for (int iLim = 0; iLim < regionLimits; iLim++) {
@@ -521,7 +545,7 @@ struct HfTaskCorrelationLcHadrons {
         registry.fill(HIST("hCorrel2DVsPtSignMass"), deltaPhi, deltaEta, ptLc, ptHadron, massLc, signPair, poolBin, efficiencyWeight);
       }
       // check if correlation entry belongs to signal region, sidebands or is outside both, and fill correlation plots
-      if (storeMass) {
+      if (storeMass != 0) {
         registry.fill(HIST("hCorrel2DVsPtGlobalRegion"), deltaPhi, deltaEta, ptLc, ptHadron, poolBin, massLc, efficiencyWeight);
         continue;
       }