fix: decrement_borrowed_publisher_num before calling publish_core (#360)

* minimum shm

Signed-off-by: Koichi Imai <koichi.imai.2@tier4.jp>

* fix for unit test

Signed-off-by: Koichi Imai <koichi.imai.2@tier4.jp>

---------

Signed-off-by: Koichi Imai <koichi.imai.2@tier4.jp>

src/agnocastlib/include/agnocast_publisher.hpp

@@ -90,6 +90,8 @@ class Publisher
       exit(EXIT_FAILURE);
     }
 
+    decrement_borrowed_publisher_num();
+
     const union ioctl_publish_args publish_args = publish_core(
       topic_name_, id_, static_cast<uint32_t>(qos_.depth()),
       reinterpret_cast<uint64_t>(message.get()), opened_mqs_);
@@ -100,12 +102,6 @@ class Publisher
       publish_args.ret_entry_id);
 #endif
 
-    // We need to decrement borrowed_publisher_num before ros2_publish, otherwise the buffers used
-    // for ROS2 serialization will also use shared memory. Since we don't need to store any
-    // additional data in shared memory after the agnocast publish operation, here is the ideal
-    // point to decrement.
-    decrement_borrowed_publisher_num();
-
     for (uint32_t i = 0; i < publish_args.ret_released_num; i++) {
       MessageT * release_ptr = reinterpret_cast<MessageT *>(publish_args.ret_released_addrs[i]);
       delete release_ptr;