Skip to content

Commit b8ab7f0

Browse files
committed
Fix performance
1 parent cf5e5a7 commit b8ab7f0

File tree

9 files changed

+13
-10
lines changed

9 files changed

+13
-10
lines changed

PWGHF/D2H/TableProducer/dataCreatorCharmHadPiReduced.cxx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1330,6 +1330,7 @@ struct HfDataCreatorCharmHadPiReduced {
13301330

13311331
if constexpr (doMc) {
13321332
std::vector<typename TTracks::iterator> beautyHadDauTracks{};
1333+
beautyHadDauTracks.reserve(charmHadDauTracks.size());
13331334
for (const auto& track : charmHadDauTracks) {
13341335
beautyHadDauTracks.push_back(track);
13351336
}

PWGHF/D2H/TableProducer/dataCreatorCharmResoReduced.cxx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -443,8 +443,8 @@ struct HfDataCreatorCharmResoReduced {
443443
template <typename Coll, typename Tr>
444444
bool buildAndSelectV0(const Coll& collision, const std::array<int, 3>& dDaughtersIds, const std::array<Tr, 2>& dauTracks)
445445
{
446-
auto trackPos = dauTracks[0];
447-
auto trackNeg = dauTracks[1];
446+
const auto& trackPos = dauTracks[0];
447+
const auto& trackNeg = dauTracks[1];
448448
// single-tracks selection
449449
if (!selectV0Daughter(trackPos, dDaughtersIds) || !selectV0Daughter(trackNeg, dDaughtersIds))
450450
return false;

PWGHF/D2H/TableProducer/dataCreatorJpsiHadReduced.cxx

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -913,6 +913,7 @@ struct HfDataCreatorJpsiHadReduced {
913913

914914
if constexpr (doMc) {
915915
std::vector<typename TTracks::iterator> beautyHadDauTracks{};
916+
beautyHadDauTracks.reserve(jPsiDauTracks.size());
916917
for (const auto& track : jPsiDauTracks) {
917918
beautyHadDauTracks.push_back(track);
918919
}
@@ -1034,6 +1035,7 @@ struct HfDataCreatorJpsiHadReduced {
10341035

10351036
if constexpr (doMc) {
10361037
std::vector<typename TTracks::iterator> beautyHadDauTracks{};
1038+
beautyHadDauTracks.reserve(jPsiDauTracks.size());
10371039
for (const auto& track : jPsiDauTracks) {
10381040
beautyHadDauTracks.push_back(track);
10391041
}

PWGHF/HFC/TableProducer/femtoDreamProducer.cxx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -264,7 +264,7 @@ struct HfFemtoDreamProducer {
264264
}
265265

266266
/// Function to retrieve the nominal magnetic field in kG (0.1T) and convert it directly to T
267-
void getMagneticFieldTesla(aod::BCsWithTimestamps::iterator bc)
267+
void getMagneticFieldTesla(const aod::BCsWithTimestamps::iterator& bc)
268268
{
269269
initCCDB(bc, runNumber, ccdb, !isRun3 ? ccdbPathGrp : ccdbPathGrpMag, lut, !isRun3);
270270
}

PWGHF/HFL/Tasks/taskElectronWeakBoson.cxx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -366,7 +366,7 @@ struct HfTaskElectronWeakBoson {
366366
return std::make_pair(trackCount - 1, isoMomentum);
367367
}
368368

369-
void recoMassZee(KFParticle kfpIsoEle,
369+
void recoMassZee(KFParticle const& kfpIsoEle,
370370
int charge,
371371
float centrality,
372372
TrackEle const& tracks)

PWGHF/TableProducer/candidateCreatorLb.cxx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -248,7 +248,7 @@ struct HfCandidateCreatorLb {
248248
rowCandidateProngs(lcCand.globalIndex(), trackPion.globalIndex());
249249
// calculate invariant mass
250250
auto arrayMomenta = std::array{pvecLc, pvecPion};
251-
massLcPi = RecoDecay::m(std::move(arrayMomenta), std::array{massLc, massPi});
251+
massLcPi = RecoDecay::m(arrayMomenta, std::array{massLc, massPi});
252252
if (lcCand.isSelLcToPKPi() > 0) {
253253
hMassLbToLcPi->Fill(massLcPi);
254254
}

PWGHF/TableProducer/candidateCreatorXicToXiPiPi.cxx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -311,7 +311,7 @@ struct HfCandidateCreatorXicToXiPiPi {
311311

312312
// get invariant mass of Xic candidate
313313
auto arrayMomenta = std::array{pVecXi, pVecPi0, pVecPi1};
314-
massXiPiPi = RecoDecay::m(std::move(arrayMomenta), std::array{MassXiMinus, MassPiPlus, MassPiPlus});
314+
massXiPiPi = RecoDecay::m(arrayMomenta, std::array{MassXiMinus, MassPiPlus, MassPiPlus});
315315

316316
// get track impact parameters
317317
// This modifies track momenta!
@@ -336,9 +336,9 @@ struct HfCandidateCreatorXicToXiPiPi {
336336

337337
// get invariant mass of Xi-pi pairs
338338
auto arrayMomentaXiPi0 = std::array{pVecXi, pVecPi0};
339-
massXiPi0 = RecoDecay::m(std::move(arrayMomentaXiPi0), std::array{MassXiMinus, MassPiPlus});
339+
massXiPi0 = RecoDecay::m(arrayMomentaXiPi0, std::array{MassXiMinus, MassPiPlus});
340340
auto arrayMomentaXiPi1 = std::array{pVecXi, pVecPi1};
341-
massXiPi1 = RecoDecay::m(std::move(arrayMomentaXiPi1), std::array{MassXiMinus, MassPiPlus});
341+
massXiPi1 = RecoDecay::m(arrayMomentaXiPi1, std::array{MassXiMinus, MassPiPlus});
342342

343343
// get uncertainty of the decay length
344344
float phi, theta;

PWGHF/TableProducer/trackIndexSkimCreator.cxx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1933,7 +1933,7 @@ struct HfTrackIndexSkimCreator {
19331933
aod::BCsWithTimestamps const&,
19341934
std::vector<int64_t> const& vecPvContributorGlobId,
19351935
std::vector<o2::track::TrackParCov> const& vecPvContributorTrackParCov,
1936-
std::vector<int64_t> vecCandPvContributorGlobId,
1936+
std::vector<int64_t> const& vecCandPvContributorGlobId,
19371937
std::array<float, 3>& pvCoord,
19381938
std::array<float, 6>& pvCovMatrix)
19391939
{

PWGHF/TableProducer/treeCreatorTccToD0D0Pi.cxx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -646,7 +646,7 @@ struct HfTreeCreatorTccToD0D0Pi {
646646
auto massKpipi1 = RecoDecay::m(std::array{pVecD1Prong0, pVecD1Prong1, pVecSoftPi}, std::array{massD1Daus[0], massD1Daus[1], MassPiPlus});
647647
auto massKpipi2 = RecoDecay::m(std::array{pVecD2Prong0, pVecD2Prong1, pVecSoftPi}, std::array{massD2Daus[0], massD2Daus[1], MassPiPlus});
648648
auto arrayMomentaDDpi = std::array{pVecD1, pVecD2, pVecSoftPi};
649-
const auto massD0D0Pi = RecoDecay::m(std::move(arrayMomentaDDpi), std::array{MassD0, MassD0, MassPiPlus});
649+
const auto massD0D0Pi = RecoDecay::m(arrayMomentaDDpi, std::array{MassD0, MassD0, MassPiPlus});
650650
const auto deltaMassD0D0Pi = massD0D0Pi - (massD01 + massD02);
651651

652652
deltaMassD01 = massKpipi1 - massD01;

0 commit comments

Comments
 (0)