lib: make trace IR API const-correct

[babeltrace.git] / lib / trace-ir / clock-class.c

diff --git a/lib/trace-ir/clock-class.c b/lib/trace-ir/clock-class.c
index c8e5f3fdbeee85a3a39840f26b5caadc3b428497..4ab90d24cdb9c761659720bdcdde9fa9c404d2a9 100644 (file)
--- a/lib/trace-ir/clock-class.c
+++ b/lib/trace-ir/clock-class.c
@@ -1,8 +1,4 @@
 /*
- * clock-class.c
- *
- * Babeltrace trace IR - Clock class
- *
  * Copyright 2013, 2014 Jérémie Galarneau <jeremie.galarneau@efficios.com>
  *
  * Author: Jérémie Galarneau <jeremie.galarneau@efficios.com>
@@ -34,7 +30,7 @@
 #include <babeltrace/trace-ir/clock-class-internal.h>
 #include <babeltrace/trace-ir/clock-value-internal.h>
 #include <babeltrace/trace-ir/utils-internal.h>
-#include <babeltrace/ref.h>
+#include <babeltrace/object.h>
 #include <babeltrace/compiler-internal.h>
 #include <babeltrace/types.h>
 #include <babeltrace/compat/string-internal.h>
@@ -54,10 +50,14 @@ void destroy_clock_class(struct bt_object *obj)
 
        if (clock_class->name.str) {
                g_string_free(clock_class->name.str, TRUE);
+               clock_class->name.str = NULL;
+               clock_class->name.value = NULL;
        }
 
        if (clock_class->description.str) {
                g_string_free(clock_class->description.str, TRUE);
+               clock_class->description.str = NULL;
+               clock_class->description.value = NULL;
        }
 
        bt_object_pool_finalize(&clock_class->cv_pool);
@@ -150,14 +150,13 @@ struct bt_clock_class *bt_clock_class_create(void)
        goto end;
 
 error:
-       BT_PUT(clock_class);
+       BT_OBJECT_PUT_REF_AND_RESET(clock_class);
 
 end:
        return clock_class;
 }
 
-const char *bt_clock_class_get_name(
-               struct bt_clock_class *clock_class)
+const char *bt_clock_class_get_name(const struct bt_clock_class *clock_class)
 {
        BT_ASSERT_PRE_NON_NULL(clock_class, "Clock class");
        return clock_class->name.value;
@@ -175,7 +174,8 @@ int bt_clock_class_set_name(struct bt_clock_class *clock_class,
        return 0;
 }
 
-const char *bt_clock_class_get_description(struct bt_clock_class *clock_class)
+const char *bt_clock_class_get_description(
+               const struct bt_clock_class *clock_class)
 {
        BT_ASSERT_PRE_NON_NULL(clock_class, "Clock class");
        return clock_class->description.value;
@@ -194,13 +194,13 @@ int bt_clock_class_set_description(struct bt_clock_class *clock_class,
        return 0;
 }
 
-uint64_t bt_clock_class_get_frequency(struct bt_clock_class *clock_class)
+uint64_t bt_clock_class_get_frequency(const struct bt_clock_class *clock_class)
 {
        BT_ASSERT_PRE_NON_NULL(clock_class, "Clock class");
        return clock_class->frequency;
 }
 
-int bt_clock_class_set_frequency(struct bt_clock_class *clock_class,
+void bt_clock_class_set_frequency(struct bt_clock_class *clock_class,
                uint64_t frequency)
 {
        BT_ASSERT_PRE_NON_NULL(clock_class, "Clock class");
@@ -214,16 +214,15 @@ int bt_clock_class_set_frequency(struct bt_clock_class *clock_class,
        clock_class->frequency = frequency;
        set_base_offset(clock_class);
        BT_LIB_LOGV("Set clock class's frequency: %!+K", clock_class);
-       return 0;
 }
 
-uint64_t bt_clock_class_get_precision(struct bt_clock_class *clock_class)
+uint64_t bt_clock_class_get_precision(const struct bt_clock_class *clock_class)
 {
        BT_ASSERT_PRE_NON_NULL(clock_class, "Clock class");
        return clock_class->precision;
 }
 
-int bt_clock_class_set_precision(struct bt_clock_class *clock_class,
+void bt_clock_class_set_precision(struct bt_clock_class *clock_class,
                uint64_t precision)
 {
        BT_ASSERT_PRE_NON_NULL(clock_class, "Clock class");
@@ -233,10 +232,9 @@ int bt_clock_class_set_precision(struct bt_clock_class *clock_class,
                clock_class, precision);
        clock_class->precision = precision;
        BT_LIB_LOGV("Set clock class's precision: %!+K", clock_class);
-       return 0;
 }
 
-void bt_clock_class_get_offset(struct bt_clock_class *clock_class,
+void bt_clock_class_get_offset(const struct bt_clock_class *clock_class,
                int64_t *seconds, uint64_t *cycles)
 {
        BT_ASSERT_PRE_NON_NULL(clock_class, "Clock class");
@@ -246,7 +244,7 @@ void bt_clock_class_get_offset(struct bt_clock_class *clock_class,
        *cycles = clock_class->offset_cycles;
 }
 
-int bt_clock_class_set_offset(struct bt_clock_class *clock_class,
+void bt_clock_class_set_offset(struct bt_clock_class *clock_class,
                int64_t seconds, uint64_t cycles)
 {
        BT_ASSERT_PRE_NON_NULL(clock_class, "Clock class");
@@ -258,16 +256,15 @@ int bt_clock_class_set_offset(struct bt_clock_class *clock_class,
        clock_class->offset_cycles = cycles;
        set_base_offset(clock_class);
        BT_LIB_LOGV("Set clock class's offset: %!+K", clock_class);
-       return 0;
 }
 
-bt_bool bt_clock_class_is_absolute(struct bt_clock_class *clock_class)
+bt_bool bt_clock_class_is_absolute(const struct bt_clock_class *clock_class)
 {
        BT_ASSERT_PRE_NON_NULL(clock_class, "Clock class");
        return (bool) clock_class->is_absolute;
 }
 
-int bt_clock_class_set_is_absolute(struct bt_clock_class *clock_class,
+void bt_clock_class_set_is_absolute(struct bt_clock_class *clock_class,
                bt_bool is_absolute)
 {
        BT_ASSERT_PRE_NON_NULL(clock_class, "Clock class");
@@ -275,16 +272,15 @@ int bt_clock_class_set_is_absolute(struct bt_clock_class *clock_class,
        clock_class->is_absolute = (bool) is_absolute;
        BT_LIB_LOGV("Set clock class's absolute property: %!+K",
                clock_class);
-       return 0;
 }
 
-bt_uuid bt_clock_class_get_uuid(struct bt_clock_class *clock_class)
+bt_uuid bt_clock_class_get_uuid(const struct bt_clock_class *clock_class)
 {
        BT_ASSERT_PRE_NON_NULL(clock_class, "Clock class");
        return clock_class->uuid.value;
 }
 
-int bt_clock_class_set_uuid(struct bt_clock_class *clock_class,
+void bt_clock_class_set_uuid(struct bt_clock_class *clock_class,
                bt_uuid uuid)
 {
        BT_ASSERT_PRE_NON_NULL(clock_class, "Clock class");
@@ -293,11 +289,10 @@ int bt_clock_class_set_uuid(struct bt_clock_class *clock_class,
        memcpy(clock_class->uuid.uuid, uuid, BABELTRACE_UUID_LEN);
        clock_class->uuid.value = clock_class->uuid.uuid;
        BT_LIB_LOGV("Set clock class's UUID: %!+K", clock_class);
-       return 0;
 }
 
 BT_HIDDEN
-void _bt_clock_class_freeze(struct bt_clock_class *clock_class)
+void _bt_clock_class_freeze(const struct bt_clock_class *clock_class)
 {
        BT_ASSERT(clock_class);
 
@@ -306,10 +301,11 @@ void _bt_clock_class_freeze(struct bt_clock_class *clock_class)
        }
 
        BT_LIB_LOGD("Freezing clock class: %!+K", clock_class);
-       clock_class->frozen = 1;
+       ((struct bt_clock_class *) clock_class)->frozen = 1;
 }
 
-int bt_clock_class_cycles_to_ns_from_origin(struct bt_clock_class *clock_class,
+int bt_clock_class_cycles_to_ns_from_origin(
+               const struct bt_clock_class *clock_class,
                uint64_t cycles, int64_t *ns)
 {
        int ret;