ctf: make ctf_trace_class_configure_ir_trace use bt2::Trace

Use the bt2::Trace wrapper instead of a raw `bt_trace *`.  Change the
return type to void, since failures are now reported using exceptions.

Change-Id: I2e30b0aa9a6f33e765ac32d274032ef4ab3aa9f8
Reviewed-on: https://review.lttng.org/c/babeltrace/+/8417
Reviewed-by: Philippe Proulx <eeppeliteloop@gmail.com>

diff --git a/src/plugins/ctf/common/src/metadata/tsdl/ctf-meta-configure-ir-trace.cpp b/src/plugins/ctf/common/src/metadata/tsdl/ctf-meta-configure-ir-trace.cpp
index fbda3d73fdddf1e05737ca471319948625bbbd0b..c36b95f65a11705d71b5ab066e9452006dfa32f9 100644 (file)
--- a/src/plugins/ctf/common/src/metadata/tsdl/ctf-meta-configure-ir-trace.cpp
+++ b/src/plugins/ctf/common/src/metadata/tsdl/ctf-meta-configure-ir-trace.cpp
@@ -7,18 +7,18 @@
 #include <babeltrace2/babeltrace.h>
 
 #include "ctf-meta-configure-ir-trace.hpp"
+#include "cpp-common/uuid.hpp"
 
 BT_HIDDEN
-int ctf_trace_class_configure_ir_trace(struct ctf_trace_class *tc, bt_trace *ir_trace)
+void ctf_trace_class_configure_ir_trace(struct ctf_trace_class *tc, bt2::Trace ir_trace)
 {
-    int ret = 0;
     uint64_t i;
 
     BT_ASSERT(tc);
-    BT_ASSERT(ir_trace);
 
     if (tc->is_uuid_set) {
-        bt_trace_set_uuid(ir_trace, tc->uuid);
+        bt2_common::Uuid uuid {tc->uuid};
+        ir_trace.uuid(uuid);
     }
 
     for (i = 0; i < tc->env_entries->len; i++) {
@@ -27,22 +27,13 @@ int ctf_trace_class_configure_ir_trace(struct ctf_trace_class *tc, bt_trace *ir_
 
         switch (env_entry->type) {
         case CTF_TRACE_CLASS_ENV_ENTRY_TYPE_INT:
-            ret = bt_trace_set_environment_entry_integer(ir_trace, env_entry->name->str,
-                                                         env_entry->value.i);
+            ir_trace.environmentEntry(env_entry->name->str, env_entry->value.i);
             break;
         case CTF_TRACE_CLASS_ENV_ENTRY_TYPE_STR:
-            ret = bt_trace_set_environment_entry_string(ir_trace, env_entry->name->str,
-                                                        env_entry->value.str->str);
+            ir_trace.environmentEntry(env_entry->name->str, env_entry->value.str->str);
             break;
         default:
             bt_common_abort();
         }
-
-        if (ret) {
-            goto end;
-        }
     }
-
-end:
-    return ret;
 }

diff --git a/src/plugins/ctf/common/src/metadata/tsdl/ctf-meta-configure-ir-trace.hpp b/src/plugins/ctf/common/src/metadata/tsdl/ctf-meta-configure-ir-trace.hpp
index af84fc8aae24f3329dc5c43b4e0d60e3f437d4a2..c7a1faaec5f388eca84b68ca09c5d26470c22645 100644 (file)
--- a/src/plugins/ctf/common/src/metadata/tsdl/ctf-meta-configure-ir-trace.hpp
+++ b/src/plugins/ctf/common/src/metadata/tsdl/ctf-meta-configure-ir-trace.hpp
@@ -10,8 +10,9 @@
 #include "common/macros.h"
 
 #include "ctf-meta.hpp"
+#include "cpp-common/bt2/trace-ir.hpp"
 
 BT_HIDDEN
-int ctf_trace_class_configure_ir_trace(struct ctf_trace_class *tc, bt_trace *ir_trace);
+void ctf_trace_class_configure_ir_trace(struct ctf_trace_class *tc, bt2::Trace ir_trace);
 
 #endif /* _CTF_META_CONFIGURE_IR_TRACE_H */

diff --git a/src/plugins/ctf/fs-src/fs.cpp b/src/plugins/ctf/fs-src/fs.cpp
index 7aee72ca55d7b41f16547079e023f4533c665c28..9e025566e8e3052ae30267c08930c6e1079d8541 100644 (file)
--- a/src/plugins/ctf/fs-src/fs.cpp
+++ b/src/plugins/ctf/fs-src/fs.cpp
@@ -651,11 +651,7 @@ static ctf_fs_trace::UP ctf_fs_trace_create(const char *path, const char *name,
     }
 
     if (ctf_fs_trace->trace) {
-        ret = ctf_trace_class_configure_ir_trace(ctf_fs_trace->metadata->tc,
-                                                 (*ctf_fs_trace->trace)->libObjPtr());
-        if (ret) {
-            return nullptr;
-        }
+        ctf_trace_class_configure_ir_trace(ctf_fs_trace->metadata->tc, **ctf_fs_trace->trace);
 
         ret = set_trace_name((*ctf_fs_trace->trace)->libObjPtr(), name, logCfg);
         if (ret) {

diff --git a/src/plugins/ctf/lttng-live/metadata.cpp b/src/plugins/ctf/lttng-live/metadata.cpp
index fc573e9c46140afc1078af2eb198d21ddac43108..dd1bb3cdd43ab907ac9a50da9bae2130bd4aa4df 100644 (file)
--- a/src/plugins/ctf/lttng-live/metadata.cpp
+++ b/src/plugins/ctf/lttng-live/metadata.cpp
@@ -223,10 +223,9 @@ enum lttng_live_iterator_status lttng_live_metadata_update(struct lttng_live_tra
                 BT_CLOGE_APPEND_CAUSE("Failed to create bt_trace");
                 goto error;
             }
-            if (ctf_trace_class_configure_ir_trace(tc, (*trace->trace)->libObjPtr())) {
-                BT_CLOGE_APPEND_CAUSE("Failed to configure ctf trace class");
-                goto error;
-            }
+
+            ctf_trace_class_configure_ir_trace(tc, **trace->trace);
+
             if (!stream_classes_all_have_default_clock_class((*trace->trace_class)->libObjPtr(),
                                                              logCfg)) {
                 /* Error logged in function. */