src.ctf.lttng-live: add lttng_live_stream_iterator destructor

lttng_live_stream_iterator_destroy is still used for a GPtrArray, so it
is kept as a wrapper to the destructor for now.

Change-Id: Idc3fb90f2b15f44d92d9edd578783f4939474fdf
Signed-off-by: Simon Marchi <simon.marchi@efficios.com>
Reviewed-on: https://review.lttng.org/c/babeltrace/+/8459
Reviewed-by: Philippe Proulx <eeppeliteloop@gmail.com>

diff --git a/src/plugins/ctf/lttng-live/data-stream.cpp b/src/plugins/ctf/lttng-live/data-stream.cpp
index ab0de6047723f8a7e8bed368ad371c44f6120268..64407f072e65a190c56eb375ead1a48a60458d18 100644 (file)
--- a/src/plugins/ctf/lttng-live/data-stream.cpp
+++ b/src/plugins/ctf/lttng-live/data-stream.cpp
@@ -225,21 +225,20 @@ lttng_live_stream_iterator_create(struct lttng_live_session *session, uint64_t c
 
     goto end;
 error:
-    lttng_live_stream_iterator_destroy(stream_iter);
+    delete stream_iter;
     stream_iter = NULL;
 end:
     return stream_iter;
 }
 
-BT_HIDDEN
-void lttng_live_stream_iterator_destroy(struct lttng_live_stream_iterator *stream_iter)
+lttng_live_stream_iterator::~lttng_live_stream_iterator()
 {
-    if (!stream_iter) {
-        return;
-    }
-
     /* Track the number of active stream iterator. */
-    stream_iter->trace->session->lttng_live_msg_iter->active_stream_iter--;
+    this->trace->session->lttng_live_msg_iter->active_stream_iter--;
+}
 
+BT_HIDDEN
+void lttng_live_stream_iterator_destroy(struct lttng_live_stream_iterator *stream_iter)
+{
     delete stream_iter;
 }

diff --git a/src/plugins/ctf/lttng-live/lttng-live.hpp b/src/plugins/ctf/lttng-live/lttng-live.hpp
index 928ec5b0030ef6df627893a546ec1c4ebe512b30..cdfcb99e928fb6d5ad1eeaa6f702e7f416e30edd 100644 (file)
--- a/src/plugins/ctf/lttng-live/lttng-live.hpp
+++ b/src/plugins/ctf/lttng-live/lttng-live.hpp
@@ -56,6 +56,8 @@ struct lttng_live_stream_iterator
     {
     }
 
+    ~lttng_live_stream_iterator();
+
     const bt2_common::LogCfg logCfg;
 
     /* Owned by this. */