Skip to content

Commit

Permalink
fix: use the default destructor that automatically drops the zenoh re…
Browse files Browse the repository at this point in the history
…ply/query and hence sends the final signal (#473)

(cherry picked from commit 4fc36b7)
  • Loading branch information
YuanYuYuan authored and mergify[bot] committed Feb 20, 2025
1 parent f071960 commit 032a306
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 8 deletions.
6 changes: 0 additions & 6 deletions rmw_zenoh_cpp/src/detail/zenoh_utils.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -53,9 +53,6 @@ std::chrono::nanoseconds::rep ZenohQuery::get_received_timestamp() const
return received_timestamp_;
}

///=============================================================================
ZenohQuery::~ZenohQuery() {}

///=============================================================================
const zenoh::Query & ZenohQuery::get_query() const {return query_;}

Expand All @@ -68,9 +65,6 @@ ZenohReply::ZenohReply(
received_timestamp_ = received_timestamp;
}

///=============================================================================
ZenohReply::~ZenohReply() {}

///=============================================================================
const zenoh::Reply & ZenohReply::get_sample() const
{
Expand Down
4 changes: 2 additions & 2 deletions rmw_zenoh_cpp/src/detail/zenoh_utils.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ class ZenohReply final
public:
ZenohReply(const zenoh::Reply & reply, std::chrono::nanoseconds::rep received_timestamp);

~ZenohReply();
~ZenohReply() = default;

const zenoh::Reply & get_sample() const;

Expand All @@ -58,7 +58,7 @@ class ZenohQuery final
public:
ZenohQuery(const zenoh::Query & query, std::chrono::nanoseconds::rep received_timestamp);

~ZenohQuery();
~ZenohQuery() = default;

const zenoh::Query & get_query() const;

Expand Down

0 comments on commit 032a306

Please # to comment.