lib: make public reference count functions have strict types
[babeltrace.git] / lib / trace-ir / stream-class.c
index 3d8d9c37d6a7a6c81a9c9141e355566d8a3b4cb8..b032b8ee6f2f2a704a2033d22fad80d1b2318a63 100644 (file)
@@ -1,8 +1,7 @@
 /*
+ * Copyright 2017-2018 Philippe Proulx <pproulx@efficios.com>
  * Copyright 2013, 2014 Jérémie Galarneau <jeremie.galarneau@efficios.com>
  *
- * Author: Jérémie Galarneau <jeremie.galarneau@efficios.com>
- *
  * Permission is hereby granted, free of charge, to any person obtaining a copy
  * of this software and associated documentation files (the "Software"), to deal
  * in the Software without restriction, including without limitation the rights
 #include <babeltrace/assert-pre-internal.h>
 #include <babeltrace/trace-ir/clock-class-internal.h>
 #include <babeltrace/trace-ir/event-class-internal.h>
-#include <babeltrace/trace-ir/field-classes-internal.h>
-#include <babeltrace/trace-ir/fields-internal.h>
+#include <babeltrace/trace-ir/field-class-internal.h>
+#include <babeltrace/trace-ir/field-internal.h>
 #include <babeltrace/trace-ir/stream-class-internal.h>
 #include <babeltrace/trace-ir/trace-const.h>
 #include <babeltrace/trace-ir/trace-internal.h>
 #include <babeltrace/trace-ir/utils-internal.h>
 #include <babeltrace/trace-ir/field-wrapper-internal.h>
 #include <babeltrace/trace-ir/resolve-field-path-internal.h>
-#include <babeltrace/object.h>
 #include <babeltrace/compiler-internal.h>
 #include <babeltrace/align-internal.h>
 #include <babeltrace/endian-internal.h>
@@ -70,11 +68,11 @@ void destroy_stream_class(struct bt_object *obj)
                stream_class->name.value = NULL;
        }
 
-       BT_LOGD_STR("Putting event header field classe.");
+       BT_LOGD_STR("Putting event header field class.");
        BT_OBJECT_PUT_REF_AND_RESET(stream_class->event_header_fc);
-       BT_LOGD_STR("Putting packet context field classe.");
+       BT_LOGD_STR("Putting packet context field class.");
        BT_OBJECT_PUT_REF_AND_RESET(stream_class->packet_context_fc);
-       BT_LOGD_STR("Putting event common context field classe.");
+       BT_LOGD_STR("Putting event common context field class.");
        BT_OBJECT_PUT_REF_AND_RESET(stream_class->event_common_context_fc);
        bt_object_pool_finalize(&stream_class->event_header_field_pool);
        bt_object_pool_finalize(&stream_class->packet_context_field_pool);
@@ -90,14 +88,14 @@ void free_field_wrapper(struct bt_field_wrapper *field_wrapper,
 
 BT_ASSERT_PRE_FUNC
 static
-bool stream_class_id_is_unique(const struct bt_trace *trace, uint64_t id)
+bool stream_class_id_is_unique(const struct bt_trace_class *tc, uint64_t id)
 {
        uint64_t i;
        bool is_unique = true;
 
-       for (i = 0; i < trace->stream_classes->len; i++) {
+       for (i = 0; i < tc->stream_classes->len; i++) {
                const struct bt_stream_class *sc =
-                       trace->stream_classes->pdata[i];
+                       tc->stream_classes->pdata[i];
 
                if (sc->id == id) {
                        is_unique = false;
@@ -110,18 +108,17 @@ end:
 }
 
 static
-struct bt_stream_class *create_stream_class_with_id(struct bt_trace *trace,
-               uint64_t id)
+struct bt_stream_class *create_stream_class_with_id(
+               struct bt_trace_class *tc, uint64_t id)
 {
        struct bt_stream_class *stream_class = NULL;
        int ret;
 
-       BT_ASSERT(trace);
-       BT_ASSERT_PRE(stream_class_id_is_unique(trace, id),
-               "Duplicate stream class ID: %![trace-]+t, id=%" PRIu64,
-               trace, id);
-       BT_LIB_LOGD("Creating stream class object: %![trace-]+t, id=%" PRIu64,
-               trace, id);
+       BT_ASSERT(tc);
+       BT_ASSERT_PRE(stream_class_id_is_unique(tc, id),
+               "Duplicate stream class ID: %![tc-]+T, id=%" PRIu64, tc, id);
+       BT_LIB_LOGD("Creating stream class object: %![tc-]+T, id=%" PRIu64,
+               tc, id);
        stream_class = g_new0(struct bt_stream_class, 1);
        if (!stream_class) {
                BT_LOGE_STR("Failed to allocate one stream class.");
@@ -168,9 +165,9 @@ struct bt_stream_class *create_stream_class_with_id(struct bt_trace *trace,
                goto error;
        }
 
-       bt_object_set_parent(&stream_class->base, &trace->base);
-       g_ptr_array_add(trace->stream_classes, stream_class);
-       bt_trace_freeze(trace);
+       bt_object_set_parent(&stream_class->base, &tc->base);
+       g_ptr_array_add(tc->stream_classes, stream_class);
+       bt_trace_class_freeze(tc);
        BT_LIB_LOGD("Created stream class object: %!+S", stream_class);
        goto end;
 
@@ -181,37 +178,37 @@ end:
        return stream_class;
 }
 
-struct bt_stream_class *bt_stream_class_create(struct bt_trace *trace)
+struct bt_stream_class *bt_stream_class_create(struct bt_trace_class *tc)
 {
-       BT_ASSERT_PRE_NON_NULL(trace, "Trace");
-       BT_ASSERT_PRE(trace->assigns_automatic_stream_class_id,
-               "Trace does not automatically assigns stream class IDs: "
-               "%![sc-]+t", trace);
-       return create_stream_class_with_id(trace,
-               (uint64_t) trace->stream_classes->len);
+       BT_ASSERT_PRE_NON_NULL(tc, "Trace class");
+       BT_ASSERT_PRE(tc->assigns_automatic_stream_class_id,
+               "Trace class does not automatically assigns stream class IDs: "
+               "%![sc-]+T", tc);
+       return create_stream_class_with_id(tc,
+               (uint64_t) tc->stream_classes->len);
 }
 
 struct bt_stream_class *bt_stream_class_create_with_id(
-               struct bt_trace *trace, uint64_t id)
+               struct bt_trace_class *tc, uint64_t id)
 {
-       BT_ASSERT_PRE_NON_NULL(trace, "Trace");
-       BT_ASSERT_PRE(!trace->assigns_automatic_stream_class_id,
-               "Trace automatically assigns stream class IDs: "
-               "%![sc-]+t", trace);
-       return create_stream_class_with_id(trace, id);
+       BT_ASSERT_PRE_NON_NULL(tc, "Trace class");
+       BT_ASSERT_PRE(!tc->assigns_automatic_stream_class_id,
+               "Trace class automatically assigns stream class IDs: "
+               "%![sc-]+T", tc);
+       return create_stream_class_with_id(tc, id);
 }
 
-struct bt_trace *bt_stream_class_borrow_trace(
+struct bt_trace_class *bt_stream_class_borrow_trace_class(
                struct bt_stream_class *stream_class)
 {
        BT_ASSERT_PRE_NON_NULL(stream_class, "Stream class");
-       return bt_stream_class_borrow_trace_inline(stream_class);
+       return bt_stream_class_borrow_trace_class_inline(stream_class);
 }
 
-const struct bt_trace *bt_stream_class_borrow_trace_const(
+const struct bt_trace_class *bt_stream_class_borrow_trace_class_const(
                const struct bt_stream_class *stream_class)
 {
-       return bt_stream_class_borrow_trace((void *) stream_class);
+       return bt_stream_class_borrow_trace_class((void *) stream_class);
 }
 
 const char *bt_stream_class_get_name(const struct bt_stream_class *stream_class)
@@ -268,7 +265,7 @@ struct bt_event_class *bt_stream_class_borrow_event_class_by_id(
        struct bt_event_class *event_class = NULL;
        uint64_t i;
 
-       BT_ASSERT_PRE_NON_NULL(stream_class, "Trace");
+       BT_ASSERT_PRE_NON_NULL(stream_class, "Stream class");
 
        for (i = 0; i < stream_class->event_classes->len; i++) {
                struct bt_event_class *event_class_candidate =
@@ -319,21 +316,21 @@ int bt_stream_class_set_packet_context_field_class(
        BT_ASSERT_PRE_STREAM_CLASS_HOT(stream_class);
        BT_ASSERT_PRE(bt_field_class_get_type(field_class) ==
                BT_FIELD_CLASS_TYPE_STRUCTURE,
-               "Packet context field classe is not a structure field classe: %!+F",
+               "Packet context field class is not a structure field class: %!+F",
                field_class);
        resolve_ctx.packet_header =
-               bt_stream_class_borrow_trace_inline(stream_class)->packet_header_fc;
+               bt_stream_class_borrow_trace_class_inline(stream_class)->packet_header_fc;
        ret = bt_resolve_field_paths(field_class, &resolve_ctx);
        if (ret) {
                goto end;
        }
 
-       bt_field_class_make_part_of_trace(field_class);
+       bt_field_class_make_part_of_trace_class(field_class);
        bt_object_put_ref(stream_class->packet_context_fc);
        stream_class->packet_context_fc = field_class;
        bt_object_get_no_null_check(stream_class->packet_context_fc);
        bt_field_class_freeze(field_class);
-       BT_LIB_LOGV("Set stream class's packet context field classe: %!+S",
+       BT_LIB_LOGV("Set stream class's packet context field class: %!+S",
                stream_class);
 
 end:
@@ -366,22 +363,22 @@ int bt_stream_class_set_event_header_field_class(
        BT_ASSERT_PRE_STREAM_CLASS_HOT(stream_class);
        BT_ASSERT_PRE(bt_field_class_get_type(field_class) ==
                BT_FIELD_CLASS_TYPE_STRUCTURE,
-               "Event header field classe is not a structure field classe: %!+F",
+               "Event header field class is not a structure field class: %!+F",
                field_class);
        resolve_ctx.packet_header =
-               bt_stream_class_borrow_trace_inline(stream_class)->packet_header_fc;
+               bt_stream_class_borrow_trace_class_inline(stream_class)->packet_header_fc;
        resolve_ctx.packet_context = stream_class->packet_context_fc;
        ret = bt_resolve_field_paths(field_class, &resolve_ctx);
        if (ret) {
                goto end;
        }
 
-       bt_field_class_make_part_of_trace(field_class);
+       bt_field_class_make_part_of_trace_class(field_class);
        bt_object_put_ref(stream_class->event_header_fc);
        stream_class->event_header_fc = field_class;
        bt_object_get_no_null_check(stream_class->event_header_fc);
        bt_field_class_freeze(field_class);
-       BT_LIB_LOGV("Set stream class's event header field classe: %!+S",
+       BT_LIB_LOGV("Set stream class's event header field class: %!+S",
                stream_class);
 
 end:
@@ -415,10 +412,10 @@ int bt_stream_class_set_event_common_context_field_class(
        BT_ASSERT_PRE_STREAM_CLASS_HOT(stream_class);
        BT_ASSERT_PRE(bt_field_class_get_type(field_class) ==
                BT_FIELD_CLASS_TYPE_STRUCTURE,
-               "Event common context field classe is not a structure field classe: %!+F",
+               "Event common context field class is not a structure field class: %!+F",
                field_class);
        resolve_ctx.packet_header =
-               bt_stream_class_borrow_trace_inline(stream_class)->packet_header_fc;
+               bt_stream_class_borrow_trace_class_inline(stream_class)->packet_header_fc;
        resolve_ctx.packet_context = stream_class->packet_context_fc;
        resolve_ctx.event_header = stream_class->event_header_fc;
        ret = bt_resolve_field_paths(field_class, &resolve_ctx);
@@ -426,12 +423,12 @@ int bt_stream_class_set_event_common_context_field_class(
                goto end;
        }
 
-       bt_field_class_make_part_of_trace(field_class);
+       bt_field_class_make_part_of_trace_class(field_class);
        bt_object_put_ref(stream_class->event_common_context_fc);
        stream_class->event_common_context_fc = field_class;
        bt_object_get_no_null_check(stream_class->event_common_context_fc);
        bt_field_class_freeze(field_class);
-       BT_LIB_LOGV("Set stream class's event common context field classe: %!+S",
+       BT_LIB_LOGV("Set stream class's event common context field class: %!+S",
                stream_class);
 
 end:
@@ -603,3 +600,13 @@ bt_bool bt_stream_class_default_clock_is_always_known(
        /* BT_CLOCK_VALUE_STATUS_UNKNOWN is not supported as of 2.0 */
        return BT_TRUE;
 }
+
+void bt_stream_class_get_ref(const struct bt_stream_class *stream_class)
+{
+       bt_object_get_ref(stream_class);
+}
+
+void bt_stream_class_put_ref(const struct bt_stream_class *stream_class)
+{
+       bt_object_put_ref(stream_class);
+}
This page took 0.040326 seconds and 4 git commands to generate.