From: Simon Marchi <simon.marchi@efficios.com>
Date: Thu, 7 Dec 2023 04:55:31 +0000 (+0000)
Subject: src.ctf.lttng-live: add lttng_live_stream_iterator destructor
X-Git-Url: https://git.efficios.com/?p=babeltrace.git;a=commitdiff_plain;h=ce4ee876334689bc631100a219b1546a2e3e0bda

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>
Reviewed-on: https://review.lttng.org/c/babeltrace/+/12375
Tested-by: jenkins <jenkins@lttng.org>
---

diff --git a/src/plugins/ctf/lttng-live/data-stream.cpp b/src/plugins/ctf/lttng-live/data-stream.cpp
index cf9771ae..49915d2c 100644
--- a/src/plugins/ctf/lttng-live/data-stream.cpp
+++ b/src/plugins/ctf/lttng-live/data-stream.cpp
@@ -220,20 +220,19 @@ 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;
 }
 
-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--;
+}
 
+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 56c86d39..2ff02ee6 100644
--- a/src/plugins/ctf/lttng-live/lttng-live.hpp
+++ b/src/plugins/ctf/lttng-live/lttng-live.hpp
@@ -73,6 +73,8 @@ struct lttng_live_stream_iterator
     {
     }
 
+    ~lttng_live_stream_iterator();
+
     bt2c::Logger logger;
 
     bt2::Stream::Shared stream;