Minor optimizations

Signed-off-by: Yadunund <[email protected]>
ros2 · Dec 11, 2024 · ceb542d · ceb542d
1 parent ef5685c
commit ceb542d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/rmw_zenoh_cpp/src/detail/rmw_client_data.cpp b/rmw_zenoh_cpp/src/detail/rmw_client_data.cpp
@@ -42,11 +42,11 @@
 
 namespace
 {
-
+///=============================================================================
 struct ClientDataWrapper
 {
   explicit ClientDataWrapper(std::shared_ptr<rmw_zenoh_cpp::ClientData> data)
-  : client_data(data)
+  : client_data(std::move(data))
   {
   }
 

diff --git a/rmw_zenoh_cpp/src/detail/rmw_node_data.cpp b/rmw_zenoh_cpp/src/detail/rmw_node_data.cpp
@@ -383,10 +383,6 @@ ClientDataPtr NodeData::get_client_data(const rmw_client_t * const client)
 void NodeData::delete_client_data(const rmw_client_t * const client)
 {
   std::lock_guard<std::recursive_mutex> lock_guard(mutex_);
-  auto client_it = clients_.find(client);
-  if (client_it == clients_.end()) {
-    return;
-  }
   clients_.erase(client);
 }