lib: use common precond. assert. macros from `assert-cond.h` thru lib

[babeltrace.git] / src / lib / graph / message / event.c

diff --git a/src/lib/graph/message/event.c b/src/lib/graph/message/event.c
index e194fce0b1822b7c35140a46607707f4665169d8..ac567bfa4a954b49bb771bcb11aa2efefb40120e 100644 (file)
--- a/src/lib/graph/message/event.c
+++ b/src/lib/graph/message/event.c
@@ -59,6 +59,50 @@ end:
        return (void *) message;
 }
 
+static
+struct bt_event *create_event(struct bt_event_class *event_class,
+               struct bt_packet *packet, struct bt_stream *stream)
+{
+       struct bt_event *event = NULL;
+
+       BT_ASSERT_DBG(event_class);
+       BT_ASSERT_DBG(stream);
+       event = bt_object_pool_create_object(&event_class->event_pool);
+       if (G_UNLIKELY(!event)) {
+               BT_LIB_LOGE_APPEND_CAUSE(
+                       "Cannot allocate one event from event class's event pool: "
+                       "%![ec-]+E", event_class);
+               goto end;
+       }
+
+       if (G_LIKELY(!event->class)) {
+               event->class = event_class;
+               bt_object_get_ref_no_null_check(&event_class->base);
+       }
+
+       BT_ASSERT_DBG(!event->stream);
+       event->stream = stream;
+       bt_object_get_ref_no_null_check_no_parent_check(&event->stream->base);
+       BT_LIB_LOGD("Set event's stream: %![event-]+e, %![stream-]+s",
+               event, stream);
+
+       if (packet) {
+               BT_ASSERT_PRE_DEV(bt_event_class_borrow_stream_class(
+                       event_class) == packet->stream->class,
+                       "Packet's stream class and event class's stream class differ: "
+                       "%![ec-]+E, %![packet-]+a", event, packet);
+               BT_ASSERT_DBG(event->stream->class->supports_packets);
+               BT_ASSERT_DBG(!event->packet);
+               event->packet = packet;
+               bt_object_get_ref_no_null_check_no_parent_check(&event->packet->base);
+               BT_LIB_LOGD("Set event's packet: %![event-]+e, %![packet-]+a",
+                       event, packet);
+       }
+
+end:
+       return event;
+}
+
 static inline
 struct bt_message *create_event_message(
                struct bt_self_message_iterator *self_msg_iter,
@@ -77,10 +121,14 @@ struct bt_message *create_event_message(
        struct bt_event *event;
 
        BT_ASSERT_DBG(stream);
-       BT_ASSERT_PRE_NON_NULL(msg_iter, "Message iterator");
-       BT_ASSERT_PRE_NON_NULL(event_class, "Event class");
+       BT_ASSERT_PRE_MSG_ITER_NON_NULL(msg_iter);
+       BT_ASSERT_PRE_EC_NON_NULL(event_class);
        BT_ASSERT_PRE(event_class_has_trace(event_class),
                "Event class is not part of a trace: %!+E", event_class);
+       BT_ASSERT_PRE_DEV(bt_event_class_borrow_stream_class(event_class) ==
+               stream->class,
+               "Stream's class and event's stream class differ: "
+               "%![ec-]+E, %![stream-]+s", event_class, stream);
        stream_class = bt_event_class_borrow_stream_class_inline(event_class);
        BT_ASSERT_DBG(stream_class);
        BT_ASSERT_PRE((with_cs && stream_class->default_clock_class) ||
@@ -92,7 +140,7 @@ struct bt_message *create_event_message(
                "cs-val=%" PRIu64,
                event_class, stream_class, with_cs, raw_value);
        BT_LIB_LOGD("Creating event message object: %![ec-]+E", event_class);
-       event = bt_event_create(event_class, packet, stream);
+       event = create_event(event_class, packet, stream);
        if (G_UNLIKELY(!event)) {
                BT_LIB_LOGE_APPEND_CAUSE(
                        "Cannot create event from event class: "
@@ -160,7 +208,7 @@ struct bt_message *bt_message_event_create(
                const struct bt_stream *stream)
 {
        BT_ASSERT_PRE_DEV_NO_ERROR();
-       BT_ASSERT_PRE_NON_NULL(stream, "Stream");
+       BT_ASSERT_PRE_STREAM_NON_NULL(stream);
        return create_event_message(msg_iter, event_class, NULL, stream, false, 0);
 }
 
@@ -170,7 +218,7 @@ struct bt_message *bt_message_event_create_with_packet(
                const struct bt_packet *packet)
 {
        BT_ASSERT_PRE_DEV_NO_ERROR();
-       BT_ASSERT_PRE_NON_NULL(packet, "Packet");
+       BT_ASSERT_PRE_PACKET_NON_NULL(packet);
        return create_event_message(msg_iter, event_class, packet,
                packet->stream, false, 0);
 }
@@ -182,7 +230,7 @@ struct bt_message *bt_message_event_create_with_default_clock_snapshot(
                uint64_t raw_value)
 {
        BT_ASSERT_PRE_DEV_NO_ERROR();
-       BT_ASSERT_PRE_NON_NULL(stream, "Stream");
+       BT_ASSERT_PRE_STREAM_NON_NULL(stream);
        return create_event_message(msg_iter, event_class, NULL, stream,
                true, raw_value);
 }
@@ -195,7 +243,7 @@ bt_message_event_create_with_packet_and_default_clock_snapshot(
                uint64_t raw_value)
 {
        BT_ASSERT_PRE_DEV_NO_ERROR();
-       BT_ASSERT_PRE_NON_NULL(packet, "Packet");
+       BT_ASSERT_PRE_PACKET_NON_NULL(packet);
        return create_event_message(msg_iter, event_class, packet,
                packet->stream, true, raw_value);
 }
@@ -256,8 +304,8 @@ struct bt_event *borrow_event(struct bt_message *message)
 {
        struct bt_message_event *event_message;
 
-       BT_ASSERT_PRE_DEV_NON_NULL(message, "Message");
-       BT_ASSERT_PRE_DEV_MSG_IS_TYPE(message, BT_MESSAGE_TYPE_EVENT);
+       BT_ASSERT_PRE_DEV_MSG_NON_NULL(message);
+       BT_ASSERT_PRE_DEV_MSG_HAS_TYPE(message, BT_MESSAGE_TYPE_EVENT);
        event_message = container_of(message,
                        struct bt_message_event, parent);
        return event_message->event;
@@ -282,14 +330,12 @@ bt_message_event_borrow_default_clock_snapshot_const(
        struct bt_message_event *event_msg = (void *) msg;
        struct bt_stream_class *stream_class;
 
-       BT_ASSERT_PRE_DEV_NON_NULL(msg, "Message");
-       BT_ASSERT_PRE_DEV_MSG_IS_TYPE(msg, BT_MESSAGE_TYPE_EVENT);
+       BT_ASSERT_PRE_DEV_MSG_NON_NULL(msg);
+       BT_ASSERT_PRE_DEV_MSG_HAS_TYPE(msg, BT_MESSAGE_TYPE_EVENT);
        stream_class = bt_event_class_borrow_stream_class_inline(
                event_msg->event->class);
        BT_ASSERT_DBG(stream_class);
-       BT_ASSERT_PRE_DEV(stream_class->default_clock_class,
-               "Message's stream's class has no default clock class: "
-               "%![msg-]+n, %![sc-]+S", msg, stream_class);
+       BT_ASSERT_PRE_DEV_MSG_SC_DEF_CLK_CLS(msg, stream_class);
        return event_msg->default_cs;
 }
 
@@ -300,8 +346,8 @@ bt_message_event_borrow_stream_class_default_clock_class_const(
        struct bt_message_event *event_msg = (void *) msg;
        struct bt_stream_class *stream_class;
 
-       BT_ASSERT_PRE_DEV_NON_NULL(msg, "Message");
-       BT_ASSERT_PRE_DEV_MSG_IS_TYPE(msg, BT_MESSAGE_TYPE_EVENT);
+       BT_ASSERT_PRE_DEV_MSG_NON_NULL(msg);
+       BT_ASSERT_PRE_DEV_MSG_HAS_TYPE(msg, BT_MESSAGE_TYPE_EVENT);
        stream_class = bt_event_class_borrow_stream_class_inline(
                event_msg->event->class);
        BT_ASSERT_DBG(stream_class);