Skip to content

Commit

Permalink
fix lint
Browse files Browse the repository at this point in the history
  • Loading branch information
JEnoch committed Sep 23, 2024
1 parent eb0c327 commit 5f57e6f
Showing 1 changed file with 7 additions and 5 deletions.
12 changes: 7 additions & 5 deletions rmw_zenoh_cpp/src/detail/graph_cache.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -595,8 +595,9 @@ void GraphCache::parse_del(
return entity->zid() == node_it.second->zid_ && entity->nid() == node_it.second->nid_;
});
if (node_it == range.second) {
// Node does not exist or its liveliness token has been unregistered before one of its pubs/subs/service liveliness token.
// This could happen since Zenoh doesn't guarantee any order for unregistration events if the remote Node closed abruptly or was disconnected.
// Node does not exist or its liveliness token has been unregistered before one of its
// pubs/subs/service liveliness token. This could happen since Zenoh doesn't guarantee
// any order for unregistration events if the remote Node closed abruptly or was disconnected.
RMW_ZENOH_LOG_DEBUG_NAMED(
"rmw_zenoh_cpp",
"Received liveliness token to remove unknown node /%s from the graph. Ignoring...",
Expand All @@ -607,9 +608,10 @@ void GraphCache::parse_del(

if (entity->type() == EntityType::Node) {
// Node
// In case the remote Node closed abruptly or was disconnected, Zenoh could deliver the liveliness tokens unregistration events
// in any order. If the event for Node unregistration comes before the unregistration of its pubs/subs/services,
// we should update the count in graph_topics_ and graph_services_.
// In case the remote Node closed abruptly or was disconnected, Zenoh could deliver the
// liveliness tokens unregistration events in any order.
// If the event for Node unregistration comes before the unregistration of its
// pubs/subs/services, we should update the count in graph_topics_ and graph_services_.
const GraphNodePtr graph_node = node_it->second;
if (!graph_node->pubs_.empty() ||
!graph_node->subs_.empty() ||
Expand Down

0 comments on commit 5f57e6f

Please sign in to comment.