Add TODO comment explaining issue #323

naturerobots · Jul 27, 2023 · d46ce9e · d46ce9e
1 parent 9c65437
commit d46ce9e
Showing 1 changed file with 1 addition and 0 deletions.
diff --git a/mbf_abstract_nav/include/mbf_abstract_nav/abstract_action_base.hpp b/mbf_abstract_nav/include/mbf_abstract_nav/abstract_action_base.hpp
@@ -143,6 +143,7 @@ class AbstractActionBase
         // if there is already a plugin running on the same slot, cancel it
         slot_it->second.execution->cancel();
 
+        // TODO + WARNING: this will block the main thread for an arbitrary time during which we won't execute callbacks
         if (slot_it->second.thread_ptr->joinable()) {
           slot_it->second.thread_ptr->join();
         }