Cleanup

Signed-off-by: Yadunund <[email protected]>
ros2 · Dec 21, 2023 · 2461896 · 2461896
1 parent d6ef8ac
commit 2461896
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 19 deletions.
diff --git a/rmw_zenoh_cpp/src/detail/graph_cache.cpp b/rmw_zenoh_cpp/src/detail/graph_cache.cpp
@@ -55,16 +55,6 @@ TopicData::TopicData(
   stats_(std::move(stats))
 {}
 
-///=============================================================================
-// bool operator==(const GraphNodePtr & lhs, const GraphNodePtr & rhs) {
-//   return lhs->id_ == rhs->id_ && lhs->ns_ == rhs->ns_ && lhs->name_ == rhs->name_;
-// }
-
-///=============================================================================
-// bool GraphCache::NodeComparator::operator()(const GraphNodePtr & lhs, const GraphNodePtr & rhs) const {
-//   return std::hash<std::string>{}(lhs->id_) < std::hash<std::string>{}(rhs->id_);
-// }
-
 ///=============================================================================
 void GraphCache::parse_put(const std::string & keyexpr)
 {

diff --git a/rmw_zenoh_cpp/src/detail/graph_cache.hpp b/rmw_zenoh_cpp/src/detail/graph_cache.hpp
@@ -72,9 +72,6 @@ struct GraphNode
   TopicMap subs_ = {};
 };
 using GraphNodePtr = std::shared_ptr<GraphNode>;
-// Two nodes are the same if their id, name and namespaces are identical.
-// TODO(Yadunund): Once zenoh API gives us globally unique IDs for entities, rely on only the id.
-// bool operator==(const GraphNodePtr & lhs, const GraphNodePtr & rhs);
 
 ///=============================================================================
 class GraphCache final
@@ -135,12 +132,6 @@ class GraphCache final
     node_n:
   */
 
-  // // A comparator for sorting nodes that have the same name based on their
-  // // zenoh session ids.
-  // struct NodeComparator {
-  //   bool operator()(const GraphNodePtr & lhs, const GraphNodePtr & rhs) const;
-  // };
-
   // We rely on a multimap to store nodes with duplicate names.
   using NodeMap = std::multimap<std::string, GraphNodePtr>;
   using NamespaceMap = std::unordered_map<std::string, NodeMap>;