remove redundant check

Francesco Mazzaschi · Francesco Mazzaschi · commit 4ae732a3031b · 2025-09-10T17:30:19.000+02:00
diff --git a/Detectors/AOD/src/AODProducerWorkflowSpec.cxx b/Detectors/AOD/src/AODProducerWorkflowSpec.cxx
@@ -1141,18 +1141,16 @@ void AODProducerWorkflowDPL::fillMCTrackLabelsTable(MCTrackLabelCursorType& mcTr
             }
             if (trackIndex.includesDet(DetID::ITS)) {
               auto itsGID = data.getITSContributorGID(trackIndex);
-              if (itsGID.isIndexSet()) {
-                auto itsSource = itsGID.getSource();
-                if (itsSource == GIndex::ITS) {
-                  auto& itsTrack = data.getITSTrack(itsGID);
-                  for (unsigned int iL = 0; iL < 7; ++iL) {
-                    if (itsTrack.isFakeOnLayer(iL)) {
-                      labelHolder.labelMask |= (0x1 << iL);
-                    }
+              auto itsSource = itsGID.getSource();
+              if (itsSource == GIndex::ITS) {
+                auto& itsTrack = data.getITSTrack(itsGID);
+                for (unsigned int iL = 0; iL < 7; ++iL) {
+                  if (itsTrack.isFakeOnLayer(iL)) {
+                    labelHolder.labelMask |= (0x1 << iL);
                   }
-                } else if (itsSource == GIndex::ITSAB) {
-                  labelHolder.labelMask |= (data.getTrackMCLabel(itsGID).isFake() << 12);
                 }
+              } else if (itsSource == GIndex::ITSAB) {
+                labelHolder.labelMask |= (data.getTrackMCLabel(itsGID).isFake() << 12);
               }
             }
 

Original file line number	Diff line number	Diff line change
`@@ -1141,18 +1141,16 @@ void AODProducerWorkflowDPL::fillMCTrackLabelsTable(MCTrackLabelCursorType& mcTr`
`1141`	`1141`	`}`
`1142`	`1142`	`if (trackIndex.includesDet(DetID::ITS)) {`
`1143`	`1143`	`auto itsGID = data.getITSContributorGID(trackIndex);`
`1144`		`- if (itsGID.isIndexSet()) {`
`1145`		`- auto itsSource = itsGID.getSource();`
`1146`		`- if (itsSource == GIndex::ITS) {`
`1147`		`- auto& itsTrack = data.getITSTrack(itsGID);`
`1148`		`- for (unsigned int iL = 0; iL < 7; ++iL) {`
`1149`		`- if (itsTrack.isFakeOnLayer(iL)) {`
`1150`		`- labelHolder.labelMask \|= (0x1 << iL);`
`1151`		`- }`
	`1144`	`+ auto itsSource = itsGID.getSource();`
	`1145`	`+ if (itsSource == GIndex::ITS) {`
	`1146`	`+ auto& itsTrack = data.getITSTrack(itsGID);`
	`1147`	`+ for (unsigned int iL = 0; iL < 7; ++iL) {`
	`1148`	`+ if (itsTrack.isFakeOnLayer(iL)) {`
	`1149`	`+ labelHolder.labelMask \|= (0x1 << iL);`
`1152`	`1150`	`}`
`1153`		`- } else if (itsSource == GIndex::ITSAB) {`
`1154`		`- labelHolder.labelMask \|= (data.getTrackMCLabel(itsGID).isFake() << 12);`
`1155`	`1151`	`}`
	`1152`	`+ } else if (itsSource == GIndex::ITSAB) {`
	`1153`	`+ labelHolder.labelMask \|= (data.getTrackMCLabel(itsGID).isFake() << 12);`
`1156`	`1154`	`}`
`1157`	`1155`	`}`
`1158`	`1156`