Merge pull request #25 from dkargin/more_checks

More checks when connection is dropped
dkargin · Jan 9, 2025 · 2ea66b3 · 2ea66b3
2 parents 5c3427e + d614009
commit 2ea66b3
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 3 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -6,7 +6,7 @@ cmake_minimum_required(VERSION 3.13)
 set(CMAKE_POLICY_DEFAULT_CMP0077 NEW)
 cmake_policy(SET CMP0077 NEW)
 
-project(miniros VERSION 0.4)
+project(miniros VERSION 0.4.1)
 
 include(CheckIncludeFile)
 include(CheckIncludeFiles)

diff --git a/include/miniros/transport/poll_manager.h b/include/miniros/transport/poll_manager.h
@@ -28,6 +28,7 @@
 #ifndef ROSCPP_POLL_MANAGER_H
 #define ROSCPP_POLL_MANAGER_H
 
+#include "miniros/macros.h"
 #include "miniros/internal/forwards.h"
 #include "miniros/internal/observer.h"
 

diff --git a/package.xml b/package.xml
@@ -1,6 +1,6 @@
 <package>
   <name>miniros</name>
-  <version>0.3.0</version>
+  <version>0.4.1</version>
   <description>
     ROS CXX contains a stripped version of ROS C++ code.
     It should become a standalone ROS client without any complex external dependencies.

diff --git a/src/transport/connection.cpp b/src/transport/connection.cpp
@@ -358,7 +358,7 @@ void Connection::drop(DropReason reason)
         for (auto it = drop_watchers_.begin(); it != drop_watchers_.end(); it++)
         {
             if (!it)
-              continue;
+              break;
             it->onConnectionDropped(shared_from_this(), reason);
         }
     }