static
void destroy_pretty_data(struct pretty_component *pretty)
{
- bt_put(pretty->input_iterator);
+ bt_object_put_ref(pretty->input_iterator);
if (pretty->string) {
(void) g_string_free(pretty->string, TRUE);
BT_ASSERT(pretty);
switch (bt_notification_get_type(notification)) {
+ case BT_NOTIFICATION_TYPE_PACKET_BEGIN:
+ ret = pretty_print_packet(pretty, notification);
+ break;
case BT_NOTIFICATION_TYPE_EVENT:
ret = pretty_print_event(pretty, notification);
break;
case BT_NOTIFICATION_TYPE_INACTIVITY:
fprintf(stderr, "Inactivity notification\n");
break;
- case BT_NOTIFICATION_TYPE_DISCARDED_PACKETS:
- case BT_NOTIFICATION_TYPE_DISCARDED_EVENTS:
- ret = pretty_print_discarded_elements(pretty, notification);
- break;
default:
break;
}
}
BT_HIDDEN
-void pretty_port_connected(
+enum bt_component_status pretty_port_connected(
struct bt_private_component *component,
struct bt_private_port *self_port,
struct bt_port *other_port)
{
+ enum bt_component_status status = BT_COMPONENT_STATUS_OK;
enum bt_connection_status conn_status;
struct bt_private_connection *connection;
struct pretty_component *pretty;
conn_status = bt_private_connection_create_notification_iterator(
connection, &pretty->input_iterator);
if (conn_status != BT_CONNECTION_STATUS_OK) {
- pretty->error = true;
+ status = BT_COMPONENT_STATUS_ERROR;
}
- bt_put(connection);
+ bt_object_put_ref(connection);
+ return status;
}
BT_HIDDEN
enum bt_component_status pretty_consume(struct bt_private_component *component)
{
enum bt_component_status ret;
- struct bt_notification *notif = NULL;
+ bt_notification_array notifs;
struct bt_notification_iterator *it;
struct pretty_component *pretty =
bt_private_component_get_user_data(component);
enum bt_notification_iterator_status it_ret;
-
- if (unlikely(pretty->error)) {
- ret = BT_COMPONENT_STATUS_ERROR;
- goto end;
- }
+ uint64_t count = 0;
+ uint64_t i = 0;
it = pretty->input_iterator;
- it_ret = bt_private_connection_notification_iterator_next(it, ¬if);
+ it_ret = bt_private_connection_notification_iterator_next(it, ¬ifs,
+ &count);
switch (it_ret) {
case BT_NOTIFICATION_ITERATOR_STATUS_END:
ret = BT_COMPONENT_STATUS_END;
- BT_PUT(pretty->input_iterator);
+ BT_OBJECT_PUT_REF_AND_RESET(pretty->input_iterator);
goto end;
case BT_NOTIFICATION_ITERATOR_STATUS_AGAIN:
ret = BT_COMPONENT_STATUS_AGAIN;
goto end;
}
- BT_ASSERT(notif);
- ret = handle_notification(pretty, notif);
+ BT_ASSERT(it_ret == BT_NOTIFICATION_ITERATOR_STATUS_OK);
+
+ for (i = 0; i < count; i++) {
+ ret = handle_notification(pretty, notifs[i]);
+ if (ret) {
+ goto end;
+ }
+
+ bt_object_put_ref(notifs[i]);
+ }
end:
- bt_put(notif);
+ for (; i < count; i++) {
+ bt_object_put_ref(notifs[i]);
+ }
+
return ret;
}
static
-enum bt_component_status add_params_to_map(struct bt_value *plugin_opt_map)
+enum bt_component_status add_params_to_map(
+ struct bt_private_value *plugin_opt_map)
{
enum bt_component_status ret = BT_COMPONENT_STATUS_OK;
unsigned int i;
const char *key = plugin_options[i];
enum bt_value_status status;
- status = bt_value_map_insert(plugin_opt_map, key, bt_value_null);
+ status = bt_private_value_map_insert_entry(plugin_opt_map, key,
+ bt_value_null);
switch (status) {
case BT_VALUE_STATUS_OK:
break;
bt_bool check_param_exists(const char *key, struct bt_value *object, void *data)
{
struct pretty_component *pretty = data;
- struct bt_value *plugin_opt_map = pretty->plugin_opt_map;
- if (!bt_value_map_has_key(plugin_opt_map, key)) {
+ if (!bt_value_map_has_entry(
+ bt_value_borrow_from_private(pretty->plugin_opt_map),
+ key)) {
fprintf(pretty->err,
"[warning] Parameter \"%s\" unknown to \"text.pretty\" sink component\n", key);
}
{
enum bt_component_status ret = BT_COMPONENT_STATUS_OK;
struct bt_value *value = NULL;
- enum bt_value_status status;
const char *str;
- value = bt_value_map_borrow(params, key);
+ value = bt_value_map_borrow_entry_value(params, key);
if (!value) {
goto end;
}
if (bt_value_is_null(value)) {
goto end;
}
- status = bt_value_string_get(value, &str);
- switch (status) {
- case BT_VALUE_STATUS_OK:
- break;
- default:
- ret = BT_COMPONENT_STATUS_ERROR;
- goto end;
- }
+ str = bt_value_string_get(value);
*option = g_strdup(str);
+
end:
return ret;
}
{
enum bt_component_status ret = BT_COMPONENT_STATUS_OK;
struct bt_value *value = NULL;
- enum bt_value_status status;
bt_bool bool_val;
- value = bt_value_map_borrow(params, key);
+ value = bt_value_map_borrow_entry_value(params, key);
if (!value) {
goto end;
}
- status = bt_value_bool_get(value, &bool_val);
- switch (status) {
- case BT_VALUE_STATUS_OK:
- break;
- default:
- ret = BT_COMPONENT_STATUS_ERROR;
- goto end;
- }
+ bool_val = bt_value_bool_get(value);
*option = (bool) bool_val;
if (found) {
*found = true;
bool value, found;
char *str = NULL;
- pretty->plugin_opt_map = bt_value_map_create();
+ pretty->plugin_opt_map = bt_private_value_map_create();
if (!pretty->plugin_opt_map) {
ret = BT_COMPONENT_STATUS_ERROR;
goto end;
goto end;
}
/* Report unknown parameters. */
- status = bt_value_map_foreach(params, check_param_exists, pretty);
+ status = bt_value_map_foreach_entry(params, check_param_exists, pretty);
switch (status) {
case BT_VALUE_STATUS_OK:
break;
}
/* Known parameters. */
pretty->options.color = PRETTY_COLOR_OPT_AUTO;
- if (bt_value_map_has_key(params, "color")) {
+ if (bt_value_map_has_entry(params, "color")) {
struct bt_value *color_value;
const char *color;
- color_value = bt_value_map_borrow(params, "color");
+ color_value = bt_value_map_borrow_entry_value(params, "color");
if (!color_value) {
goto end;
}
- status = bt_value_string_get(color_value, &color);
- if (status) {
- warn_wrong_color_param(pretty);
+ color = bt_value_string_get(color_value);
+
+ if (strcmp(color, "never") == 0) {
+ pretty->options.color = PRETTY_COLOR_OPT_NEVER;
+ } else if (strcmp(color, "auto") == 0) {
+ pretty->options.color = PRETTY_COLOR_OPT_AUTO;
+ } else if (strcmp(color, "always") == 0) {
+ pretty->options.color = PRETTY_COLOR_OPT_ALWAYS;
} else {
- if (strcmp(color, "never") == 0) {
- pretty->options.color = PRETTY_COLOR_OPT_NEVER;
- } else if (strcmp(color, "auto") == 0) {
- pretty->options.color = PRETTY_COLOR_OPT_AUTO;
- } else if (strcmp(color, "always") == 0) {
- pretty->options.color = PRETTY_COLOR_OPT_ALWAYS;
- } else {
- warn_wrong_color_param(pretty);
- }
+ warn_wrong_color_param(pretty);
}
}
}
end:
- bt_put(pretty->plugin_opt_map);
+ bt_object_put_ref(pretty->plugin_opt_map);
pretty->plugin_opt_map = NULL;
g_free(str);
return ret;