From 49d4a51abf15a884489706e6b46d36fc2daff5b8 Mon Sep 17 00:00:00 2001 From: Ivan Santiago Paunovic Date: Thu, 20 Feb 2020 14:11:35 -0300 Subject: [PATCH] Update usage of rmw_topic_endpoint_info_array (#101) Signed-off-by: Ivan Santiago Paunovic --- rmw_cyclonedds_cpp/src/rmw_node.cpp | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/rmw_cyclonedds_cpp/src/rmw_node.cpp b/rmw_cyclonedds_cpp/src/rmw_node.cpp index 2f5f030..1f8797e 100644 --- a/rmw_cyclonedds_cpp/src/rmw_node.cpp +++ b/rmw_cyclonedds_cpp/src/rmw_node.cpp @@ -3108,7 +3108,6 @@ get_endpoint_info_by_topic( err_msg.str); return ret; } - endpoint_info->count = 0; node_cache_t node_cache; dds_entity_t ppant_rd; @@ -3120,25 +3119,23 @@ get_endpoint_info_by_topic( handle_topic_endpoint_info_array_fini(endpoint_info, allocator); return RMW_RET_ERROR; } + size_t i = 0; for (const auto & epi : endpoint_info_vector) { - endpoint_info->info_array[endpoint_info->count] = - rmw_get_zero_initialized_topic_endpoint_info(); if ((ret = set_rmw_topic_endpoint_info( node_cache, ppant_rd, allocator, epi, is_publisher, - endpoint_info->info_array[endpoint_info->count])) != RMW_RET_OK) + endpoint_info->info_array[i])) != RMW_RET_OK) { - endpoint_info->count++; handle_topic_endpoint_info_array_fini(endpoint_info, allocator); dds_delete(ppant_rd); return ret; } - endpoint_info->count++; + i++; } - assert(endpoint_info->count == epi_count); + assert(i == epi_count); dds_delete(ppant_rd); return RMW_RET_OK; }