refactor(multi_object_tracker): use const reference in loop and simpl…

…ify tracker update logic Signed-off-by: Taekjin LEE <[email protected]>
autowarefoundation · Feb 28, 2025 · f8caa05 · f8caa05
1 parent 745df6c
commit f8caa05
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/perception/autoware_multi_object_tracker/src/debugger/debug_object.cpp b/perception/autoware_multi_object_tracker/src/debugger/debug_object.cpp
@@ -140,7 +140,7 @@ void TrackerObjectDebugger::process()
   if (object_data_list_.empty()) return;
 
   // update uuid_int
-  for (auto & object_data : object_data_list_) {
+  for (const auto & object_data : object_data_list_) {
     current_ids_.insert(uuidToInt(object_data.uuid));
   }
 

diff --git a/perception/autoware_multi_object_tracker/src/processor/processor.cpp b/perception/autoware_multi_object_tracker/src/processor/processor.cpp
@@ -111,7 +111,7 @@ void TrackerProcessor::spawn(
       new_object.channel_index, new_object.existence_probability);
 
     // Update the tracker with the new object
-    if (tracker) list_tracker_.push_back(tracker);
+    list_tracker_.push_back(tracker);
   }
 }