From 2ba831b6b7d37a25c57b08c4b60e41d2b9cf27a0 Mon Sep 17 00:00:00 2001
From: Dominik Smith <d.smith@gsi.de>
Date: Wed, 28 Aug 2024 17:39:14 +0200
Subject: [PATCH] Fixed branching logic in
 ca::TrackFinderWindow::DoCompetitionLoop().

---
 algo/ca/core/tracking/CaTrackFinderWindow.cxx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/algo/ca/core/tracking/CaTrackFinderWindow.cxx b/algo/ca/core/tracking/CaTrackFinderWindow.cxx
index 7b8e04d7de..ed2b455cd4 100644
--- a/algo/ca/core/tracking/CaTrackFinderWindow.cxx
+++ b/algo/ca/core/tracking/CaTrackFinderWindow.cxx
@@ -540,7 +540,7 @@ void TrackFinderWindow::DoCompetitionLoop(const WindowData& wData)
         auto updateStrip = [&](int& strip) {
           if ((strip >= 0) && (strip != tr.Id())) {  // strip is used by other candidate
             const auto& other = fvTrackCandidates[strip];
-            if (!other.IsAlive() && tr.IsBetterThan(other)) {  // D.S. 29.7.24: should this be || instead of && ?
+            if (!other.IsAlive() || tr.IsBetterThan(other)) {
               strip = tr.Id();
             }
             else {
-- 
GitLab