use cv::DestroyAllWindows (#863)

This ports #816 to ROS 2 and prevents weird exit conditions if you already closed the window
ros-perception · Jan 19, 2024 · 750db49 · 750db49
1 parent 675ea0b
commit 750db49
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/image_view/src/disparity_view_node.cpp b/image_view/src/disparity_view_node.cpp
@@ -88,7 +88,7 @@ DisparityViewNode::DisparityViewNode(const rclcpp::NodeOptions & options)
 
 DisparityViewNode::~DisparityViewNode()
 {
-  cv::destroyWindow(window_name_);
+  cv::destroyAllWindows();
 }
 
 void DisparityViewNode::imageCb(const stereo_msgs::msg::DisparityImage::SharedPtr msg)

diff --git a/image_view/src/image_view_node.cpp b/image_view/src/image_view_node.cpp
@@ -259,7 +259,7 @@ void ImageViewNode::windowThread()
     }
   }
 
-  cv::destroyWindow(window_name_);
+  cv::destroyAllWindows();
 
   if (rclcpp::ok()) {
     rclcpp::shutdown();