earlier executor_thread_.reset() in nav2_costmap_ros (#4385)

* reorder cleanup_queue Signed-off-by: goes <[email protected]> * codestyle Signed-off-by: goes <[email protected]> * codestyle Signed-off-by: goes <[email protected]> --------- Signed-off-by: goes <[email protected]> Co-authored-by: goes <[email protected]>
ros-navigation · May 31, 2024 · d302f1a · d302f1a
1 parent 30e2cde
commit d302f1a
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/nav2_costmap_2d/src/costmap_2d_ros.cpp b/nav2_costmap_2d/src/costmap_2d_ros.cpp
@@ -354,6 +354,7 @@ nav2_util::CallbackReturn
 Costmap2DROS::on_cleanup(const rclcpp_lifecycle::State & /*state*/)
 {
   RCLCPP_INFO(get_logger(), "Cleaning up");
+  executor_thread_.reset();
 
   costmap_publisher_.reset();
   clear_costmap_service_.reset();
@@ -368,8 +369,6 @@ Costmap2DROS::on_cleanup(const rclcpp_lifecycle::State & /*state*/)
   footprint_sub_.reset();
   footprint_pub_.reset();
 
-
-  executor_thread_.reset();
   return nav2_util::CallbackReturn::SUCCESS;
 }