lib: remove unused public `enum bt_plugin_status`
[babeltrace.git] / lib / plugin / plugin-so.c
index b06a1b6b689538478ae7fd29e40b2552c6ae4c6a..a535d9d877ec98cbe844992173f9492d7036e78f 100644 (file)
@@ -71,7 +71,7 @@ BT_PLUGIN_MODULE();
  *
  * It allows this use-case:
  *
- *        my_plugins = bt_plugin_create_all_from_file("/path/to/my-plugin.so");
+ *        my_plugins = bt_plugin_find_all_from_file("/path/to/my-plugin.so");
  *        // instantiate components from a plugin's component classes
  *        // put plugins and free my_plugins here
  *        // user code of instantiated components still exists
@@ -102,6 +102,21 @@ void fini_comp_class_list(void)
        BT_LOGD_STR("Released references from all component classes to shared library handles.");
 }
 
+static inline
+const char *bt_plugin_init_status_string(enum bt_plugin_init_status status)
+{
+       switch (status) {
+       case BT_PLUGIN_INIT_STATUS_OK:
+               return "BT_PLUGIN_INIT_STATUS_OK";
+       case BT_PLUGIN_INIT_STATUS_ERROR:
+               return "BT_PLUGIN_INIT_STATUS_ERROR";
+       case BT_PLUGIN_INIT_STATUS_NOMEM:
+               return "BT_PLUGIN_INIT_STATUS_NOMEM";
+       default:
+               return "(unknown)";
+       }
+}
+
 static
 void bt_plugin_so_shared_lib_handle_destroy(struct bt_object *obj)
 {
@@ -117,17 +132,9 @@ void bt_plugin_so_shared_lib_handle_destroy(struct bt_object *obj)
                shared_lib_handle, path);
 
        if (shared_lib_handle->init_called && shared_lib_handle->exit) {
-               enum bt_plugin_status status;
-
                BT_LOGD_STR("Calling user's plugin exit function.");
-               status = shared_lib_handle->exit();
-               BT_LOGD("User function returned: %s",
-                       bt_plugin_status_string(status));
-
-               if (status < 0) {
-                       BT_LOGW("User's plugin exit function failed: "
-                               "path=\"%s\"", path);
-               }
+               shared_lib_handle->exit();
+               BT_LOGD_STR("User function returned.");
        }
 
        if (shared_lib_handle->module) {
@@ -198,7 +205,7 @@ struct bt_plugin_so_shared_lib_handle *bt_plugin_so_shared_lib_handle_create(
                 * DEBUG-level logging because we're only _trying_ to
                 * open this file as a Babeltrace plugin: if it's not,
                 * it's not an error. And because this can be tried
-                * during bt_plugin_create_all_from_dir(), it's not even
+                * during bt_plugin_find_all_from_dir(), it's not even
                 * a warning.
                 */
                BT_LOGD("Cannot open GModule: %s: path=\"%s\"",
@@ -291,8 +298,8 @@ enum bt_plugin_status bt_plugin_so_init(
                                bt_component_class_source_accept_output_port_connection_method accept_output_port_connection;
                                bt_component_class_source_output_port_connected_method output_port_connected;
                                bt_component_class_source_output_port_disconnected_method output_port_disconnected;
-                               bt_component_class_source_notification_iterator_init_method notif_iter_init;
-                               bt_component_class_source_notification_iterator_finalize_method notif_iter_finalize;
+                               bt_component_class_source_message_iterator_init_method msg_iter_init;
+                               bt_component_class_source_message_iterator_finalize_method msg_iter_finalize;
                        } source;
 
                        struct {
@@ -305,8 +312,8 @@ enum bt_plugin_status bt_plugin_so_init(
                                bt_component_class_filter_output_port_connected_method output_port_connected;
                                bt_component_class_filter_input_port_disconnected_method input_port_disconnected;
                                bt_component_class_filter_output_port_disconnected_method output_port_disconnected;
-                               bt_component_class_filter_notification_iterator_init_method notif_iter_init;
-                               bt_component_class_filter_notification_iterator_finalize_method notif_iter_finalize;
+                               bt_component_class_filter_message_iterator_init_method msg_iter_init;
+                               bt_component_class_filter_message_iterator_finalize_method msg_iter_finalize;
                        } filter;
 
                        struct {
@@ -613,29 +620,29 @@ enum bt_plugin_status bt_plugin_so_init(
                                        abort();
                                }
                                break;
-                       case BT_PLUGIN_COMPONENT_CLASS_DESCRIPTOR_ATTRIBUTE_TYPE_NOTIF_ITER_INIT_METHOD:
+                       case BT_PLUGIN_COMPONENT_CLASS_DESCRIPTOR_ATTRIBUTE_TYPE_MSG_ITER_INIT_METHOD:
                                switch (cc_type) {
                                case BT_COMPONENT_CLASS_TYPE_SOURCE:
-                                       cc_full_descr->methods.source.notif_iter_init =
-                                               cur_cc_descr_attr->value.source_notif_iter_init_method;
+                                       cc_full_descr->methods.source.msg_iter_init =
+                                               cur_cc_descr_attr->value.source_msg_iter_init_method;
                                        break;
                                case BT_COMPONENT_CLASS_TYPE_FILTER:
-                                       cc_full_descr->methods.filter.notif_iter_init =
-                                               cur_cc_descr_attr->value.filter_notif_iter_init_method;
+                                       cc_full_descr->methods.filter.msg_iter_init =
+                                               cur_cc_descr_attr->value.filter_msg_iter_init_method;
                                        break;
                                default:
                                        abort();
                                }
                                break;
-                       case BT_PLUGIN_COMPONENT_CLASS_DESCRIPTOR_ATTRIBUTE_TYPE_NOTIF_ITER_FINALIZE_METHOD:
+                       case BT_PLUGIN_COMPONENT_CLASS_DESCRIPTOR_ATTRIBUTE_TYPE_MSG_ITER_FINALIZE_METHOD:
                                switch (cc_type) {
                                case BT_COMPONENT_CLASS_TYPE_SOURCE:
-                                       cc_full_descr->methods.source.notif_iter_finalize =
-                                               cur_cc_descr_attr->value.source_notif_iter_finalize_method;
+                                       cc_full_descr->methods.source.msg_iter_finalize =
+                                               cur_cc_descr_attr->value.source_msg_iter_finalize_method;
                                        break;
                                case BT_COMPONENT_CLASS_TYPE_FILTER:
-                                       cc_full_descr->methods.filter.notif_iter_finalize =
-                                               cur_cc_descr_attr->value.filter_notif_iter_finalize_method;
+                                       cc_full_descr->methods.filter.msg_iter_finalize =
+                                               cur_cc_descr_attr->value.filter_msg_iter_finalize_method;
                                        break;
                                default:
                                        abort();
@@ -673,13 +680,15 @@ enum bt_plugin_status bt_plugin_so_init(
 
        /* Initialize plugin */
        if (spec->init) {
+               enum bt_plugin_init_status init_status;
                BT_LOGD_STR("Calling user's plugin initialization function.");
-               status = spec->init(plugin);
+               init_status = spec->init(plugin);
                BT_LOGD("User function returned: %s",
-                       bt_plugin_status_string(status));
+                       bt_plugin_init_status_string(init_status));
 
-               if (status < 0) {
+               if (init_status < 0) {
                        BT_LOGW_STR("User's plugin initialization function failed.");
+                       status = BT_PLUGIN_STATUS_ERROR;
                        goto end;
                }
        }
@@ -711,14 +720,14 @@ enum bt_plugin_status bt_plugin_so_init(
                case BT_COMPONENT_CLASS_TYPE_SOURCE:
                        src_comp_class = bt_component_class_source_create(
                                cc_full_descr->descriptor->name,
-                               cc_full_descr->descriptor->methods.source.notif_iter_next);
+                               cc_full_descr->descriptor->methods.source.msg_iter_next);
                        comp_class = bt_component_class_source_as_component_class(
                                src_comp_class);
                        break;
                case BT_COMPONENT_CLASS_TYPE_FILTER:
                        flt_comp_class = bt_component_class_filter_create(
                                cc_full_descr->descriptor->name,
-                               cc_full_descr->descriptor->methods.source.notif_iter_next);
+                               cc_full_descr->descriptor->methods.source.msg_iter_next);
                        comp_class = bt_component_class_filter_as_component_class(
                                flt_comp_class);
                        break;
@@ -851,24 +860,24 @@ enum bt_plugin_status bt_plugin_so_init(
                                }
                        }
 
-                       if (cc_full_descr->methods.source.notif_iter_init) {
-                               ret = bt_component_class_source_set_notification_iterator_init_method(
+                       if (cc_full_descr->methods.source.msg_iter_init) {
+                               ret = bt_component_class_source_set_message_iterator_init_method(
                                        src_comp_class,
-                                       cc_full_descr->methods.source.notif_iter_init);
+                                       cc_full_descr->methods.source.msg_iter_init);
                                if (ret) {
-                                       BT_LOGE_STR("Cannot set source component class's notification iterator initialization method.");
+                                       BT_LOGE_STR("Cannot set source component class's message iterator initialization method.");
                                        status = BT_PLUGIN_STATUS_ERROR;
                                        BT_OBJECT_PUT_REF_AND_RESET(src_comp_class);
                                        goto end;
                                }
                        }
 
-                       if (cc_full_descr->methods.source.notif_iter_finalize) {
-                               ret = bt_component_class_source_set_notification_iterator_finalize_method(
+                       if (cc_full_descr->methods.source.msg_iter_finalize) {
+                               ret = bt_component_class_source_set_message_iterator_finalize_method(
                                        src_comp_class,
-                                       cc_full_descr->methods.source.notif_iter_finalize);
+                                       cc_full_descr->methods.source.msg_iter_finalize);
                                if (ret) {
-                                       BT_LOGE_STR("Cannot set source component class's notification iterator finalization method.");
+                                       BT_LOGE_STR("Cannot set source component class's message iterator finalization method.");
                                        status = BT_PLUGIN_STATUS_ERROR;
                                        BT_OBJECT_PUT_REF_AND_RESET(src_comp_class);
                                        goto end;
@@ -985,24 +994,24 @@ enum bt_plugin_status bt_plugin_so_init(
                                }
                        }
 
-                       if (cc_full_descr->methods.filter.notif_iter_init) {
-                               ret = bt_component_class_filter_set_notification_iterator_init_method(
+                       if (cc_full_descr->methods.filter.msg_iter_init) {
+                               ret = bt_component_class_filter_set_message_iterator_init_method(
                                        flt_comp_class,
-                                       cc_full_descr->methods.filter.notif_iter_init);
+                                       cc_full_descr->methods.filter.msg_iter_init);
                                if (ret) {
-                                       BT_LOGE_STR("Cannot set filter component class's notification iterator initialization method.");
+                                       BT_LOGE_STR("Cannot set filter component class's message iterator initialization method.");
                                        status = BT_PLUGIN_STATUS_ERROR;
                                        BT_OBJECT_PUT_REF_AND_RESET(flt_comp_class);
                                        goto end;
                                }
                        }
 
-                       if (cc_full_descr->methods.filter.notif_iter_finalize) {
-                               ret = bt_component_class_filter_set_notification_iterator_finalize_method(
+                       if (cc_full_descr->methods.filter.msg_iter_finalize) {
+                               ret = bt_component_class_filter_set_message_iterator_finalize_method(
                                        flt_comp_class,
-                                       cc_full_descr->methods.filter.notif_iter_finalize);
+                                       cc_full_descr->methods.filter.msg_iter_finalize);
                                if (ret) {
-                                       BT_LOGE_STR("Cannot set filter component class's notification iterator finalization method.");
+                                       BT_LOGE_STR("Cannot set filter component class's message iterator finalization method.");
                                        status = BT_PLUGIN_STATUS_ERROR;
                                        BT_OBJECT_PUT_REF_AND_RESET(flt_comp_class);
                                        goto end;
@@ -1223,7 +1232,7 @@ struct bt_plugin_set *bt_plugin_so_create_all_from_sections(
                         * _trying_ to open this file as a compatible
                         * Babeltrace plugin: if it's not, it's not an
                         * error. And because this can be tried during
-                        * bt_plugin_create_all_from_dir(), it's not
+                        * bt_plugin_find_all_from_dir(), it's not
                         * even a warning.
                         */
                        BT_LOGD("Unknown ABI major version: abi-major=%d",
@@ -1250,7 +1259,7 @@ struct bt_plugin_set *bt_plugin_so_create_all_from_sections(
                         * _trying_ to open this file as a compatible
                         * Babeltrace plugin: if it's not, it's not an
                         * error. And because this can be tried during
-                        * bt_plugin_create_all_from_dir(), it's not
+                        * bt_plugin_find_all_from_dir(), it's not
                         * even a warning.
                         */
                        BT_LOGD_STR("Cannot initialize SO plugin object from sections.");
This page took 0.029203 seconds and 4 git commands to generate.