Fix a couple of warnings pointed out by clang. (#133)

1. Add braces around initialization of array elements. 2. Remove an unused structure member. Signed-off-by: Chris Lalancette <[email protected]>
ros2 · Oct 7, 2023 · c7c6cc7 · c7c6cc7
1 parent 4fba5cc
commit c7c6cc7
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/rmw_connextdds_common/include/rmw_connextdds/context.hpp b/rmw_connextdds_common/include/rmw_connextdds/context.hpp
@@ -176,8 +176,6 @@ struct rmw_context_impl_s
   rmw_ret_t
   finalize_participant();
 
-  DDS_DomainParticipantFactory * factory{nullptr};
-
   /* Manage the memory of the domain tag */
   char * domain_tag{nullptr};
 

diff --git a/rmw_connextdds_common/src/common/rmw_context.cpp b/rmw_connextdds_common/src/common/rmw_context.cpp
@@ -301,8 +301,8 @@ rmw_context_impl_s::initialize_discovery_options(DDS_DomainParticipantQos & dp_q
       // If it's too large then we will send a lot of announcement traffic.
       // The default number here is picked arbitrarily.
       const rmw_peer_address_t localhost_peers[2] = {
-        "[email protected]://127.0.0.1",
-        "[email protected]://",
+        {"[email protected]://127.0.0.1"},
+        {"[email protected]://"},
       };
       const auto rc2 = rmw_connextdds_extend_initial_peer_list(
         localhost_peers,