fix: use the default destructor to drop the member Payload (#419)

(cherry picked from commit 2930620)
ros2 · Jan 15, 2025 · 7356c1c · 7356c1c
1 parent c648ba4
commit 7356c1c
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 6 deletions.
diff --git a/rmw_zenoh_cpp/src/detail/rmw_subscription_data.cpp b/rmw_zenoh_cpp/src/detail/rmw_subscription_data.cpp
@@ -49,11 +49,6 @@ SubscriptionData::Message::Message(
 {
 }
 
-///=============================================================================
-SubscriptionData::Message::~Message()
-{
-}
-
 ///=============================================================================
 std::shared_ptr<SubscriptionData> SubscriptionData::make(
   std::shared_ptr<zenoh::Session> session,

diff --git a/rmw_zenoh_cpp/src/detail/rmw_subscription_data.hpp b/rmw_zenoh_cpp/src/detail/rmw_subscription_data.hpp
@@ -54,7 +54,7 @@ class SubscriptionData final : public std::enable_shared_from_this<SubscriptionD
       uint64_t recv_ts,
       AttachmentData && attachment);
 
-    ~Message();
+    ~Message() = default;
 
     Payload payload;
     uint64_t recv_timestamp;