lib: de-duplicate "no error" assertion in add port functions

[babeltrace.git] / src / lib / graph / component-source.c

diff --git a/src/lib/graph/component-source.c b/src/lib/graph/component-source.c
index c1638fd3d4bd041a050f87e53d31d5cb5c8d6c42..83f1ef6973194b4e4195848df332cb7d3cd07cd8 100644 (file)
--- a/src/lib/graph/component-source.c
+++ b/src/lib/graph/component-source.c
@@ -21,19 +21,15 @@
 #include "message/iterator.h"
 #include "lib/func-status.h"
 
-BT_HIDDEN
 void bt_component_source_destroy(struct bt_component *component)
 {
 }
 
-BT_HIDDEN
 struct bt_component *bt_component_source_create(
                const struct bt_component_class *class)
 {
        struct bt_component_source *source = NULL;
 
-       BT_ASSERT_PRE_NO_ERROR();
-
        source = g_new0(struct bt_component_source, 1);
        if (!source) {
                BT_LIB_LOGE_APPEND_CAUSE(
@@ -45,6 +41,7 @@ end:
        return (void *) source;
 }
 
+BT_EXPORT
 const bt_component_class_source *
 bt_component_source_borrow_class_const(
                const bt_component_source *component)
@@ -61,42 +58,67 @@ bt_component_source_borrow_class_const(
        return (bt_component_class_source *) cls;
 }
 
+BT_EXPORT
 uint64_t bt_component_source_get_output_port_count(
                const struct bt_component_source *comp)
 {
-       return bt_component_get_output_port_count((void *) comp);
+       /* bt_component_get_output_port_count() checks preconditions */
+       return bt_component_get_output_port_count((void *) comp, __func__);
 }
 
+BT_EXPORT
 const struct bt_port_output *
 bt_component_source_borrow_output_port_by_name_const(
                const struct bt_component_source *comp, const char *name)
 {
-       return bt_component_borrow_output_port_by_name((void *) comp, name);
+       /*
+        * bt_component_borrow_output_port_by_name() logs details/errors
+        * and checks preconditions.
+        */
+       return bt_component_borrow_output_port_by_name((void *) comp, name,
+               __func__);
 }
 
+BT_EXPORT
 struct bt_self_component_port_output *
 bt_self_component_source_borrow_output_port_by_name(
                struct bt_self_component_source *comp, const char *name)
 {
+       /*
+        * bt_component_borrow_output_port_by_name() logs details/errors
+        * and checks preconditions.
+        */
        return (void *) bt_component_borrow_output_port_by_name(
-               (void *) comp, name);
+               (void *) comp, name, __func__);
 }
 
+BT_EXPORT
 const struct bt_port_output *
 bt_component_source_borrow_output_port_by_index_const(
                const struct bt_component_source *comp, uint64_t index)
 {
-       return bt_component_borrow_output_port_by_index((void *) comp, index);
+       /*
+        * bt_component_borrow_output_port_by_index() logs
+        * details/errors and checks preconditions.
+        */
+       return bt_component_borrow_output_port_by_index((void *) comp, index,
+               __func__);
 }
 
+BT_EXPORT
 struct bt_self_component_port_output *
 bt_self_component_source_borrow_output_port_by_index(
                struct bt_self_component_source *comp, uint64_t index)
 {
+       /*
+        * bt_component_borrow_output_port_by_index() logs
+        * details/errors and checks preconditions.
+        */
        return (void *) bt_component_borrow_output_port_by_index(
-               (void *) comp, index);
+               (void *) comp, index, __func__);
 }
 
+BT_EXPORT
 enum bt_self_component_add_port_status bt_self_component_source_add_output_port(
                struct bt_self_component_source *self_comp,
                const char *name, void *user_data,
@@ -106,11 +128,12 @@ enum bt_self_component_add_port_status bt_self_component_source_add_output_port(
        enum bt_self_component_add_port_status status;
        struct bt_port *port = NULL;
 
-       BT_ASSERT_PRE_NO_ERROR();
-       BT_ASSERT_PRE_OUTPUT_PORT_NAME_UNIQUE(comp, name);
-
-       /* bt_component_add_output_port() logs details and errors */
-       status = bt_component_add_output_port(comp, name, user_data, &port);
+       /*
+        * bt_component_add_output_port() logs details/errors and checks
+        * preconditions.
+        */
+       status = bt_component_add_output_port(comp, name, user_data, &port,
+               __func__);
        if (status != BT_FUNC_STATUS_OK) {
                goto end;
        }
@@ -125,12 +148,14 @@ end:
        return status;
 }
 
+BT_EXPORT
 void bt_component_source_get_ref(
                const struct bt_component_source *component_source)
 {
        bt_object_get_ref(component_source);
 }
 
+BT_EXPORT
 void bt_component_source_put_ref(
                const struct bt_component_source *component_source)
 {