lib: make public reference count functions have strict types
[babeltrace.git] / plugins / text / dmesg / dmesg.c
index 36efc583ed6a886e78dd93c8b5ed9ba583455403..ffe35fdebc701b864298cf34244d8eb434079020 100644 (file)
@@ -31,7 +31,7 @@
 #include <babeltrace/assert-internal.h>
 #include <babeltrace/common-internal.h>
 #include <babeltrace/babeltrace.h>
-#include <babeltrace/values-internal.h>
+#include <babeltrace/value-internal.h>
 #include <babeltrace/compat/utc-internal.h>
 #include <babeltrace/compat/stdio-internal.h>
 #include <glib.h>
@@ -49,7 +49,7 @@ struct dmesg_notif_iter {
        char *linebuf;
        size_t linebuf_len;
        FILE *fp;
-       struct bt_private_notification *tmp_event_notif;
+       struct bt_notification *tmp_event_notif;
 
        enum {
                STATE_EMIT_STREAM_BEGINNING,
@@ -68,34 +68,35 @@ struct dmesg_component {
                bt_bool no_timestamp;
        } params;
 
-       struct bt_private_trace *trace;
-       struct bt_private_stream_class *stream_class;
-       struct bt_private_event_class *event_class;
-       struct bt_private_stream *stream;
-       struct bt_private_packet *packet;
-       struct bt_private_clock_class *clock_class;
+       struct bt_trace_class *trace_class;
+       struct bt_stream_class *stream_class;
+       struct bt_event_class *event_class;
+       struct bt_trace *trace;
+       struct bt_stream *stream;
+       struct bt_packet *packet;
+       struct bt_clock_class *clock_class;
 };
 
 static
-struct bt_private_field_class *create_event_payload_fc(void)
+struct bt_field_class *create_event_payload_fc(void)
 {
-       struct bt_private_field_class *root_fc = NULL;
-       struct bt_private_field_class *fc = NULL;
+       struct bt_field_class *root_fc = NULL;
+       struct bt_field_class *fc = NULL;
        int ret;
 
-       root_fc = bt_private_field_class_structure_create();
+       root_fc = bt_field_class_structure_create();
        if (!root_fc) {
                BT_LOGE_STR("Cannot create an empty structure field class object.");
                goto error;
        }
 
-       fc = bt_private_field_class_string_create();
+       fc = bt_field_class_string_create();
        if (!fc) {
                BT_LOGE_STR("Cannot create a string field class object.");
                goto error;
        }
 
-       ret = bt_private_field_class_structure_append_member(root_fc,
+       ret = bt_field_class_structure_append_member(root_fc,
                "str", fc);
        if (ret) {
                BT_LOGE("Cannot add `str` member to structure field class: "
@@ -106,79 +107,55 @@ struct bt_private_field_class *create_event_payload_fc(void)
        goto end;
 
 error:
-       BT_OBJECT_PUT_REF_AND_RESET(root_fc);
+       BT_FIELD_CLASS_PUT_REF_AND_RESET(root_fc);
 
 end:
-       bt_object_put_ref(fc);
+       bt_field_class_put_ref(fc);
        return root_fc;
 }
 
 static
 int create_meta(struct dmesg_component *dmesg_comp, bool has_ts)
 {
-       struct bt_private_field_class *fc = NULL;
-       const char *trace_name = NULL;
-       gchar *basename = NULL;
+       struct bt_field_class *fc = NULL;
        int ret = 0;
 
-       dmesg_comp->trace = bt_private_trace_create();
-       if (!dmesg_comp->trace) {
-               BT_LOGE_STR("Cannot create an empty trace object.");
+       dmesg_comp->trace_class = bt_trace_class_create();
+       if (!dmesg_comp->trace_class) {
+               BT_LOGE_STR("Cannot create an empty trace class object.");
                goto error;
        }
 
-       if (dmesg_comp->params.read_from_stdin) {
-               trace_name = "STDIN";
-       } else {
-               basename = g_path_get_basename(dmesg_comp->params.path->str);
-               BT_ASSERT(basename);
-
-               if (strcmp(basename, G_DIR_SEPARATOR_S) != 0 &&
-                               strcmp(basename, ".") != 0) {
-                       trace_name = basename;
-               }
-       }
-
-       if (trace_name) {
-               ret = bt_private_trace_set_name(dmesg_comp->trace, trace_name);
-               if (ret) {
-                       BT_LOGE("Cannot set trace's name: name=\"%s\"", trace_name);
-                       goto error;
-               }
-       }
-
-       dmesg_comp->stream_class = bt_private_stream_class_create(
-               dmesg_comp->trace);
+       dmesg_comp->stream_class = bt_stream_class_create(
+               dmesg_comp->trace_class);
        if (!dmesg_comp->stream_class) {
                BT_LOGE_STR("Cannot create a stream class object.");
                goto error;
        }
 
        if (has_ts) {
-               dmesg_comp->clock_class = bt_private_clock_class_create();
+               dmesg_comp->clock_class = bt_clock_class_create();
                if (!dmesg_comp->clock_class) {
                        BT_LOGE_STR("Cannot create clock class.");
                        goto error;
                }
 
-               ret = bt_private_stream_class_set_default_clock_class(
-                       dmesg_comp->stream_class,
-                       bt_private_clock_class_borrow_clock_class(
-                               dmesg_comp->clock_class));
+               ret = bt_stream_class_set_default_clock_class(
+                       dmesg_comp->stream_class, dmesg_comp->clock_class);
                if (ret) {
                        BT_LOGE_STR("Cannot set stream class's default clock class.");
                        goto error;
                }
        }
 
-       dmesg_comp->event_class = bt_private_event_class_create(
+       dmesg_comp->event_class = bt_event_class_create(
                dmesg_comp->stream_class);
        if (!dmesg_comp->event_class) {
                BT_LOGE_STR("Cannot create an event class object.");
                goto error;
        }
 
-       ret = bt_private_event_class_set_name(dmesg_comp->event_class, "string");
+       ret = bt_event_class_set_name(dmesg_comp->event_class, "string");
        if (ret) {
                BT_LOGE_STR("Cannot set event class's name.");
                goto error;
@@ -190,7 +167,7 @@ int create_meta(struct dmesg_component *dmesg_comp, bool has_ts)
                goto error;
        }
 
-       ret = bt_private_event_class_set_payload_field_class(dmesg_comp->event_class, fc);
+       ret = bt_event_class_set_payload_field_class(dmesg_comp->event_class, fc);
        if (ret) {
                BT_LOGE_STR("Cannot set event class's event payload field class.");
                goto error;
@@ -202,24 +179,20 @@ error:
        ret = -1;
 
 end:
-       bt_object_put_ref(fc);
-
-       if (basename) {
-               g_free(basename);
-       }
-
+       bt_field_class_put_ref(fc);
        return ret;
 }
 
 static
-int handle_params(struct dmesg_component *dmesg_comp, struct bt_value *params)
+int handle_params(struct dmesg_component *dmesg_comp,
+               const struct bt_value *params)
 {
-       struct bt_value *no_timestamp = NULL;
-       struct bt_value *path = NULL;
+       const struct bt_value *no_timestamp = NULL;
+       const struct bt_value *path = NULL;
        const char *path_str;
        int ret = 0;
 
-       no_timestamp = bt_value_map_borrow_entry_value(params,
+       no_timestamp = bt_value_map_borrow_entry_value_const(params,
                "no-extract-timestamp");
        if (no_timestamp) {
                if (!bt_value_is_bool(no_timestamp)) {
@@ -234,7 +207,7 @@ int handle_params(struct dmesg_component *dmesg_comp, struct bt_value *params)
                        bt_value_bool_get(no_timestamp);
        }
 
-       path = bt_value_map_borrow_entry_value(params, "path");
+       path = bt_value_map_borrow_entry_value_const(params, "path");
        if (path) {
                if (dmesg_comp->params.read_from_stdin) {
                        BT_LOGE_STR("Cannot specify both `read-from-stdin` and `path` parameters.");
@@ -265,23 +238,53 @@ end:
 }
 
 static
-int create_packet_and_stream(struct dmesg_component *dmesg_comp)
+int create_packet_and_stream_and_trace(struct dmesg_component *dmesg_comp)
 {
        int ret = 0;
+       const char *trace_name;
+       gchar *basename = NULL;
 
-       dmesg_comp->stream = bt_private_stream_create(dmesg_comp->stream_class);
+       dmesg_comp->trace = bt_trace_create(dmesg_comp->trace_class);
+       if (!dmesg_comp->trace) {
+               BT_LOGE_STR("Cannot create trace object.");
+               goto error;
+       }
+
+       if (dmesg_comp->params.read_from_stdin) {
+               trace_name = "STDIN";
+       } else {
+               basename = g_path_get_basename(dmesg_comp->params.path->str);
+               BT_ASSERT(basename);
+
+               if (strcmp(basename, G_DIR_SEPARATOR_S) != 0 &&
+                               strcmp(basename, ".") != 0) {
+                       trace_name = basename;
+               }
+       }
+
+       if (trace_name) {
+               ret = bt_trace_set_name(dmesg_comp->trace, trace_name);
+               if (ret) {
+                       BT_LOGE("Cannot set trace's name: name=\"%s\"",
+                               trace_name);
+                       goto error;
+               }
+       }
+
+       dmesg_comp->stream = bt_stream_create(dmesg_comp->stream_class,
+               dmesg_comp->trace);
        if (!dmesg_comp->stream) {
                BT_LOGE_STR("Cannot create stream object.");
                goto error;
        }
 
-       dmesg_comp->packet = bt_private_packet_create(dmesg_comp->stream);
+       dmesg_comp->packet = bt_packet_create(dmesg_comp->stream);
        if (!dmesg_comp->packet) {
                BT_LOGE_STR("Cannot create packet object.");
                goto error;
        }
 
-       ret = bt_private_trace_make_static(dmesg_comp->trace);
+       ret = bt_trace_make_static(dmesg_comp->trace);
        if (ret) {
                BT_LOGE_STR("Cannot make trace static.");
                goto error;
@@ -293,6 +296,10 @@ error:
        ret = -1;
 
 end:
+       if (basename) {
+               g_free(basename);
+       }
+
        return ret;
 }
 
@@ -314,7 +321,7 @@ int try_create_meta_stream_packet(struct dmesg_component *dmesg_comp,
                goto error;
        }
 
-       ret = create_packet_and_stream(dmesg_comp);
+       ret = create_packet_and_stream_and_trace(dmesg_comp);
        if (ret) {
                BT_LOGE("Cannot create packet and stream objects: "
                        "dmesg-comp-addr=%p", dmesg_comp);
@@ -341,12 +348,12 @@ void destroy_dmesg_component(struct dmesg_component *dmesg_comp)
                g_string_free(dmesg_comp->params.path, TRUE);
        }
 
-       bt_object_put_ref(dmesg_comp->packet);
-       bt_object_put_ref(dmesg_comp->trace);
-       bt_object_put_ref(dmesg_comp->stream_class);
-       bt_object_put_ref(dmesg_comp->event_class);
-       bt_object_put_ref(dmesg_comp->stream);
-       bt_object_put_ref(dmesg_comp->clock_class);
+       bt_packet_put_ref(dmesg_comp->packet);
+       bt_trace_put_ref(dmesg_comp->trace);
+       bt_stream_class_put_ref(dmesg_comp->stream_class);
+       bt_event_class_put_ref(dmesg_comp->event_class);
+       bt_stream_put_ref(dmesg_comp->stream);
+       bt_clock_class_put_ref(dmesg_comp->clock_class);
        g_free(dmesg_comp);
 }
 
@@ -399,14 +406,14 @@ enum bt_self_component_status dmesg_init(
        }
 
        bt_self_component_set_data(
-               bt_self_component_source_borrow_self_component(self_comp),
+               bt_self_component_source_as_self_component(self_comp),
                dmesg_comp);
        goto end;
 
 error:
        destroy_dmesg_component(dmesg_comp);
        bt_self_component_set_data(
-               bt_self_component_source_borrow_self_component(self_comp),
+               bt_self_component_source_as_self_component(self_comp),
                NULL);
 
        if (status >= 0) {
@@ -421,16 +428,16 @@ BT_HIDDEN
 void dmesg_finalize(struct bt_self_component_source *self_comp)
 {
        destroy_dmesg_component(bt_self_component_get_data(
-               bt_self_component_source_borrow_self_component(self_comp)));
+               bt_self_component_source_as_self_component(self_comp)));
 }
 
 static
-struct bt_private_notification *create_init_event_notif_from_line(
+struct bt_notification *create_init_event_notif_from_line(
                struct dmesg_notif_iter *notif_iter,
                const char *line, const char **new_start)
 {
-       struct bt_private_event *event;
-       struct bt_private_notification *notif = NULL;
+       struct bt_event *event;
+       struct bt_notification *notif = NULL;
        bool has_timestamp = false;
        unsigned long sec, usec, msec;
        unsigned int year, mon, mday, hour, min;
@@ -500,25 +507,24 @@ skip_ts:
                goto error;
        }
 
-       notif = bt_private_notification_event_create(notif_iter->pc_notif_iter,
+       notif = bt_notification_event_create(notif_iter->pc_notif_iter,
                dmesg_comp->event_class, dmesg_comp->packet);
        if (!notif) {
                BT_LOGE_STR("Cannot create event notification.");
                goto error;
        }
 
-       event = bt_private_notification_event_borrow_event(notif);
+       event = bt_notification_event_borrow_event(notif);
        BT_ASSERT(event);
 
        if (dmesg_comp->clock_class) {
-               ret = bt_private_event_set_default_clock_value(event, ts);
-               BT_ASSERT(ret == 0);
+               bt_event_set_default_clock_value(event, ts);
        }
 
        goto end;
 
 error:
-       BT_OBJECT_PUT_REF_AND_RESET(notif);
+       BT_NOTIFICATION_PUT_REF_AND_RESET(notif);
 
 end:
        return notif;
@@ -526,16 +532,16 @@ end:
 
 static
 int fill_event_payload_from_line(const char *line,
-               struct bt_private_event *event)
+               struct bt_event *event)
 {
-       struct bt_private_field *ep_field = NULL;
-       struct bt_private_field *str_field = NULL;
+       struct bt_field *ep_field = NULL;
+       struct bt_field *str_field = NULL;
        size_t len;
        int ret;
 
-       ep_field = bt_private_event_borrow_payload_field(event);
+       ep_field = bt_event_borrow_payload_field(event);
        BT_ASSERT(ep_field);
-       str_field = bt_private_field_structure_borrow_member_field_by_index(
+       str_field = bt_field_structure_borrow_member_field_by_index(
                ep_field, 0);
        if (!str_field) {
                BT_LOGE_STR("Cannot borrow `timestamp` field from event payload structure field.");
@@ -548,13 +554,13 @@ int fill_event_payload_from_line(const char *line,
                len--;
        }
 
-       ret = bt_private_field_string_clear(str_field);
+       ret = bt_field_string_clear(str_field);
        if (ret) {
                BT_LOGE_STR("Cannot clear string field object.");
                goto error;
        }
 
-       ret = bt_private_field_string_append_with_length(str_field, line, len);
+       ret = bt_field_string_append_with_length(str_field, line, len);
        if (ret) {
                BT_LOGE("Cannot append value to string field object: "
                        "len=%zu", len);
@@ -571,11 +577,11 @@ end:
 }
 
 static
-struct bt_private_notification *create_notif_from_line(
+struct bt_notification *create_notif_from_line(
                struct dmesg_notif_iter *dmesg_notif_iter, const char *line)
 {
-       struct bt_private_event *event = NULL;
-       struct bt_private_notification *notif = NULL;
+       struct bt_event *event = NULL;
+       struct bt_notification *notif = NULL;
        const char *new_start;
        int ret;
 
@@ -586,7 +592,7 @@ struct bt_private_notification *create_notif_from_line(
                goto error;
        }
 
-       event = bt_private_notification_event_borrow_event(notif);
+       event = bt_notification_event_borrow_event(notif);
        BT_ASSERT(event);
        ret = fill_event_payload_from_line(new_start, event);
        if (ret) {
@@ -598,7 +604,7 @@ struct bt_private_notification *create_notif_from_line(
        goto end;
 
 error:
-       BT_OBJECT_PUT_REF_AND_RESET(notif);
+       BT_NOTIFICATION_PUT_REF_AND_RESET(notif);
 
 end:
        return notif;
@@ -617,7 +623,7 @@ void destroy_dmesg_notif_iter(struct dmesg_notif_iter *dmesg_notif_iter)
                }
        }
 
-       bt_object_put_ref(dmesg_notif_iter->tmp_event_notif);
+       bt_notification_put_ref(dmesg_notif_iter->tmp_event_notif);
        free(dmesg_notif_iter->linebuf);
        g_free(dmesg_notif_iter);
 }
@@ -640,7 +646,7 @@ enum bt_self_notification_iterator_status dmesg_notif_iter_init(
        }
 
        dmesg_comp = bt_self_component_get_data(
-               bt_self_component_source_borrow_self_component(self_comp));
+               bt_self_component_source_as_self_component(self_comp));
        BT_ASSERT(dmesg_comp);
        dmesg_notif_iter->dmesg_comp = dmesg_comp;
        dmesg_notif_iter->pc_notif_iter = self_notif_iter;
@@ -682,7 +688,7 @@ void dmesg_notif_iter_finalize(
 static
 enum bt_self_notification_iterator_status dmesg_notif_iter_next_one(
                struct dmesg_notif_iter *dmesg_notif_iter,
-               struct bt_private_notification **notif)
+               struct bt_notification **notif)
 {
        ssize_t len;
        struct dmesg_component *dmesg_comp;
@@ -762,13 +768,13 @@ handle_state:
        switch (dmesg_notif_iter->state) {
        case STATE_EMIT_STREAM_BEGINNING:
                BT_ASSERT(dmesg_notif_iter->tmp_event_notif);
-               *notif = bt_private_notification_stream_begin_create(
+               *notif = bt_notification_stream_beginning_create(
                        dmesg_notif_iter->pc_notif_iter, dmesg_comp->stream);
                dmesg_notif_iter->state = STATE_EMIT_PACKET_BEGINNING;
                break;
        case STATE_EMIT_PACKET_BEGINNING:
                BT_ASSERT(dmesg_notif_iter->tmp_event_notif);
-               *notif = bt_private_notification_packet_begin_create(
+               *notif = bt_notification_packet_beginning_create(
                        dmesg_notif_iter->pc_notif_iter, dmesg_comp->packet);
                dmesg_notif_iter->state = STATE_EMIT_EVENT;
                break;
@@ -778,12 +784,12 @@ handle_state:
                dmesg_notif_iter->tmp_event_notif = NULL;
                break;
        case STATE_EMIT_PACKET_END:
-               *notif = bt_private_notification_packet_end_create(
+               *notif = bt_notification_packet_end_create(
                        dmesg_notif_iter->pc_notif_iter, dmesg_comp->packet);
                dmesg_notif_iter->state = STATE_EMIT_STREAM_END;
                break;
        case STATE_EMIT_STREAM_END:
-               *notif = bt_private_notification_stream_end_create(
+               *notif = bt_notification_stream_end_create(
                        dmesg_notif_iter->pc_notif_iter, dmesg_comp->stream);
                dmesg_notif_iter->state = STATE_DONE;
                break;
@@ -804,7 +810,7 @@ end:
 BT_HIDDEN
 enum bt_self_notification_iterator_status dmesg_notif_iter_next(
                struct bt_self_notification_iterator *self_notif_iter,
-               bt_notification_array notifs, uint64_t capacity,
+               bt_notification_array_const notifs, uint64_t capacity,
                uint64_t *count)
 {
        struct dmesg_notif_iter *dmesg_notif_iter =
@@ -816,12 +822,11 @@ enum bt_self_notification_iterator_status dmesg_notif_iter_next(
 
        while (i < capacity &&
                        status == BT_SELF_NOTIFICATION_ITERATOR_STATUS_OK) {
-               struct bt_private_notification *priv_notif = NULL;
+               struct bt_notification *priv_notif = NULL;
 
                status = dmesg_notif_iter_next_one(dmesg_notif_iter,
                        &priv_notif);
-               notifs[i] = bt_private_notification_borrow_notification(
-                       priv_notif);
+               notifs[i] = priv_notif;
                if (status == BT_SELF_NOTIFICATION_ITERATOR_STATUS_OK) {
                        i++;
                }
This page took 0.030989 seconds and 4 git commands to generate.