Switch return calls to continue calls when moving TPC only tracks

romainschotter · romainschotter · commit 260d89f71efa · 2025-07-30T12:06:28.000+02:00
diff --git a/PWGLF/TableProducer/Strangeness/strangenessbuilder.cxx b/PWGLF/TableProducer/Strangeness/strangenessbuilder.cxx
@@ -905,15 +905,15 @@ struct StrangenessBuilder {
           posTrackPar.setPID(o2::track::PID::Electron);
           negTrackPar.setPID(o2::track::PID::Electron);
           if (!mVDriftMgr.moveTPCTrack<TBCs, TCollisions>(collision, pTrack, posTrackPar)) {
-            continue;
+	    continue;
           }
         }
         if (isNegTPCOnly) {
           // Nota bene: negative is TPC-only -> this entire V0 merits treatment as photon candidate
           posTrackPar.setPID(o2::track::PID::Electron);
           negTrackPar.setPID(o2::track::PID::Electron);
           if (!mVDriftMgr.moveTPCTrack<TBCs, TCollisions>(collision, nTrack, negTrackPar)) {
-            continue;
+	    continue;
           }
         }
       } // end TPC drift treatment
@@ -1648,7 +1648,8 @@ struct StrangenessBuilder {
 
           auto const& collision = collisions.rawIteratorAt(v0.collisionId);
           if (!mVDriftMgr.moveTPCTrack<TBCs, TCollisions>(collision, posTrack, posTrackPar)) {
-            return;
+	    products.v0dataLink(-1, -1);
+            continue;
           }
         }
 
@@ -1660,7 +1661,8 @@ struct StrangenessBuilder {
 
           auto const& collision = collisions.rawIteratorAt(v0.collisionId);
           if (!mVDriftMgr.moveTPCTrack<TBCs, TCollisions>(collision, negTrack, negTrackPar)) {
-            return;
+	    products.v0dataLink(-1, -1);
+            continue;
           }
         }
       }
diff --git a/PWGLF/Utils/strangenessBuilderModule.h b/PWGLF/Utils/strangenessBuilderModule.h
@@ -894,15 +894,15 @@ class BuilderModule
                 posTrackPar.setPID(o2::track::PID::Electron);
                 negTrackPar.setPID(o2::track::PID::Electron);
                 if (!mVDriftMgr.moveTPCTrack<TBCs, TCollisions>(collision, pTrack, posTrackPar)) {
-                  return;
+                  continue;
                 }
               }
               if (isNegTPCOnly) {
                 // Nota bene: negative is TPC-only -> this entire V0 merits treatment as photon candidate
                 posTrackPar.setPID(o2::track::PID::Electron);
                 negTrackPar.setPID(o2::track::PID::Electron);
                 if (!mVDriftMgr.moveTPCTrack<TBCs, TCollisions>(collision, nTrack, negTrackPar)) {
-                  return;
+                  continue;
                 }
               }
             } // end TPC drift treatment
@@ -1370,7 +1370,8 @@ class BuilderModule
 
           auto const& collision = collisions.rawIteratorAt(v0.collisionId);
           if (!mVDriftMgr.moveTPCTrack<TBCs, TCollisions>(collision, posTrack, posTrackPar)) {
-            return;
+	    products.v0dataLink(-1, -1);
+            continue;
           }
         }
 
@@ -1382,7 +1383,8 @@ class BuilderModule
 
           auto const& collision = collisions.rawIteratorAt(v0.collisionId);
           if (!mVDriftMgr.moveTPCTrack<TBCs, TCollisions>(collision, negTrack, negTrackPar)) {
-            return;
+	    products.v0dataLink(-1, -1);
+            continue;
           }
         }
       }

Original file line number	Diff line number	Diff line change
`@@ -905,15 +905,15 @@ struct StrangenessBuilder {`
`905`	`905`	`posTrackPar.setPID(o2::track::PID::Electron);`
`906`	`906`	`negTrackPar.setPID(o2::track::PID::Electron);`
`907`	`907`	`if (!mVDriftMgr.moveTPCTrack<TBCs, TCollisions>(collision, pTrack, posTrackPar)) {`
`908`		`- continue;`
	`908`	`+ continue;`
`909`	`909`	`}`
`910`	`910`	`}`
`911`	`911`	`if (isNegTPCOnly) {`
`912`	`912`	`// Nota bene: negative is TPC-only -> this entire V0 merits treatment as photon candidate`
`913`	`913`	`posTrackPar.setPID(o2::track::PID::Electron);`
`914`	`914`	`negTrackPar.setPID(o2::track::PID::Electron);`
`915`	`915`	`if (!mVDriftMgr.moveTPCTrack<TBCs, TCollisions>(collision, nTrack, negTrackPar)) {`
`916`		`- continue;`
	`916`	`+ continue;`
`917`	`917`	`}`
`918`	`918`	`}`
`919`	`919`	`} // end TPC drift treatment`
`@@ -1648,7 +1648,8 @@ struct StrangenessBuilder {`
`1648`	`1648`
`1649`	`1649`	`auto const& collision = collisions.rawIteratorAt(v0.collisionId);`
`1650`	`1650`	`if (!mVDriftMgr.moveTPCTrack<TBCs, TCollisions>(collision, posTrack, posTrackPar)) {`
`1651`		`- return;`
	`1651`	`+ products.v0dataLink(-1, -1);`
	`1652`	`+ continue;`
`1652`	`1653`	`}`
`1653`	`1654`	`}`
`1654`	`1655`
`@@ -1660,7 +1661,8 @@ struct StrangenessBuilder {`
`1660`	`1661`
`1661`	`1662`	`auto const& collision = collisions.rawIteratorAt(v0.collisionId);`
`1662`	`1663`	`if (!mVDriftMgr.moveTPCTrack<TBCs, TCollisions>(collision, negTrack, negTrackPar)) {`
`1663`		`- return;`
	`1664`	`+ products.v0dataLink(-1, -1);`
	`1665`	`+ continue;`
`1664`	`1666`	`}`
`1665`	`1667`	`}`
`1666`	`1668`	`}`
Original file line number	Diff line number	Diff line change
`@@ -894,15 +894,15 @@ class BuilderModule`
`894`	`894`	`posTrackPar.setPID(o2::track::PID::Electron);`
`895`	`895`	`negTrackPar.setPID(o2::track::PID::Electron);`
`896`	`896`	`if (!mVDriftMgr.moveTPCTrack<TBCs, TCollisions>(collision, pTrack, posTrackPar)) {`
`897`		`- return;`
	`897`	`+ continue;`
`898`	`898`	`}`
`899`	`899`	`}`
`900`	`900`	`if (isNegTPCOnly) {`
`901`	`901`	`// Nota bene: negative is TPC-only -> this entire V0 merits treatment as photon candidate`
`902`	`902`	`posTrackPar.setPID(o2::track::PID::Electron);`
`903`	`903`	`negTrackPar.setPID(o2::track::PID::Electron);`
`904`	`904`	`if (!mVDriftMgr.moveTPCTrack<TBCs, TCollisions>(collision, nTrack, negTrackPar)) {`
`905`		`- return;`
	`905`	`+ continue;`
`906`	`906`	`}`
`907`	`907`	`}`
`908`	`908`	`} // end TPC drift treatment`
`@@ -1370,7 +1370,8 @@ class BuilderModule`
`1370`	`1370`
`1371`	`1371`	`auto const& collision = collisions.rawIteratorAt(v0.collisionId);`
`1372`	`1372`	`if (!mVDriftMgr.moveTPCTrack<TBCs, TCollisions>(collision, posTrack, posTrackPar)) {`
`1373`		`- return;`
	`1373`	`+ products.v0dataLink(-1, -1);`
	`1374`	`+ continue;`
`1374`	`1375`	`}`
`1375`	`1376`	`}`
`1376`	`1377`
`@@ -1382,7 +1383,8 @@ class BuilderModule`
`1382`	`1383`
`1383`	`1384`	`auto const& collision = collisions.rawIteratorAt(v0.collisionId);`
`1384`	`1385`	`if (!mVDriftMgr.moveTPCTrack<TBCs, TCollisions>(collision, negTrack, negTrackPar)) {`
`1385`		`- return;`
	`1386`	`+ products.v0dataLink(-1, -1);`
	`1387`	`+ continue;`
`1386`	`1388`	`}`
`1387`	`1389`	`}`
`1388`	`1390`	`}`