ros2 · ahcorde · Jan 9, 2025 · Jan 9, 2025
diff --git a/rmw_zenoh_cpp/config/DEFAULT_RMW_ZENOH_ROUTER_CONFIG.json5 b/rmw_zenoh_cpp/config/DEFAULT_RMW_ZENOH_ROUTER_CONFIG.json5
@@ -447,8 +447,8 @@
         connect_certificate: null,
         // Whether or not to verify the matching between hostname/dns and certificate when connecting,
         // if set to false zenoh will disregard the common names of the certificates when verifying servers.
-        // This could be dangerous because your CA can have signed a server cert for foo.com, that's later being used to host a server at baz.com. If you wan't your
-        // ca to verify that the server at baz.com is actually baz.com, let this be true (default).
+        // This could be dangerous because your CA can have signed a server cert for foo.com, that's later being used to host a server at baz.com.
+        // If you want your ca to verify that the server at baz.com is actually baz.com, let this be true (default).
         verify_name_on_connect: true,
       },
     },

diff --git a/rmw_zenoh_cpp/config/DEFAULT_RMW_ZENOH_SESSION_CONFIG.json5 b/rmw_zenoh_cpp/config/DEFAULT_RMW_ZENOH_SESSION_CONFIG.json5
@@ -450,8 +450,8 @@
         connect_certificate: null,
         // Whether or not to verify the matching between hostname/dns and certificate when connecting,
         // if set to false zenoh will disregard the common names of the certificates when verifying servers.
-        // This could be dangerous because your CA can have signed a server cert for foo.com, that's later being used to host a server at baz.com. If you wan't your
-        // ca to verify that the server at baz.com is actually baz.com, let this be true (default).
+        // This could be dangerous because your CA can have signed a server cert for foo.com, that's later being used to host a server at baz.com.
+        // If you want your ca to verify that the server at baz.com is actually baz.com, let this be true (default).
         verify_name_on_connect: true,
       },
     },

diff --git a/rmw_zenoh_cpp/src/detail/graph_cache.cpp b/rmw_zenoh_cpp/src/detail/graph_cache.cpp
@@ -356,7 +356,7 @@ void GraphCache::parse_put(
     range.first, range.second,
     [entity](const std::pair<std::string, GraphNodePtr> & node_it)
     {
-      // Match nodes if their zenoh sesion and node ids match.
+      // Match nodes if their zenoh session and node ids match.
       return entity->zid() == node_it.second->zid_ && entity->nid() == node_it.second->nid_;
     });
   if (node_it == range.second) {
@@ -573,7 +573,7 @@ void GraphCache::parse_del(
     range.first, range.second,
     [entity](const std::pair<std::string, GraphNodePtr> & node_it)
     {
-      // Match nodes if their zenoh sesion and node ids match.
+      // Match nodes if their zenoh session and node ids match.
       return entity->zid() == node_it.second->zid_ && entity->nid() == node_it.second->nid_;
     });
   if (node_it == range.second) {

diff --git a/rmw_zenoh_cpp/src/detail/graph_cache.hpp b/rmw_zenoh_cpp/src/detail/graph_cache.hpp
@@ -91,7 +91,7 @@ struct GraphNode
   TopicMap pubs_ = {};
   TopicMap subs_ = {};
 
-  // Entires for service/client.
+  // Entries for service/client.
   TopicMap clients_ = {};
   TopicMap services_ = {};
 };

diff --git a/rmw_zenoh_cpp/src/detail/liveliness_utils.hpp b/rmw_zenoh_cpp/src/detail/liveliness_utils.hpp
@@ -65,7 +65,7 @@ struct TopicInfo
 };
 
 ///=============================================================================
-/// Retuns the keyexpr for liveliness subscription.
+/// Returns the keyexpr for liveliness subscription.
 std::string subscription_token(size_t domain_id);
 
 ///=============================================================================
@@ -96,7 +96,7 @@ enum class EntityType : uint8_t
  *  <namespace> - The ROS namespace for this entity.  If the namespace is absolute, this function will add in an _ for later parsing reasons.
  *  <nodename> - The ROS node name for this entity.
  *
- * For entities with topic infomation, the liveliness token keyexpr have additional fields:
+ * For entities with topic information, the liveliness token keyexpr have additional fields:
  *
  * <ADMIN_SPACE>/<domainid>/<zid>/<id>/<entity>/<namespace>/<nodename>/<topic_name>/<topic_type>/<topic_type_hash>/<topic_qos>
  *  <topic_name> - The ROS topic name for this entity.

diff --git a/rmw_zenoh_cpp/src/detail/ordered_hash.hpp b/rmw_zenoh_cpp/src/detail/ordered_hash.hpp
@@ -1457,7 +1457,7 @@ class ordered_hash : private Hash, private KeyEqual {
 #endif
 
     /*
-     * The insertion didn't happend at the end of the m_values container,
+     * The insertion didn't happened at the end of the m_values container,
      * we need to shift the indexes in m_buckets_data.
      */
     if (index_insert_position != m_values.size() - 1) {
@@ -1661,7 +1661,7 @@ class ordered_hash : private Hash, private KeyEqual {
   static constexpr float REHASH_ON_HIGH_NB_PROBES__MIN_LOAD_FACTOR = 0.15f;
 
   /**
-   * Protocol version currenlty used for serialization.
+   * Protocol version currently used for serialization.
    */
   static const slz_size_type SERIALIZATION_PROTOCOL_VERSION = 1;
 

diff --git a/rmw_zenoh_cpp/src/detail/rmw_wait_set_data.hpp b/rmw_zenoh_cpp/src/detail/rmw_wait_set_data.hpp
@@ -29,7 +29,7 @@ struct rmw_wait_set_data_t
   // there isn't a race in rmw_wait().  That could happen because of the following sequence:
   //
   // 1. Without taking a lock, rmw_wait() checks if any of the entities in the wait_set are ready,
-  //    and if not attachs the condition_variable to it.
+  //    and if not attaches the condition_variable to it.
   // 2. It then takes the lock, and sleeps on the condition_variable.
   //
   // However, doing step 1 takes time, and checks in a particular order: guard_conditions, events,

diff --git a/rmw_zenoh_cpp/src/detail/zenoh_config.hpp b/rmw_zenoh_cpp/src/detail/zenoh_config.hpp
@@ -57,7 +57,7 @@ std::optional<zenoh::Config> get_z_config(const ConfigurableEntity & entity);
 /// @details The behavior is as follows:
 ///   - If not set or 0, the max value is returned.
 ///   - If less than 0, std::nullopt is returned.
-///   - Else value of environemnt variable is returned.
+///   - Else value of environment variable is returned.
 /// @return The number of times to try connecting to a zenoh router and
 ///   std::nullopt if establishing a connection to a router is not required.
 std::optional<uint64_t> zenoh_router_check_attempts();

diff --git a/rmw_zenoh_cpp/src/rmw_zenoh.cpp b/rmw_zenoh_cpp/src/rmw_zenoh.cpp
@@ -1199,12 +1199,12 @@ rmw_take_sequence(
   }
 
   if (count > message_sequence->capacity) {
-    RMW_SET_ERROR_MSG("Insuffient capacity in message_sequence");
+    RMW_SET_ERROR_MSG("Insufficient capacity in message_sequence");
     return RMW_RET_INVALID_ARGUMENT;
   }
 
   if (count > message_info_sequence->capacity) {
-    RMW_SET_ERROR_MSG("Insuffient capacity in message_info_sequence");
+    RMW_SET_ERROR_MSG("Insufficient capacity in message_info_sequence");
     return RMW_RET_INVALID_ARGUMENT;
   }
 
@@ -2518,7 +2518,7 @@ rmw_service_server_is_available(
     static_cast<rmw_zenoh_cpp::ClientData *>(client->data);
   if (client_data == nullptr) {
     RMW_SET_ERROR_MSG_WITH_FORMAT_STRING(
-      "Unable to retreive client_data from client for service %s", client->service_name);
+      "Unable to retrieve client_data from client for service %s", client->service_name);
     return RMW_RET_INVALID_ARGUMENT;
   }