Skip to content

Commit 148cbd0

Browse files
authored
[PWGEM] Fix readability-container-size-empty errors (#17071)
1 parent 300b9b3 commit 148cbd0

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

PWGEM/PhotonMeson/TableProducer/skimmerGammaCalo.cxx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -258,7 +258,7 @@ struct SkimmerGammaCalo {
258258
convertForStorage<int16_t>(emccluster.m02(), Observable::kM02),
259259
convertForStorage<int16_t>(emccluster.time(), Observable::kTime));
260260

261-
if (vEta.size() > 0) {
261+
if (!vEta.empty()) {
262262
for (size_t iPart = 0; iPart < vEta.size(); ++iPart) {
263263
tableEmEmcMTracks(tableEmEmcClusters.lastIndex(), vEta[iPart], vPhi[iPart], vP[iPart], vPt[iPart]);
264264
tableMinMTracks(tableMinClusters.lastIndex(),
@@ -268,7 +268,7 @@ struct SkimmerGammaCalo {
268268
convertForStorage<uint16_t>(vPt[iPart], Observable::kEnergy));
269269
}
270270
}
271-
if (vEtaSecondaries.size() > 0) {
271+
if (!vEtaSecondaries.empty()) {
272272
for (size_t iPart = 0; iPart < vEtaSecondaries.size(); ++iPart) {
273273
tableEmEmcMSTracks(tableEmEmcClusters.lastIndex(), vEtaSecondaries[iPart], vPhiSecondaries[iPart], vPSecondaries[iPart], vPtSecondaries[iPart]);
274274
tableMinMSTracks(tableMinClusters.lastIndex(),

PWGEM/PhotonMeson/TableProducer/skimmerPrimaryElectronFromDalitzEE.cxx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -510,7 +510,7 @@ struct SkimmerPrimaryElectronFromDalitzEE {
510510
fillPairInfo<false, 2>(collision, negTracksPerColl, negTracksPerColl); // LS--
511511
}
512512

513-
if ((v0photonsPerColl.size() >= 1 && acceptedPosTrackIdsPerCollision.size() >= 1 && acceptedNegTrackIdsPerCollision.size() >= 1) || (acceptedPosTrackIdsPerCollision.size() >= 2 && acceptedNegTrackIdsPerCollision.size() >= 2)) { // o2-linter: disable=magic-number (check to see if we have enough particles that it make sense to fill the tables)
513+
if ((v0photonsPerColl.size() >= 1 && !acceptedPosTrackIdsPerCollision.empty() && !acceptedNegTrackIdsPerCollision.empty()) || (acceptedPosTrackIdsPerCollision.size() >= 2 && acceptedNegTrackIdsPerCollision.size() >= 2)) { // o2-linter: disable=magic-number (check to see if we have enough particles that it make sense to fill the tables)
514514
// LOGF(info, "v0photonsPerColl.size() = %d, acceptedPosTrackIdsPerCollision.size() = %d, acceptedNegTrackIdsPerCollision.size() = %d", v0photonsPerColl.size(), acceptedPosTrackIdsPerCollision.size(), acceptedNegTrackIdsPerCollision.size());
515515
for (const auto& posId : acceptedPosTrackIdsPerCollision) {
516516
const auto& pos = tracks.rawIteratorAt(posId);
@@ -612,7 +612,7 @@ struct SkimmerPrimaryElectronFromDalitzEE {
612612
fillPairInfo<true, 1>(collision, posTracksPerColl, posTracksPerColl); // LS++
613613
fillPairInfo<true, 2>(collision, negTracksPerColl, negTracksPerColl); // LS--
614614
}
615-
if ((v0photonsPerColl.size() >= 1 && acceptedPosTrackIdsPerCollision.size() >= 1 && acceptedNegTrackIdsPerCollision.size() >= 1) || (acceptedPosTrackIdsPerCollision.size() >= 2 && acceptedNegTrackIdsPerCollision.size() >= 2)) { // o2-linter: disable=magic-number (check to see if we have enough particles that it make sense to fill the tables)
615+
if ((v0photonsPerColl.size() >= 1 && !acceptedPosTrackIdsPerCollision.empty() && !acceptedNegTrackIdsPerCollision.empty()) || (acceptedPosTrackIdsPerCollision.size() >= 2 && acceptedNegTrackIdsPerCollision.size() >= 2)) { // o2-linter: disable=magic-number (check to see if we have enough particles that it make sense to fill the tables)
616616
// LOGF(info, "v0photonsPerColl.size() = %d, acceptedPosTrackIdsPerCollision.size() = %d, acceptedNegTrackIdsPerCollision.size() = %d", v0photonsPerColl.size(), acceptedPosTrackIdsPerCollision.size(), acceptedNegTrackIdsPerCollision.size());
617617
for (const auto& posId : acceptedPosTrackIdsPerCollision) {
618618
const auto& pos = tracks.rawIteratorAt(posId);

0 commit comments

Comments
 (0)