X-Git-Url: http://git.efficios.com/?a=blobdiff_plain;f=src%2Flib%2Ftrace-ir%2Fresolve-field-path.c;h=d92a7f3d458fd020456f5d8d5f07d59e2cd33c44;hb=43c59509042845f8d42c3e99ec74d45fa2dc0908;hp=adf4c139f1e4fdd018bb998c3d6389b58dc1f339;hpb=de821fe507f035622c474f097b226f30cd3c7491;p=babeltrace.git diff --git a/src/lib/trace-ir/resolve-field-path.c b/src/lib/trace-ir/resolve-field-path.c index adf4c139..d92a7f3d 100644 --- a/src/lib/trace-ir/resolve-field-path.c +++ b/src/lib/trace-ir/resolve-field-path.c @@ -25,8 +25,9 @@ #include "lib/assert-pre.h" #include "common/assert.h" -#include +#include #include +#include #include #include #include @@ -46,12 +47,7 @@ bool find_field_class_recursive(struct bt_field_class *fc, goto end; } - switch (fc->type) { - case BT_FIELD_CLASS_TYPE_OPTION_WITHOUT_SELECTOR_FIELD: - case BT_FIELD_CLASS_TYPE_OPTION_WITH_BOOL_SELECTOR_FIELD: - case BT_FIELD_CLASS_TYPE_OPTION_WITH_UNSIGNED_INTEGER_SELECTOR_FIELD: - case BT_FIELD_CLASS_TYPE_OPTION_WITH_SIGNED_INTEGER_SELECTOR_FIELD: - { + if (bt_field_class_type_is(fc->type, BT_FIELD_CLASS_TYPE_OPTION)) { struct bt_field_class_option *opt_fc = (void *) fc; struct bt_field_path_item item = { .type = BT_FIELD_PATH_ITEM_TYPE_CURRENT_OPTION_CONTENT, @@ -66,13 +62,9 @@ bool find_field_class_recursive(struct bt_field_class *fc, } bt_field_path_remove_last_item(field_path); - break; - } - case BT_FIELD_CLASS_TYPE_STRUCTURE: - case BT_FIELD_CLASS_TYPE_VARIANT_WITHOUT_SELECTOR_FIELD: - case BT_FIELD_CLASS_TYPE_VARIANT_WITH_UNSIGNED_INTEGER_SELECTOR_FIELD: - case BT_FIELD_CLASS_TYPE_VARIANT_WITH_SIGNED_INTEGER_SELECTOR_FIELD: - { + } else if (fc->type == BT_FIELD_CLASS_TYPE_STRUCTURE || + bt_field_class_type_is(fc->type, + BT_FIELD_CLASS_TYPE_VARIANT)) { struct bt_field_class_named_field_class_container *container_fc = (void *) fc; uint64_t i; @@ -94,13 +86,7 @@ bool find_field_class_recursive(struct bt_field_class *fc, bt_field_path_remove_last_item(field_path); } - - break; - } - case BT_FIELD_CLASS_TYPE_STATIC_ARRAY: - case BT_FIELD_CLASS_TYPE_DYNAMIC_ARRAY_WITHOUT_LENGTH_FIELD: - case BT_FIELD_CLASS_TYPE_DYNAMIC_ARRAY_WITH_LENGTH_FIELD: - { + } else if (bt_field_class_type_is(fc->type, BT_FIELD_CLASS_TYPE_ARRAY)) { struct bt_field_class_array *array_fc = (void *) fc; struct bt_field_path_item item = { .type = BT_FIELD_PATH_ITEM_TYPE_CURRENT_ARRAY_ELEMENT, @@ -115,10 +101,6 @@ bool find_field_class_recursive(struct bt_field_class *fc, } bt_field_path_remove_last_item(field_path); - break; - } - default: - break; } end: @@ -248,7 +230,7 @@ struct bt_field_class *borrow_root_field_class( case BT_FIELD_PATH_SCOPE_EVENT_PAYLOAD: return ctx->event_payload; default: - abort(); + bt_common_abort(); } return NULL; @@ -262,24 +244,16 @@ struct bt_field_class *borrow_child_field_class( { struct bt_field_class *child_fc = NULL; - switch (parent_fc->type) { - case BT_FIELD_CLASS_TYPE_OPTION_WITHOUT_SELECTOR_FIELD: - case BT_FIELD_CLASS_TYPE_OPTION_WITH_BOOL_SELECTOR_FIELD: - case BT_FIELD_CLASS_TYPE_OPTION_WITH_UNSIGNED_INTEGER_SELECTOR_FIELD: - case BT_FIELD_CLASS_TYPE_OPTION_WITH_SIGNED_INTEGER_SELECTOR_FIELD: - { + if (bt_field_class_type_is(parent_fc->type, + BT_FIELD_CLASS_TYPE_OPTION)) { struct bt_field_class_option *opt_fc = (void *) parent_fc; BT_ASSERT(fp_item->type == BT_FIELD_PATH_ITEM_TYPE_CURRENT_OPTION_CONTENT); child_fc = opt_fc->content_fc; - break; - } - case BT_FIELD_CLASS_TYPE_STRUCTURE: - case BT_FIELD_CLASS_TYPE_VARIANT_WITHOUT_SELECTOR_FIELD: - case BT_FIELD_CLASS_TYPE_VARIANT_WITH_UNSIGNED_INTEGER_SELECTOR_FIELD: - case BT_FIELD_CLASS_TYPE_VARIANT_WITH_SIGNED_INTEGER_SELECTOR_FIELD: - { + } else if (parent_fc->type == BT_FIELD_CLASS_TYPE_STRUCTURE || + bt_field_class_type_is(parent_fc->type, + BT_FIELD_CLASS_TYPE_VARIANT)) { struct bt_field_class_named_field_class_container *container_fc = (void *) parent_fc; struct bt_named_field_class *named_fc; @@ -287,21 +261,13 @@ struct bt_field_class *borrow_child_field_class( BT_ASSERT(fp_item->type == BT_FIELD_PATH_ITEM_TYPE_INDEX); named_fc = container_fc->named_fcs->pdata[fp_item->index]; child_fc = named_fc->fc; - break; - } - case BT_FIELD_CLASS_TYPE_STATIC_ARRAY: - case BT_FIELD_CLASS_TYPE_DYNAMIC_ARRAY_WITHOUT_LENGTH_FIELD: - case BT_FIELD_CLASS_TYPE_DYNAMIC_ARRAY_WITH_LENGTH_FIELD: - { + } else if (bt_field_class_type_is(parent_fc->type, + BT_FIELD_CLASS_TYPE_ARRAY)) { struct bt_field_class_array *array_fc = (void *) parent_fc; BT_ASSERT(fp_item->type == BT_FIELD_PATH_ITEM_TYPE_CURRENT_ARRAY_ELEMENT); child_fc = array_fc->element_fc; - break; - } - default: - break; } return child_fc; @@ -329,16 +295,12 @@ bool target_field_path_in_different_scope_has_struct_fc_only( bt_field_path_borrow_item_by_index_inline( tgt_field_path, i); - if (fc->type == BT_FIELD_CLASS_TYPE_STATIC_ARRAY || - fc->type == BT_FIELD_CLASS_TYPE_DYNAMIC_ARRAY_WITHOUT_LENGTH_FIELD || - fc->type == BT_FIELD_CLASS_TYPE_DYNAMIC_ARRAY_WITH_LENGTH_FIELD || - fc->type == BT_FIELD_CLASS_TYPE_OPTION_WITHOUT_SELECTOR_FIELD || - fc->type == BT_FIELD_CLASS_TYPE_OPTION_WITH_BOOL_SELECTOR_FIELD || - fc->type == BT_FIELD_CLASS_TYPE_OPTION_WITH_UNSIGNED_INTEGER_SELECTOR_FIELD || - fc->type == BT_FIELD_CLASS_TYPE_OPTION_WITH_SIGNED_INTEGER_SELECTOR_FIELD || - fc->type == BT_FIELD_CLASS_TYPE_VARIANT_WITHOUT_SELECTOR_FIELD || - fc->type == BT_FIELD_CLASS_TYPE_VARIANT_WITH_UNSIGNED_INTEGER_SELECTOR_FIELD || - fc->type == BT_FIELD_CLASS_TYPE_VARIANT_WITH_SIGNED_INTEGER_SELECTOR_FIELD) { + if (bt_field_class_type_is(fc->type, + BT_FIELD_CLASS_TYPE_ARRAY) || + bt_field_class_type_is(fc->type, + BT_FIELD_CLASS_TYPE_OPTION) || + bt_field_class_type_is(fc->type, + BT_FIELD_CLASS_TYPE_VARIANT)) { is_valid = false; goto end; } @@ -453,16 +415,12 @@ bool lca_to_target_has_struct_fc_only(struct bt_field_path *src_field_path, bt_field_path_borrow_item_by_index_inline( tgt_field_path, tgt_i); - if (tgt_fc->type == BT_FIELD_CLASS_TYPE_STATIC_ARRAY || - tgt_fc->type == BT_FIELD_CLASS_TYPE_DYNAMIC_ARRAY_WITHOUT_LENGTH_FIELD || - tgt_fc->type == BT_FIELD_CLASS_TYPE_DYNAMIC_ARRAY_WITH_LENGTH_FIELD || - tgt_fc->type == BT_FIELD_CLASS_TYPE_OPTION_WITHOUT_SELECTOR_FIELD || - tgt_fc->type == BT_FIELD_CLASS_TYPE_OPTION_WITH_BOOL_SELECTOR_FIELD || - tgt_fc->type == BT_FIELD_CLASS_TYPE_OPTION_WITH_UNSIGNED_INTEGER_SELECTOR_FIELD || - tgt_fc->type == BT_FIELD_CLASS_TYPE_OPTION_WITH_SIGNED_INTEGER_SELECTOR_FIELD || - tgt_fc->type == BT_FIELD_CLASS_TYPE_VARIANT_WITHOUT_SELECTOR_FIELD || - tgt_fc->type == BT_FIELD_CLASS_TYPE_VARIANT_WITH_UNSIGNED_INTEGER_SELECTOR_FIELD || - tgt_fc->type == BT_FIELD_CLASS_TYPE_VARIANT_WITH_SIGNED_INTEGER_SELECTOR_FIELD) { + if (bt_field_class_type_is(tgt_fc->type, + BT_FIELD_CLASS_TYPE_ARRAY) || + bt_field_class_type_is(tgt_fc->type, + BT_FIELD_CLASS_TYPE_OPTION) || + bt_field_class_type_is(tgt_fc->type, + BT_FIELD_CLASS_TYPE_VARIANT)) { is_valid = false; goto end; } @@ -571,11 +529,8 @@ int bt_resolve_field_paths(struct bt_field_class *fc, BT_ASSERT(fc); /* Resolving part for dynamic array and variant field classes */ - switch (fc->type) { - case BT_FIELD_CLASS_TYPE_OPTION_WITH_BOOL_SELECTOR_FIELD: - case BT_FIELD_CLASS_TYPE_OPTION_WITH_UNSIGNED_INTEGER_SELECTOR_FIELD: - case BT_FIELD_CLASS_TYPE_OPTION_WITH_SIGNED_INTEGER_SELECTOR_FIELD: - { + if (bt_field_class_type_is(fc->type, + BT_FIELD_CLASS_TYPE_OPTION_WITH_SELECTOR_FIELD)) { struct bt_field_class_option_with_selector_field *opt_fc = (void *) fc; BT_ASSERT(opt_fc->selector_fc); @@ -586,11 +541,7 @@ int bt_resolve_field_paths(struct bt_field_class *fc, ret = -1; goto end; } - - break; - } - case BT_FIELD_CLASS_TYPE_DYNAMIC_ARRAY_WITH_LENGTH_FIELD: - { + } else if (fc->type == BT_FIELD_CLASS_TYPE_DYNAMIC_ARRAY_WITH_LENGTH_FIELD) { struct bt_field_class_array_dynamic *dyn_array_fc = (void *) fc; BT_ASSERT(dyn_array_fc->length_fc); @@ -601,12 +552,8 @@ int bt_resolve_field_paths(struct bt_field_class *fc, ret = -1; goto end; } - - break; - } - case BT_FIELD_CLASS_TYPE_VARIANT_WITH_UNSIGNED_INTEGER_SELECTOR_FIELD: - case BT_FIELD_CLASS_TYPE_VARIANT_WITH_SIGNED_INTEGER_SELECTOR_FIELD: - { + } else if (bt_field_class_type_is(fc->type, + BT_FIELD_CLASS_TYPE_VARIANT_WITH_SELECTOR_FIELD)) { struct bt_field_class_variant_with_selector_field *var_fc = (void *) fc; @@ -621,27 +568,15 @@ int bt_resolve_field_paths(struct bt_field_class *fc, } } } - default: - break; - } /* Recursive part */ - switch (fc->type) { - case BT_FIELD_CLASS_TYPE_OPTION_WITHOUT_SELECTOR_FIELD: - case BT_FIELD_CLASS_TYPE_OPTION_WITH_BOOL_SELECTOR_FIELD: - case BT_FIELD_CLASS_TYPE_OPTION_WITH_UNSIGNED_INTEGER_SELECTOR_FIELD: - case BT_FIELD_CLASS_TYPE_OPTION_WITH_SIGNED_INTEGER_SELECTOR_FIELD: - { + if (bt_field_class_type_is(fc->type, BT_FIELD_CLASS_TYPE_OPTION)) { struct bt_field_class_option *opt_fc = (void *) fc; ret = bt_resolve_field_paths(opt_fc->content_fc, ctx); - break; - } - case BT_FIELD_CLASS_TYPE_STRUCTURE: - case BT_FIELD_CLASS_TYPE_VARIANT_WITHOUT_SELECTOR_FIELD: - case BT_FIELD_CLASS_TYPE_VARIANT_WITH_UNSIGNED_INTEGER_SELECTOR_FIELD: - case BT_FIELD_CLASS_TYPE_VARIANT_WITH_SIGNED_INTEGER_SELECTOR_FIELD: - { + } else if (fc->type == BT_FIELD_CLASS_TYPE_STRUCTURE || + bt_field_class_type_is(fc->type, + BT_FIELD_CLASS_TYPE_VARIANT)) { struct bt_field_class_named_field_class_container *container_fc = (void *) fc; uint64_t i; @@ -655,20 +590,11 @@ int bt_resolve_field_paths(struct bt_field_class *fc, goto end; } } - - break; - } - case BT_FIELD_CLASS_TYPE_STATIC_ARRAY: - case BT_FIELD_CLASS_TYPE_DYNAMIC_ARRAY_WITHOUT_LENGTH_FIELD: - case BT_FIELD_CLASS_TYPE_DYNAMIC_ARRAY_WITH_LENGTH_FIELD: - { + } else if (bt_field_class_type_is(fc->type, + BT_FIELD_CLASS_TYPE_ARRAY)) { struct bt_field_class_array *array_fc = (void *) fc; ret = bt_resolve_field_paths(array_fc->element_fc, ctx); - break; - } - default: - break; } end: