Values API: split into private and public APIs
[babeltrace.git] / cli / babeltrace-cfg-cli-args.c
index 1fb9add018139f4f8011fafd74571ff6666bf0b6..b844b4e429bcab1e1be1c7eedac2cb46b3829955 100644 (file)
@@ -35,6 +35,7 @@
 #include <babeltrace/babeltrace.h>
 #include <babeltrace/common-internal.h>
 #include <babeltrace/values.h>
+#include <babeltrace/private-values.h>
 #include <popt.h>
 #include <glib.h>
 #include <sys/types.h>
@@ -84,7 +85,7 @@ struct ini_parsing_state {
        GScanner *scanner;
 
        /* Output map value object being filled (owned by this) */
-       struct bt_value *params;
+       struct bt_private_value *params;
 
        /* Next expected FSM state */
        enum ini_parsing_fsm_state expecting;
@@ -241,7 +242,9 @@ int ini_handle_state(struct ini_parsing_state *state)
                        goto error;
                }
 
-               if (bt_value_map_has_entry(state->params, state->last_map_key)) {
+               if (bt_value_map_has_entry(
+                               bt_value_borrow_from_private(state->params),
+                               state->last_map_key)) {
                        g_string_append_printf(state->ini_error,
                                "Duplicate parameter key: `%s`\n",
                                state->last_map_key);
@@ -292,19 +295,22 @@ int ini_handle_state(struct ini_parsing_state *state)
                                goto error;
                        }
 
-                       value = bt_value_integer_create_init(
-                               (int64_t) int_val);
+                       value = bt_value_borrow_from_private(
+                               bt_private_value_integer_create_init(
+                                       (int64_t) int_val));
                        break;
                }
                case G_TOKEN_FLOAT:
                        /* Positive floating point number */
-                       value = bt_value_real_create_init(
-                               state->scanner->value.v_float);
+                       value = bt_value_borrow_from_private(
+                               bt_private_value_real_create_init(
+                                       state->scanner->value.v_float));
                        break;
                case G_TOKEN_STRING:
                        /* Quoted string */
-                       value = bt_value_string_create_init(
-                               state->scanner->value.v_string);
+                       value = bt_value_borrow_from_private(
+                               bt_private_value_string_create_init(
+                                       state->scanner->value.v_string));
                        break;
                case G_TOKEN_IDENTIFIER:
                {
@@ -326,14 +332,17 @@ int ini_handle_state(struct ini_parsing_state *state)
                        } else if (!strcmp(id, "true") || !strcmp(id, "TRUE") ||
                                        !strcmp(id, "yes") ||
                                        !strcmp(id, "YES")) {
-                               value = bt_value_bool_create_init(true);
+                               value = bt_value_borrow_from_private(
+                                       bt_private_value_bool_create_init(true));
                        } else if (!strcmp(id, "false") ||
                                        !strcmp(id, "FALSE") ||
                                        !strcmp(id, "no") ||
                                        !strcmp(id, "NO")) {
-                               value = bt_value_bool_create_init(false);
+                               value = bt_value_borrow_from_private(
+                                       bt_private_value_bool_create_init(false));
                        } else {
-                               value = bt_value_string_create_init(id);
+                               value = bt_value_borrow_from_private(
+                                       bt_private_value_string_create_init(id));
                        }
                        break;
                }
@@ -366,14 +375,16 @@ int ini_handle_state(struct ini_parsing_state *state)
                                goto error;
                        }
 
-                       value = bt_value_integer_create_init(
-                               -((int64_t) int_val));
+                       value = bt_value_borrow_from_private(
+                               bt_private_value_integer_create_init(
+                                       -((int64_t) int_val)));
                        break;
                }
                case G_TOKEN_FLOAT:
                        /* Negative floating point number */
-                       value = bt_value_real_create_init(
-                               -state->scanner->value.v_float);
+                       value = bt_value_borrow_from_private(
+                               bt_private_value_real_create_init(
+                                       -state->scanner->value.v_float));
                        break;
                default:
                        /* Unset value variable will trigger the error */
@@ -414,7 +425,7 @@ error:
 
 success:
        if (value) {
-               if (bt_value_map_insert_entry(state->params,
+               if (bt_private_value_map_insert_entry(state->params,
                                state->last_map_key, value)) {
                        /* Only override return value on error */
                        ret = -1;
@@ -432,7 +443,8 @@ end:
  * Return value is owned by the caller.
  */
 static
-struct bt_value *bt_value_from_ini(const char *arg, GString *ini_error)
+struct bt_private_value *bt_private_value_from_ini(const char *arg,
+               GString *ini_error)
 {
        /* Lexical scanner configuration */
        GScannerConfig scanner_config = {
@@ -511,7 +523,7 @@ struct bt_value *bt_value_from_ini(const char *arg, GString *ini_error)
                .ini_error = ini_error,
        };
 
-       state.params = bt_value_map_create();
+       state.params = bt_private_value_map_create();
        if (!state.params) {
                goto error;
        }
@@ -557,9 +569,9 @@ end:
  * Return value is owned by the caller.
  */
 static
-struct bt_value *bt_value_from_arg(const char *arg)
+struct bt_private_value *bt_private_value_from_arg(const char *arg)
 {
-       struct bt_value *params = NULL;
+       struct bt_private_value *params = NULL;
        GString *ini_error = NULL;
 
        ini_error = g_string_new(NULL);
@@ -569,7 +581,7 @@ struct bt_value *bt_value_from_arg(const char *arg)
        }
 
        /* Try INI-style parsing */
-       params = bt_value_from_ini(arg, ini_error);
+       params = bt_private_value_from_ini(arg, ini_error);
        if (!params) {
                printf_err("%s", ini_error->str);
                goto end;
@@ -579,6 +591,7 @@ end:
        if (ini_error) {
                g_string_free(ini_error, TRUE);
        }
+
        return params;
 }
 
@@ -811,7 +824,7 @@ struct bt_config_component *bt_config_component_create(
        }
 
        /* Start with empty parameters */
-       cfg_component->params = bt_value_map_create();
+       cfg_component->params = bt_private_value_map_create();
        if (!cfg_component->params) {
                print_err_oom();
                goto error;
@@ -1010,13 +1023,13 @@ GScanner *create_csv_identifiers_scanner(void)
  * Return value is owned by the caller.
  */
 static
-struct bt_value *names_from_arg(const char *arg)
+struct bt_private_value *names_from_arg(const char *arg)
 {
        GScanner *scanner = NULL;
-       struct bt_value *names = NULL;
+       struct bt_private_value *names = NULL;
        bool found_all = false, found_none = false, found_item = false;
 
-       names = bt_value_array_create();
+       names = bt_private_value_array_create();
        if (!names) {
                print_err_oom();
                goto error;
@@ -1041,33 +1054,33 @@ struct bt_value *names_from_arg(const char *arg)
                                        !strcmp(identifier, "args") ||
                                        !strcmp(identifier, "arg")) {
                                found_item = true;
-                               if (bt_value_array_append_string_element(names,
+                               if (bt_private_value_array_append_string_element(names,
                                                "payload")) {
                                        goto error;
                                }
                        } else if (!strcmp(identifier, "context") ||
                                        !strcmp(identifier, "ctx")) {
                                found_item = true;
-                               if (bt_value_array_append_string_element(names,
+                               if (bt_private_value_array_append_string_element(names,
                                                "context")) {
                                        goto error;
                                }
                        } else if (!strcmp(identifier, "scope") ||
                                        !strcmp(identifier, "header")) {
                                found_item = true;
-                               if (bt_value_array_append_string_element(names,
+                               if (bt_private_value_array_append_string_element(names,
                                                identifier)) {
                                        goto error;
                                }
                        } else if (!strcmp(identifier, "all")) {
                                found_all = true;
-                               if (bt_value_array_append_string_element(names,
+                               if (bt_private_value_array_append_string_element(names,
                                                identifier)) {
                                        goto error;
                                }
                        } else if (!strcmp(identifier, "none")) {
                                found_none = true;
-                               if (bt_value_array_append_string_element(names,
+                               if (bt_private_value_array_append_string_element(names,
                                                identifier)) {
                                        goto error;
                                }
@@ -1097,7 +1110,7 @@ end:
         * least one item is specified.
         */
        if (found_item && !found_none && !found_all) {
-               if (bt_value_array_append_string_element(names, "none")) {
+               if (bt_private_value_array_append_string_element(names, "none")) {
                        goto error;
                }
        }
@@ -1122,12 +1135,12 @@ error:
  * Return value is owned by the caller.
  */
 static
-struct bt_value *fields_from_arg(const char *arg)
+struct bt_private_value *fields_from_arg(const char *arg)
 {
        GScanner *scanner = NULL;
-       struct bt_value *fields;
+       struct bt_private_value *fields;
 
-       fields = bt_value_array_create();
+       fields = bt_private_value_array_create();
        if (!fields) {
                print_err_oom();
                goto error;
@@ -1157,7 +1170,7 @@ struct bt_value *fields_from_arg(const char *arg)
                                        !strcmp(identifier, "emf") ||
                                        !strcmp(identifier, "callsite") ||
                                        !strcmp(identifier, "all")) {
-                               if (bt_value_array_append_string_element(fields,
+                               if (bt_private_value_array_append_string_element(fields,
                                                identifier)) {
                                        goto error;
                                }
@@ -1385,7 +1398,7 @@ static
 int add_run_cfg_comp_check_name(struct bt_config *cfg,
                struct bt_config_component *cfg_comp,
                enum bt_config_component_dest dest,
-               struct bt_value *instance_names)
+               struct bt_private_value *instance_names)
 {
        int ret = 0;
 
@@ -1395,14 +1408,15 @@ int add_run_cfg_comp_check_name(struct bt_config *cfg,
                goto end;
        }
 
-       if (bt_value_map_has_entry(instance_names, cfg_comp->instance_name->str)) {
+       if (bt_value_map_has_entry(bt_value_borrow_from_private(instance_names),
+                       cfg_comp->instance_name->str)) {
                printf_err("Duplicate component instance name:\n    %s\n",
                        cfg_comp->instance_name->str);
                ret = -1;
                goto end;
        }
 
-       if (bt_value_map_insert_entry(instance_names,
+       if (bt_private_value_map_insert_entry(instance_names,
                        cfg_comp->instance_name->str, bt_value_null)) {
                print_err_oom();
                ret = -1;
@@ -1416,7 +1430,7 @@ end:
 }
 
 static
-int append_env_var_plugin_paths(struct bt_value *plugin_paths)
+int append_env_var_plugin_paths(struct bt_private_value *plugin_paths)
 {
        int ret = 0;
        const char *envvar;
@@ -1442,7 +1456,7 @@ end:
 }
 
 static
-int append_home_and_system_plugin_paths(struct bt_value *plugin_paths,
+int append_home_and_system_plugin_paths(struct bt_private_value *plugin_paths,
                bool omit_system_plugin_path, bool omit_home_plugin_path)
 {
        int ret;
@@ -1489,7 +1503,8 @@ int append_home_and_system_plugin_paths_cfg(struct bt_config *cfg)
 
 static
 struct bt_config *bt_config_base_create(enum bt_config_command command,
-               struct bt_value *initial_plugin_paths, bool needs_plugins)
+               struct bt_private_value *initial_plugin_paths,
+               bool needs_plugins)
 {
        struct bt_config *cfg;
 
@@ -1507,7 +1522,7 @@ struct bt_config *bt_config_base_create(enum bt_config_command command,
        if (initial_plugin_paths) {
                cfg->plugin_paths = bt_object_get_ref(initial_plugin_paths);
        } else {
-               cfg->plugin_paths = bt_value_array_create();
+               cfg->plugin_paths = bt_private_value_array_create();
                if (!cfg->plugin_paths) {
                        print_err_oom();
                        goto error;
@@ -1525,7 +1540,7 @@ end:
 
 static
 struct bt_config *bt_config_run_create(
-               struct bt_value *initial_plugin_paths)
+               struct bt_private_value *initial_plugin_paths)
 {
        struct bt_config *cfg;
 
@@ -1575,7 +1590,7 @@ end:
 
 static
 struct bt_config *bt_config_list_plugins_create(
-               struct bt_value *initial_plugin_paths)
+               struct bt_private_value *initial_plugin_paths)
 {
        struct bt_config *cfg;
 
@@ -1597,7 +1612,7 @@ end:
 
 static
 struct bt_config *bt_config_help_create(
-               struct bt_value *initial_plugin_paths)
+               struct bt_private_value *initial_plugin_paths)
 {
        struct bt_config *cfg;
 
@@ -1626,7 +1641,7 @@ end:
 
 static
 struct bt_config *bt_config_query_create(
-               struct bt_value *initial_plugin_paths)
+               struct bt_private_value *initial_plugin_paths)
 {
        struct bt_config *cfg;
 
@@ -1654,7 +1669,7 @@ end:
 
 static
 struct bt_config *bt_config_print_ctf_metadata_create(
-               struct bt_value *initial_plugin_paths)
+               struct bt_private_value *initial_plugin_paths)
 {
        struct bt_config *cfg;
 
@@ -1688,7 +1703,7 @@ end:
 
 static
 struct bt_config *bt_config_print_lttng_live_sessions_create(
-               struct bt_value *initial_plugin_paths)
+               struct bt_private_value *initial_plugin_paths)
 {
        struct bt_config *cfg;
 
@@ -1723,7 +1738,7 @@ end:
 
 static
 int bt_config_append_plugin_paths_check_setuid_setgid(
-               struct bt_value *plugin_paths, const char *arg)
+               struct bt_private_value *plugin_paths, const char *arg)
 {
        int ret = 0;
 
@@ -1824,7 +1839,7 @@ static
 struct bt_config *bt_config_help_from_args(int argc, const char *argv[],
                int *retcode, bool force_omit_system_plugin_path,
                bool force_omit_home_plugin_path,
-               struct bt_value *initial_plugin_paths)
+               struct bt_private_value *initial_plugin_paths)
 {
        poptContext pc = NULL;
        char *arg = NULL;
@@ -1988,7 +2003,7 @@ static
 struct bt_config *bt_config_query_from_args(int argc, const char *argv[],
                int *retcode, bool force_omit_system_plugin_path,
                bool force_omit_home_plugin_path,
-               struct bt_value *initial_plugin_paths)
+               struct bt_private_value *initial_plugin_paths)
 {
        poptContext pc = NULL;
        char *arg = NULL;
@@ -1996,7 +2011,7 @@ struct bt_config *bt_config_query_from_args(int argc, const char *argv[],
        int ret;
        struct bt_config *cfg = NULL;
        const char *leftover;
-       struct bt_value *params = bt_value_null;
+       struct bt_private_value *params = bt_private_value_null;
 
        *retcode = 0;
        cfg = bt_config_query_create(initial_plugin_paths);
@@ -2040,7 +2055,7 @@ struct bt_config *bt_config_query_from_args(int argc, const char *argv[],
                case OPT_PARAMS:
                {
                        bt_object_put_ref(params);
-                       params = bt_value_from_arg(arg);
+                       params = bt_private_value_from_arg(arg);
                        if (!params) {
                                printf_err("Invalid format for --params option's argument:\n    %s\n",
                                        arg);
@@ -2085,7 +2100,8 @@ struct bt_config *bt_config_query_from_args(int argc, const char *argv[],
                }
 
                BT_ASSERT(params);
-               BT_OBJECT_MOVE_REF(cfg->cmd_data.query.cfg_component->params, params);
+               BT_OBJECT_MOVE_REF(cfg->cmd_data.query.cfg_component->params,
+                       params);
        } else {
                print_query_usage(stdout);
                *retcode = -1;
@@ -2176,7 +2192,7 @@ static
 struct bt_config *bt_config_list_plugins_from_args(int argc, const char *argv[],
                int *retcode, bool force_omit_system_plugin_path,
                bool force_omit_home_plugin_path,
-               struct bt_value *initial_plugin_paths)
+               struct bt_private_value *initial_plugin_paths)
 {
        poptContext pc = NULL;
        char *arg = NULL;
@@ -2366,18 +2382,18 @@ static
 struct bt_config *bt_config_run_from_args(int argc, const char *argv[],
                int *retcode, bool force_omit_system_plugin_path,
                bool force_omit_home_plugin_path,
-               struct bt_value *initial_plugin_paths)
+               struct bt_private_value *initial_plugin_paths)
 {
        poptContext pc = NULL;
        char *arg = NULL;
        struct bt_config_component *cur_cfg_comp = NULL;
        enum bt_config_component_dest cur_cfg_comp_dest =
                        BT_CONFIG_COMPONENT_DEST_UNKNOWN;
-       struct bt_value *cur_base_params = NULL;
+       struct bt_private_value *cur_base_params = NULL;
        int opt, ret = 0;
        struct bt_config *cfg = NULL;
-       struct bt_value *instance_names = NULL;
-       struct bt_value *connection_args = NULL;
+       struct bt_private_value *instance_names = NULL;
+       struct bt_private_value *connection_args = NULL;
        GString *cur_param_key = NULL;
        char error_buf[256] = { 0 };
        long retry_duration = -1;
@@ -2419,19 +2435,19 @@ struct bt_config *bt_config_run_from_args(int argc, const char *argv[],
        cfg->cmd_data.run.retry_duration_us = 100000;
        cfg->omit_system_plugin_path = force_omit_system_plugin_path;
        cfg->omit_home_plugin_path = force_omit_home_plugin_path;
-       cur_base_params = bt_value_map_create();
+       cur_base_params = bt_private_value_map_create();
        if (!cur_base_params) {
                print_err_oom();
                goto error;
        }
 
-       instance_names = bt_value_map_create();
+       instance_names = bt_private_value_map_create();
        if (!instance_names) {
                print_err_oom();
                goto error;
        }
 
-       connection_args = bt_value_array_create();
+       connection_args = bt_private_value_array_create();
        if (!connection_args) {
                print_err_oom();
                goto error;
@@ -2505,7 +2521,8 @@ struct bt_config *bt_config_run_from_args(int argc, const char *argv[],
 
                        BT_ASSERT(cur_base_params);
                        bt_object_put_ref(cur_cfg_comp->params);
-                       cur_cfg_comp->params = bt_value_copy(cur_base_params);
+                       cur_cfg_comp->params = bt_value_copy(
+                               bt_value_borrow_from_private(cur_base_params));
                        if (!cur_cfg_comp->params) {
                                print_err_oom();
                                goto error;
@@ -2516,8 +2533,8 @@ struct bt_config *bt_config_run_from_args(int argc, const char *argv[],
                }
                case OPT_PARAMS:
                {
-                       struct bt_value *params;
-                       struct bt_value *params_to_set;
+                       struct bt_private_value *params;
+                       struct bt_private_value *params_to_set;
 
                        if (!cur_cfg_comp) {
                                printf_err("Cannot add parameters to unavailable component:\n    %s\n",
@@ -2525,15 +2542,16 @@ struct bt_config *bt_config_run_from_args(int argc, const char *argv[],
                                goto error;
                        }
 
-                       params = bt_value_from_arg(arg);
+                       params = bt_private_value_from_arg(arg);
                        if (!params) {
                                printf_err("Invalid format for --params option's argument:\n    %s\n",
                                        arg);
                                goto error;
                        }
 
-                       params_to_set = bt_value_map_extend(cur_cfg_comp->params,
-                               params);
+                       params_to_set = bt_value_map_extend(
+                               bt_value_borrow_from_private(cur_cfg_comp->params),
+                               bt_value_borrow_from_private(params));
                        BT_OBJECT_PUT_REF_AND_RESET(params);
                        if (!params_to_set) {
                                printf_err("Cannot extend current component parameters with --params option's argument:\n    %s\n",
@@ -2565,7 +2583,7 @@ struct bt_config *bt_config_run_from_args(int argc, const char *argv[],
                                goto error;
                        }
 
-                       if (bt_value_map_insert_string_entry(cur_cfg_comp->params,
+                       if (bt_private_value_map_insert_string_entry(cur_cfg_comp->params,
                                        cur_param_key->str, arg)) {
                                print_err_oom();
                                goto error;
@@ -2582,7 +2600,8 @@ struct bt_config *bt_config_run_from_args(int argc, const char *argv[],
                        break;
                case OPT_BASE_PARAMS:
                {
-                       struct bt_value *params = bt_value_from_arg(arg);
+                       struct bt_private_value *params =
+                               bt_private_value_from_arg(arg);
 
                        if (!params) {
                                printf_err("Invalid format for --base-params option's argument:\n    %s\n",
@@ -2595,14 +2614,14 @@ struct bt_config *bt_config_run_from_args(int argc, const char *argv[],
                }
                case OPT_RESET_BASE_PARAMS:
                        BT_OBJECT_PUT_REF_AND_RESET(cur_base_params);
-                       cur_base_params = bt_value_map_create();
+                       cur_base_params = bt_private_value_map_create();
                        if (!cur_base_params) {
                                print_err_oom();
                                goto error;
                        }
                        break;
                case OPT_CONNECT:
-                       if (bt_value_array_append_string_element(
+                       if (bt_private_value_array_append_string_element(
                                        connection_args, arg)) {
                                print_err_oom();
                                goto error;
@@ -2670,7 +2689,8 @@ struct bt_config *bt_config_run_from_args(int argc, const char *argv[],
                goto error;
        }
 
-       ret = bt_config_cli_args_create_connections(cfg, connection_args,
+       ret = bt_config_cli_args_create_connections(cfg,
+               bt_value_borrow_from_private(connection_args),
                error_buf, 256);
        if (ret) {
                printf_err("Cannot creation connections:\n%s", error_buf);
@@ -2704,7 +2724,7 @@ static
 struct bt_config *bt_config_run_from_args_array(struct bt_value *run_args,
                int *retcode, bool force_omit_system_plugin_path,
                bool force_omit_home_plugin_path,
-               struct bt_value *initial_plugin_paths)
+               struct bt_private_value *initial_plugin_paths)
 {
        struct bt_config *cfg = NULL;
        const char **argv;
@@ -2955,12 +2975,12 @@ struct implicit_component_args {
        GString *comp_arg;
        GString *name_arg;
        GString *params_arg;
-       struct bt_value *extra_params;
+       struct bt_private_value *extra_params;
 };
 
 static
 int assign_name_to_implicit_component(struct implicit_component_args *args,
-               const char *prefix, struct bt_value *existing_names,
+               const char *prefix, struct bt_private_value *existing_names,
                GList **comp_names, bool append_to_comp_names)
 {
        int ret = 0;
@@ -2970,7 +2990,8 @@ int assign_name_to_implicit_component(struct implicit_component_args *args,
                goto end;
        }
 
-       name = get_component_auto_name(prefix, existing_names);
+       name = get_component_auto_name(prefix,
+               bt_value_borrow_from_private(existing_names));
 
        if (!name) {
                ret = -1;
@@ -2979,7 +3000,7 @@ int assign_name_to_implicit_component(struct implicit_component_args *args,
 
        g_string_assign(args->name_arg, name->str);
 
-       if (bt_value_map_insert_entry(existing_names, name->str,
+       if (bt_private_value_map_insert_entry(existing_names, name->str,
                        bt_value_null)) {
                print_err_oom();
                ret = -1;
@@ -3002,7 +3023,7 @@ end:
 static
 int append_run_args_for_implicit_component(
                struct implicit_component_args *impl_args,
-               struct bt_value *run_args)
+               struct bt_private_value *run_args)
 {
        int ret = 0;
        size_t i;
@@ -3011,45 +3032,48 @@ int append_run_args_for_implicit_component(
                goto end;
        }
 
-       if (bt_value_array_append_string_element(run_args, "--component")) {
+       if (bt_private_value_array_append_string_element(run_args, "--component")) {
                print_err_oom();
                goto error;
        }
 
-       if (bt_value_array_append_string_element(run_args, impl_args->comp_arg->str)) {
+       if (bt_private_value_array_append_string_element(run_args, impl_args->comp_arg->str)) {
                print_err_oom();
                goto error;
        }
 
-       if (bt_value_array_append_string_element(run_args, "--name")) {
+       if (bt_private_value_array_append_string_element(run_args, "--name")) {
                print_err_oom();
                goto error;
        }
 
-       if (bt_value_array_append_string_element(run_args, impl_args->name_arg->str)) {
+       if (bt_private_value_array_append_string_element(run_args, impl_args->name_arg->str)) {
                print_err_oom();
                goto error;
        }
 
        if (impl_args->params_arg->len > 0) {
-               if (bt_value_array_append_string_element(run_args, "--params")) {
+               if (bt_private_value_array_append_string_element(run_args, "--params")) {
                        print_err_oom();
                        goto error;
                }
 
-               if (bt_value_array_append_string_element(run_args,
+               if (bt_private_value_array_append_string_element(run_args,
                                impl_args->params_arg->str)) {
                        print_err_oom();
                        goto error;
                }
        }
 
-       for (i = 0; i < bt_value_array_get_size(impl_args->extra_params); i++) {
+       for (i = 0; i < bt_value_array_get_size(
+                       bt_value_borrow_from_private(impl_args->extra_params));
+                       i++) {
                struct bt_value *elem;
                const char *arg;
 
                elem = bt_value_array_borrow_element_by_index(
-                       impl_args->extra_params, i);
+                       bt_value_borrow_from_private(impl_args->extra_params),
+                       i);
                if (!elem) {
                        goto error;
                }
@@ -3059,7 +3083,7 @@ int append_run_args_for_implicit_component(
                        goto error;
                }
 
-               ret = bt_value_array_append_string_element(run_args, arg);
+               ret = bt_private_value_array_append_string_element(run_args, arg);
                if (ret) {
                        print_err_oom();
                        goto error;
@@ -3116,7 +3140,7 @@ int init_implicit_component_args(struct implicit_component_args *args,
        args->comp_arg = g_string_new(comp_arg);
        args->name_arg = g_string_new(NULL);
        args->params_arg = g_string_new(NULL);
-       args->extra_params = bt_value_array_create();
+       args->extra_params = bt_private_value_array_create();
 
        if (!args->comp_arg || !args->name_arg ||
                        !args->params_arg || !args->extra_params) {
@@ -3150,25 +3174,25 @@ int append_implicit_component_extra_param(struct implicit_component_args *args,
        BT_ASSERT(key);
        BT_ASSERT(value);
 
-       if (bt_value_array_append_string_element(args->extra_params, "--key")) {
+       if (bt_private_value_array_append_string_element(args->extra_params, "--key")) {
                print_err_oom();
                ret = -1;
                goto end;
        }
 
-       if (bt_value_array_append_string_element(args->extra_params, key)) {
+       if (bt_private_value_array_append_string_element(args->extra_params, key)) {
                print_err_oom();
                ret = -1;
                goto end;
        }
 
-       if (bt_value_array_append_string_element(args->extra_params, "--value")) {
+       if (bt_private_value_array_append_string_element(args->extra_params, "--value")) {
                print_err_oom();
                ret = -1;
                goto end;
        }
 
-       if (bt_value_array_append_string_element(args->extra_params, value)) {
+       if (bt_private_value_array_append_string_element(args->extra_params, value)) {
                print_err_oom();
                ret = -1;
                goto end;
@@ -3181,7 +3205,8 @@ end:
 static
 int convert_append_name_param(enum bt_config_component_dest dest,
                GString *cur_name, GString *cur_name_prefix,
-               struct bt_value *run_args, struct bt_value *all_names,
+               struct bt_private_value *run_args,
+               struct bt_private_value *all_names,
                GList **source_names, GList **filter_names,
                GList **sink_names)
 {
@@ -3198,14 +3223,15 @@ int convert_append_name_param(enum bt_config_component_dest dest,
                         * component.
                         */
                        name = get_component_auto_name(cur_name_prefix->str,
-                               all_names);
+                               bt_value_borrow_from_private(all_names));
                        append_name_opt = true;
                } else {
                        /*
                         * An explicit name was provided for the user
                         * component.
                         */
-                       if (bt_value_map_has_entry(all_names,
+                       if (bt_value_map_has_entry(
+                                       bt_value_borrow_from_private(all_names),
                                        cur_name->str)) {
                                printf_err("Duplicate component instance name:\n    %s\n",
                                        cur_name->str);
@@ -3224,7 +3250,7 @@ int convert_append_name_param(enum bt_config_component_dest dest,
                 * Remember this name globally, for the uniqueness of
                 * all component names.
                 */
-               if (bt_value_map_insert_entry(all_names, name->str, bt_value_null)) {
+               if (bt_private_value_map_insert_entry(all_names, name->str, bt_value_null)) {
                        print_err_oom();
                        goto error;
                }
@@ -3233,12 +3259,12 @@ int convert_append_name_param(enum bt_config_component_dest dest,
                 * Append the --name option if necessary.
                 */
                if (append_name_opt) {
-                       if (bt_value_array_append_string_element(run_args, "--name")) {
+                       if (bt_private_value_array_append_string_element(run_args, "--name")) {
                                print_err_oom();
                                goto error;
                        }
 
-                       if (bt_value_array_append_string_element(run_args, name->str)) {
+                       if (bt_private_value_array_append_string_element(run_args, name->str)) {
                                print_err_oom();
                                goto error;
                        }
@@ -3306,7 +3332,7 @@ end:
  * function.
  */
 static
-int append_connect_arg(struct bt_value *run_args,
+int append_connect_arg(struct bt_private_value *run_args,
                const char *upstream_name, const char *downstream_name)
 {
        int ret = 0;
@@ -3320,7 +3346,7 @@ int append_connect_arg(struct bt_value *run_args,
                goto end;
        }
 
-       ret = bt_value_array_append_string_element(run_args, "--connect");
+       ret = bt_private_value_array_append_string_element(run_args, "--connect");
        if (ret) {
                print_err_oom();
                ret = -1;
@@ -3330,7 +3356,7 @@ int append_connect_arg(struct bt_value *run_args,
        g_string_append(arg, e_upstream_name->str);
        g_string_append_c(arg, ':');
        g_string_append(arg, e_downstream_name->str);
-       ret = bt_value_array_append_string_element(run_args, arg->str);
+       ret = bt_private_value_array_append_string_element(run_args, arg->str);
        if (ret) {
                print_err_oom();
                ret = -1;
@@ -3357,7 +3383,7 @@ end:
  * Appends the run command's --connect options for the convert command.
  */
 static
-int convert_auto_connect(struct bt_value *run_args,
+int convert_auto_connect(struct bt_private_value *run_args,
                GList *source_names, GList *filter_names,
                GList *sink_names)
 {
@@ -3541,7 +3567,8 @@ int fill_implicit_ctf_inputs_args(GPtrArray *implicit_ctf_inputs_args,
                 */
                BT_OBJECT_PUT_REF_AND_RESET(impl_args->extra_params);
                impl_args->extra_params =
-                       bt_value_copy(base_implicit_ctf_input_args->extra_params);
+                       bt_value_copy(bt_value_borrow_from_private(
+                               base_implicit_ctf_input_args->extra_params));
                if (!impl_args->extra_params) {
                        print_err_oom();
                        destroy_implicit_component_args(impl_args);
@@ -3578,7 +3605,7 @@ static
 struct bt_config *bt_config_convert_from_args(int argc, const char *argv[],
                int *retcode, bool force_omit_system_plugin_path,
                bool force_omit_home_plugin_path,
-               struct bt_value *initial_plugin_paths, char *log_level)
+               struct bt_private_value *initial_plugin_paths, char *log_level)
 {
        poptContext pc = NULL;
        char *arg = NULL;
@@ -3597,8 +3624,8 @@ struct bt_config *bt_config_convert_from_args(int argc, const char *argv[],
        bool print_run_args = false;
        bool print_run_args_0 = false;
        bool print_ctf_metadata = false;
-       struct bt_value *run_args = NULL;
-       struct bt_value *all_names = NULL;
+       struct bt_private_value *run_args = NULL;
+       struct bt_private_value *all_names = NULL;
        GList *source_names = NULL;
        GList *filter_names = NULL;
        GList *sink_names = NULL;
@@ -3612,7 +3639,8 @@ struct bt_config *bt_config_convert_from_args(int argc, const char *argv[],
        struct implicit_component_args implicit_debug_info_args = { 0 };
        struct implicit_component_args implicit_muxer_args = { 0 };
        struct implicit_component_args implicit_trimmer_args = { 0 };
-       struct bt_value *plugin_paths = bt_object_get_ref(initial_plugin_paths);
+       struct bt_private_value *plugin_paths =
+               bt_object_get_ref(initial_plugin_paths);
        char error_buf[256] = { 0 };
        size_t i;
        struct bt_common_lttng_live_url_parts lttng_live_url_parts = { 0 };
@@ -3673,13 +3701,13 @@ struct bt_config *bt_config_convert_from_args(int argc, const char *argv[],
                goto error;
        }
 
-       all_names = bt_value_map_create();
+       all_names = bt_private_value_map_create();
        if (!all_names) {
                print_err_oom();
                goto error;
        }
 
-       run_args = bt_value_array_create();
+       run_args = bt_private_value_array_create();
        if (!run_args) {
                print_err_oom();
                goto error;
@@ -3776,13 +3804,13 @@ struct bt_config *bt_config_convert_from_args(int argc, const char *argv[],
                                abort();
                        }
 
-                       if (bt_value_array_append_string_element(run_args,
+                       if (bt_private_value_array_append_string_element(run_args,
                                        "--component")) {
                                print_err_oom();
                                goto error;
                        }
 
-                       if (bt_value_array_append_string_element(run_args, arg)) {
+                       if (bt_private_value_array_append_string_element(run_args, arg)) {
                                print_err_oom();
                                goto error;
                        }
@@ -3805,13 +3833,13 @@ struct bt_config *bt_config_convert_from_args(int argc, const char *argv[],
                                goto error;
                        }
 
-                       if (bt_value_array_append_string_element(run_args,
+                       if (bt_private_value_array_append_string_element(run_args,
                                        "--params")) {
                                print_err_oom();
                                goto error;
                        }
 
-                       if (bt_value_array_append_string_element(run_args, arg)) {
+                       if (bt_private_value_array_append_string_element(run_args, arg)) {
                                print_err_oom();
                                goto error;
                        }
@@ -3823,22 +3851,22 @@ struct bt_config *bt_config_convert_from_args(int argc, const char *argv[],
                                goto error;
                        }
 
-                       if (bt_value_array_append_string_element(run_args, "--key")) {
+                       if (bt_private_value_array_append_string_element(run_args, "--key")) {
                                print_err_oom();
                                goto error;
                        }
 
-                       if (bt_value_array_append_string_element(run_args, "path")) {
+                       if (bt_private_value_array_append_string_element(run_args, "path")) {
                                print_err_oom();
                                goto error;
                        }
 
-                       if (bt_value_array_append_string_element(run_args, "--value")) {
+                       if (bt_private_value_array_append_string_element(run_args, "--value")) {
                                print_err_oom();
                                goto error;
                        }
 
-                       if (bt_value_array_append_string_element(run_args, arg)) {
+                       if (bt_private_value_array_append_string_element(run_args, arg)) {
                                print_err_oom();
                                goto error;
                        }
@@ -3850,22 +3878,22 @@ struct bt_config *bt_config_convert_from_args(int argc, const char *argv[],
                                goto error;
                        }
 
-                       if (bt_value_array_append_string_element(run_args, "--key")) {
+                       if (bt_private_value_array_append_string_element(run_args, "--key")) {
                                print_err_oom();
                                goto error;
                        }
 
-                       if (bt_value_array_append_string_element(run_args, "url")) {
+                       if (bt_private_value_array_append_string_element(run_args, "url")) {
                                print_err_oom();
                                goto error;
                        }
 
-                       if (bt_value_array_append_string_element(run_args, "--value")) {
+                       if (bt_private_value_array_append_string_element(run_args, "--value")) {
                                print_err_oom();
                                goto error;
                        }
 
-                       if (bt_value_array_append_string_element(run_args, arg)) {
+                       if (bt_private_value_array_append_string_element(run_args, arg)) {
                                print_err_oom();
                                goto error;
                        }
@@ -3877,12 +3905,12 @@ struct bt_config *bt_config_convert_from_args(int argc, const char *argv[],
                                goto error;
                        }
 
-                       if (bt_value_array_append_string_element(run_args, "--name")) {
+                       if (bt_private_value_array_append_string_element(run_args, "--name")) {
                                print_err_oom();
                                goto error;
                        }
 
-                       if (bt_value_array_append_string_element(run_args, arg)) {
+                       if (bt_private_value_array_append_string_element(run_args, arg)) {
                                print_err_oom();
                                goto error;
                        }
@@ -3892,20 +3920,20 @@ struct bt_config *bt_config_convert_from_args(int argc, const char *argv[],
                case OPT_OMIT_HOME_PLUGIN_PATH:
                        force_omit_home_plugin_path = true;
 
-                       if (bt_value_array_append_string_element(run_args,
+                       if (bt_private_value_array_append_string_element(run_args,
                                        "--omit-home-plugin-path")) {
                                print_err_oom();
                                goto error;
                        }
                        break;
                case OPT_RETRY_DURATION:
-                       if (bt_value_array_append_string_element(run_args,
+                       if (bt_private_value_array_append_string_element(run_args,
                                        "--retry-duration")) {
                                print_err_oom();
                                goto error;
                        }
 
-                       if (bt_value_array_append_string_element(run_args, arg)) {
+                       if (bt_private_value_array_append_string_element(run_args, arg)) {
                                print_err_oom();
                                goto error;
                        }
@@ -3913,7 +3941,7 @@ struct bt_config *bt_config_convert_from_args(int argc, const char *argv[],
                case OPT_OMIT_SYSTEM_PLUGIN_PATH:
                        force_omit_system_plugin_path = true;
 
-                       if (bt_value_array_append_string_element(run_args,
+                       if (bt_private_value_array_append_string_element(run_args,
                                        "--omit-system-plugin-path")) {
                                print_err_oom();
                                goto error;
@@ -3925,13 +3953,13 @@ struct bt_config *bt_config_convert_from_args(int argc, const char *argv[],
                                goto error;
                        }
 
-                       if (bt_value_array_append_string_element(run_args,
+                       if (bt_private_value_array_append_string_element(run_args,
                                        "--plugin-path")) {
                                print_err_oom();
                                goto error;
                        }
 
-                       if (bt_value_array_append_string_element(run_args, arg)) {
+                       if (bt_private_value_array_append_string_element(run_args, arg)) {
                                print_err_oom();
                                goto error;
                        }
@@ -4150,7 +4178,7 @@ struct bt_config *bt_config_convert_from_args(int argc, const char *argv[],
                        break;
                case OPT_FIELDS:
                {
-                       struct bt_value *fields = fields_from_arg(arg);
+                       struct bt_private_value *fields = fields_from_arg(arg);
 
                        if (!fields) {
                                goto error;
@@ -4159,7 +4187,7 @@ struct bt_config *bt_config_convert_from_args(int argc, const char *argv[],
                        implicit_text_args.exists = true;
                        ret = insert_flat_params_from_array(
                                implicit_text_args.params_arg,
-                               fields, "field");
+                               bt_value_borrow_from_private(fields), "field");
                        bt_object_put_ref(fields);
                        if (ret) {
                                goto error;
@@ -4168,7 +4196,7 @@ struct bt_config *bt_config_convert_from_args(int argc, const char *argv[],
                }
                case OPT_NAMES:
                {
-                       struct bt_value *names = names_from_arg(arg);
+                       struct bt_private_value *names = names_from_arg(arg);
 
                        if (!names) {
                                goto error;
@@ -4177,7 +4205,7 @@ struct bt_config *bt_config_convert_from_args(int argc, const char *argv[],
                        implicit_text_args.exists = true;
                        ret = insert_flat_params_from_array(
                                implicit_text_args.params_arg,
-                               names, "name");
+                               bt_value_borrow_from_private(names), "name");
                        bt_object_put_ref(names);
                        if (ret) {
                                goto error;
@@ -4667,10 +4695,12 @@ struct bt_config *bt_config_convert_from_args(int argc, const char *argv[],
                        goto error;
                }
 
-               for (i = 0; i < bt_value_array_get_size(run_args); i++) {
+               for (i = 0; i < bt_value_array_get_size(
+                               bt_value_borrow_from_private(run_args)); i++) {
                        struct bt_value *arg_value =
                                bt_value_array_borrow_element_by_index(
-                                       run_args, i);
+                                       bt_value_borrow_from_private(run_args),
+                                       i);
                        const char *arg;
                        GString *quoted = NULL;
                        const char *arg_to_print;
@@ -4696,7 +4726,8 @@ struct bt_config *bt_config_convert_from_args(int argc, const char *argv[],
                                g_string_free(quoted, TRUE);
                        }
 
-                       if (i < bt_value_array_get_size(run_args) - 1) {
+                       if (i < bt_value_array_get_size(
+                                       bt_value_borrow_from_private(run_args)) - 1) {
                                if (print_run_args) {
                                        putchar(' ');
                                } else {
@@ -4710,7 +4741,8 @@ struct bt_config *bt_config_convert_from_args(int argc, const char *argv[],
                goto end;
        }
 
-       cfg = bt_config_run_from_args_array(run_args, retcode,
+       cfg = bt_config_run_from_args_array(
+               bt_value_borrow_from_private(run_args), retcode,
                force_omit_system_plugin_path, force_omit_home_plugin_path,
                initial_plugin_paths);
        if (!cfg) {
@@ -4826,7 +4858,7 @@ char log_level_from_arg(const char *arg)
 struct bt_config *bt_config_cli_args_create(int argc, const char *argv[],
                int *retcode, bool force_omit_system_plugin_path,
                bool force_omit_home_plugin_path,
-               struct bt_value *initial_plugin_paths)
+               struct bt_private_value *initial_plugin_paths)
 {
        struct bt_config *config = NULL;
        int i;
@@ -4847,7 +4879,7 @@ struct bt_config *bt_config_cli_args_create(int argc, const char *argv[],
        *retcode = -1;
 
        if (!initial_plugin_paths) {
-               initial_plugin_paths = bt_value_array_create();
+               initial_plugin_paths = bt_private_value_array_create();
                if (!initial_plugin_paths) {
                        *retcode = 1;
                        goto end;
This page took 0.042178 seconds and 4 git commands to generate.