lib/values.c: add logging
authorPhilippe Proulx <eeppeliteloop@gmail.com>
Thu, 11 May 2017 03:50:18 +0000 (23:50 -0400)
committerJérémie Galarneau <jeremie.galarneau@efficios.com>
Sun, 28 May 2017 16:57:42 +0000 (12:57 -0400)
Signed-off-by: Philippe Proulx <eeppeliteloop@gmail.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
lib/values.c

index 24b26e11706a73c271be33188030103dbbfd66a7..8c9429636eed8ae710cfd0cec582983f69f0199d 100644 (file)
 #include <string.h>
 #include <assert.h>
 #include <string.h>
+#include <inttypes.h>
 #include <babeltrace/compiler-internal.h>
 #include <babeltrace/object-internal.h>
 #include <babeltrace/ref.h>
 #include <babeltrace/values.h>
 #include <babeltrace/compat/glib-internal.h>
 
+#define BT_LOG_TAG "VALUES"
+#include <babeltrace/lib-logging-internal.h>
+
 #define BT_VALUE_FROM_CONCRETE(_concrete) ((struct bt_value *) (_concrete))
 #define BT_VALUE_TO_BOOL(_base) ((struct bt_value_bool *) (_base))
 #define BT_VALUE_TO_INTEGER(_base) ((struct bt_value_integer *) (_base))
@@ -177,10 +181,12 @@ struct bt_value *bt_value_array_copy(const struct bt_value *array_obj)
        struct bt_value *copy_obj;
        struct bt_value_array *typed_array_obj;
 
+       BT_LOGD("Copying array value: addr=%p", array_obj);
        typed_array_obj = BT_VALUE_TO_ARRAY(array_obj);
        copy_obj = bt_value_array_create();
 
        if (!copy_obj) {
+               BT_LOGE_STR("Cannot create empty array value.");
                goto end;
        }
 
@@ -189,6 +195,9 @@ struct bt_value *bt_value_array_copy(const struct bt_value *array_obj)
                struct bt_value *element_obj = bt_value_array_get(array_obj, i);
 
                if (!element_obj) {
+                       BT_LOGE("Cannot get array value's element: "
+                               "addr=%p, index=%d",
+                               array_obj, i);
                        BT_PUT(copy_obj);
                        goto end;
                }
@@ -197,6 +206,9 @@ struct bt_value *bt_value_array_copy(const struct bt_value *array_obj)
                BT_PUT(element_obj);
 
                if (!element_obj_copy) {
+                       BT_LOGE("Cannot copy array value's element: "
+                               "array-addr=%p, index=%d",
+                               array_obj, i);
                        BT_PUT(copy_obj);
                        goto end;
                }
@@ -205,11 +217,15 @@ struct bt_value *bt_value_array_copy(const struct bt_value *array_obj)
                BT_PUT(element_obj_copy);
 
                if (ret) {
+                       BT_LOGE("Cannot append to array value: addr=%p",
+                               array_obj);
                        BT_PUT(copy_obj);
                        goto end;
                }
        }
 
+       BT_LOGD("Copied array value: addr=%p", array_obj);
+
 end:
        return copy_obj;
 }
@@ -224,6 +240,7 @@ struct bt_value *bt_value_map_copy(const struct bt_value *map_obj)
        struct bt_value *element_obj_copy;
        struct bt_value_map *typed_map_obj;
 
+       BT_LOGD("Copying map value: addr=%p", map_obj);
        typed_map_obj = BT_VALUE_TO_MAP(map_obj);
        copy_obj = bt_value_map_create();
 
@@ -239,6 +256,9 @@ struct bt_value *bt_value_map_copy(const struct bt_value *map_obj)
                element_obj_copy = bt_value_copy(element_obj);
 
                if (!element_obj_copy) {
+                       BT_LOGE("Cannot copy map value's element: "
+                               "map-addr=%p, key=\"%s\"",
+                               map_obj, key_str);
                        BT_PUT(copy_obj);
                        goto end;
                }
@@ -247,11 +267,15 @@ struct bt_value *bt_value_map_copy(const struct bt_value *map_obj)
                BT_PUT(element_obj_copy);
 
                if (ret) {
+                       BT_LOGE("Cannot insert into map value: addr=%p, key=\"%s\"",
+                               map_obj, key_str);
                        BT_PUT(copy_obj);
                        goto end;
                }
        }
 
+       BT_LOGD("Copied map value: addr=%p", map_obj);
+
 end:
        return copy_obj;
 }
@@ -321,6 +345,12 @@ bool bt_value_array_compare(const struct bt_value *object_a,
                BT_VALUE_TO_ARRAY(object_a);
 
        if (bt_value_array_size(object_a) != bt_value_array_size(object_b)) {
+               BT_LOGV("Array values are different: size mismatch "
+                       "value-a-addr=%p, value-b-addr=%p, "
+                       "value-a-size=%" PRId64 ", value-b-size=%" PRId64,
+                       object_a, object_b,
+                       bt_value_array_size(object_a),
+                       bt_value_array_size(object_b));
                ret = false;
                goto end;
        }
@@ -333,6 +363,9 @@ bool bt_value_array_compare(const struct bt_value *object_a,
                element_obj_b = bt_value_array_get(object_b, i);
 
                if (!bt_value_compare(element_obj_a, element_obj_b)) {
+                       BT_LOGV("Array values's elements are different: "
+                               "value-a-addr=%p, value-b-addr=%p, index=%d",
+                               element_obj_a, element_obj_b, index);
                        BT_PUT(element_obj_a);
                        BT_PUT(element_obj_b);
                        ret = false;
@@ -357,6 +390,12 @@ bool bt_value_map_compare(const struct bt_value *object_a,
        const struct bt_value_map *map_obj_a = BT_VALUE_TO_MAP(object_a);
 
        if (bt_value_map_size(object_a) != bt_value_map_size(object_b)) {
+               BT_LOGV("Map values are different: size mismatch "
+                       "value-a-addr=%p, value-b-addr=%p, "
+                       "value-a-size=%" PRId64 ", value-b-size=%" PRId64,
+                       object_a, object_b,
+                       bt_value_map_size(object_a),
+                       bt_value_map_size(object_b));
                ret = false;
                goto end;
        }
@@ -370,6 +409,9 @@ bool bt_value_map_compare(const struct bt_value *object_a,
                element_obj_b = bt_value_map_get(object_b, key_str);
 
                if (!bt_value_compare(element_obj_a, element_obj_b)) {
+                       BT_LOGV("Map values's elements are different: "
+                               "value-a-addr=%p, value-b-addr=%p, key=\"%s\"",
+                               element_obj_a, element_obj_b, key_str);
                        BT_PUT(element_obj_b);
                        ret = false;
                        goto end;
@@ -453,7 +495,10 @@ void bt_value_destroy(struct bt_object *obj)
        value = container_of(obj, struct bt_value, base);
        assert(value->type != BT_VALUE_TYPE_UNKNOWN);
 
+       BT_LOGD("Destroying value: addr=%p", value);
+
        if (bt_value_is_null(value)) {
+               BT_LOGD_STR("Not destroying the null value singleton.");
                return;
        }
 
@@ -469,10 +514,16 @@ enum bt_value_status bt_value_freeze(struct bt_value *object)
        enum bt_value_status ret = BT_VALUE_STATUS_OK;
 
        if (!object) {
+               BT_LOGW_STR("Invalid parameter: value object is NULL.");
                ret = BT_VALUE_STATUS_INVAL;
                goto end;
        }
 
+       if (object->is_frozen) {
+               goto end;
+       }
+
+       BT_LOGD("Freezing value: addr=%p", object);
        freeze_funcs[object->type](object);
 
 end:
@@ -487,6 +538,7 @@ bool bt_value_is_frozen(const struct bt_value *object)
 enum bt_value_type bt_value_get_type(const struct bt_value *object)
 {
        if (!object) {
+               BT_LOGW_STR("Invalid parameter: value object is NULL.");
                return BT_VALUE_TYPE_UNKNOWN;
        }
 
@@ -501,7 +553,6 @@ struct bt_value bt_value_create_base(enum bt_value_type type)
        base.type = type;
        base.is_frozen = false;
        bt_object_init(&base, bt_value_destroy);
-
        return base;
 }
 
@@ -509,14 +560,17 @@ struct bt_value *bt_value_bool_create_init(bool val)
 {
        struct bt_value_bool *bool_obj;
 
+       BT_LOGD("Creating boolean value object: val=%d", val);
        bool_obj = g_new0(struct bt_value_bool, 1);
 
        if (!bool_obj) {
+               BT_LOGE_STR("Failed to allocate one boolean value object.");
                goto end;
        }
 
        bool_obj->base = bt_value_create_base(BT_VALUE_TYPE_BOOL);
        bool_obj->value = val;
+       BT_LOGD("Created boolean value object: addr=%p", bool_obj);
 
 end:
        return BT_VALUE_FROM_CONCRETE(bool_obj);
@@ -531,14 +585,18 @@ struct bt_value *bt_value_integer_create_init(int64_t val)
 {
        struct bt_value_integer *integer_obj;
 
+       BT_LOGD("Creating integer value object: val=%" PRId64, val);
        integer_obj = g_new0(struct bt_value_integer, 1);
 
        if (!integer_obj) {
+               BT_LOGE_STR("Failed to allocate one integer value object.");
                goto end;
        }
 
        integer_obj->base = bt_value_create_base(BT_VALUE_TYPE_INTEGER);
        integer_obj->value = val;
+       BT_LOGD("Created integer value object: addr=%p",
+               integer_obj);
 
 end:
        return BT_VALUE_FROM_CONCRETE(integer_obj);
@@ -553,14 +611,18 @@ struct bt_value *bt_value_float_create_init(double val)
 {
        struct bt_value_float *float_obj;
 
+       BT_LOGD("Creating floating point number value object: val=%f", val);
        float_obj = g_new0(struct bt_value_float, 1);
 
        if (!float_obj) {
+               BT_LOGE_STR("Failed to allocate one floating point number value object.");
                goto end;
        }
 
        float_obj->base = bt_value_create_base(BT_VALUE_TYPE_FLOAT);
        float_obj->value = val;
+       BT_LOGD("Created floating point number value object: addr=%p",
+               float_obj);
 
 end:
        return BT_VALUE_FROM_CONCRETE(float_obj);
@@ -576,12 +638,15 @@ struct bt_value *bt_value_string_create_init(const char *val)
        struct bt_value_string *string_obj = NULL;
 
        if (!val) {
+               BT_LOGW_STR("Invalid parameter: value is NULL.");
                goto end;
        }
 
+       BT_LOGD("Creating string value object: val-len=%u", strlen(val));
        string_obj = g_new0(struct bt_value_string, 1);
 
        if (!string_obj) {
+               BT_LOGE_STR("Failed to allocate one string object.");
                goto end;
        }
 
@@ -589,11 +654,15 @@ struct bt_value *bt_value_string_create_init(const char *val)
        string_obj->gstr = g_string_new(val);
 
        if (!string_obj->gstr) {
+               BT_LOGE_STR("Failed to allocate a GString.");
                g_free(string_obj);
                string_obj = NULL;
                goto end;
        }
 
+       BT_LOGD("Created string value object: addr=%p",
+               string_obj);
+
 end:
        return BT_VALUE_FROM_CONCRETE(string_obj);
 }
@@ -607,9 +676,11 @@ struct bt_value *bt_value_array_create(void)
 {
        struct bt_value_array *array_obj;
 
+       BT_LOGD_STR("Creating empty array value object.");
        array_obj = g_new0(struct bt_value_array, 1);
 
        if (!array_obj) {
+               BT_LOGE_STR("Failed to allocate one array object.");
                goto end;
        }
 
@@ -618,11 +689,15 @@ struct bt_value *bt_value_array_create(void)
                (GDestroyNotify) bt_put);
 
        if (!array_obj->garray) {
+               BT_LOGE_STR("Failed to allocate a GPtrArray.");
                g_free(array_obj);
                array_obj = NULL;
                goto end;
        }
 
+       BT_LOGD("Created array value object: addr=%p",
+               array_obj);
+
 end:
        return BT_VALUE_FROM_CONCRETE(array_obj);
 }
@@ -631,9 +706,11 @@ struct bt_value *bt_value_map_create(void)
 {
        struct bt_value_map *map_obj;
 
+       BT_LOGD_STR("Creating empty map value object.");
        map_obj = g_new0(struct bt_value_map, 1);
 
        if (!map_obj) {
+               BT_LOGE_STR("Failed to allocate one map object.");
                goto end;
        }
 
@@ -642,11 +719,15 @@ struct bt_value *bt_value_map_create(void)
                NULL, (GDestroyNotify) bt_put);
 
        if (!map_obj->ght) {
+               BT_LOGE_STR("Failed to allocate a GHashTable.");
                g_free(map_obj);
                map_obj = NULL;
                goto end;
        }
 
+       BT_LOGD("Created map value object: addr=%p",
+               map_obj);
+
 end:
        return BT_VALUE_FROM_CONCRETE(map_obj);
 }
@@ -657,7 +738,17 @@ enum bt_value_status bt_value_bool_get(const struct bt_value *bool_obj,
        enum bt_value_status ret = BT_VALUE_STATUS_OK;
        struct bt_value_bool *typed_bool_obj = BT_VALUE_TO_BOOL(bool_obj);
 
-       if (!bool_obj || !bt_value_is_bool(bool_obj) || !val) {
+       if (!bool_obj || !val) {
+               BT_LOGW("Invalid parameter: value object or value is NULL: "
+                       "value-addr=%p, raw-value-addr=%p",
+                       bool_obj, val);
+               ret = BT_VALUE_STATUS_INVAL;
+               goto end;
+       }
+
+       if (!bt_value_is_bool(bool_obj)) {
+               BT_LOGW("Invalid parameter: value is not a boolean value: addr=%p, "
+                       "type=%d", bool_obj, bool_obj->type);
                ret = BT_VALUE_STATUS_INVAL;
                goto end;
        }
@@ -673,17 +764,29 @@ enum bt_value_status bt_value_bool_set(struct bt_value *bool_obj, bool val)
        enum bt_value_status ret = BT_VALUE_STATUS_OK;
        struct bt_value_bool *typed_bool_obj = BT_VALUE_TO_BOOL(bool_obj);
 
-       if (!bool_obj || !bt_value_is_bool(bool_obj)) {
+       if (!bool_obj) {
+               BT_LOGW_STR("Invalid parameter: value object is NULL.");
+               ret = BT_VALUE_STATUS_INVAL;
+               goto end;
+       }
+
+       if (!bt_value_is_bool(bool_obj)) {
+               BT_LOGW("Invalid parameter: value is not a boolean value: addr=%p, "
+                       "type=%d", bool_obj, bool_obj->type);
                ret = BT_VALUE_STATUS_INVAL;
                goto end;
        }
 
        if (bool_obj->is_frozen) {
+               BT_LOGW("Invalid parameter: value is frozen: addr=%p",
+                       bool_obj);
                ret = BT_VALUE_STATUS_FROZEN;
                goto end;
        }
 
        typed_bool_obj->value = val;
+       BT_LOGV("Set boolean value's raw value: value-addr=%p, value=%d",
+               bool_obj, val);
 
 end:
        return ret;
@@ -696,7 +799,17 @@ enum bt_value_status bt_value_integer_get(const struct bt_value *integer_obj,
        struct bt_value_integer *typed_integer_obj =
                BT_VALUE_TO_INTEGER(integer_obj);
 
-       if (!integer_obj || !bt_value_is_integer(integer_obj) || !val) {
+       if (!integer_obj || !val) {
+               BT_LOGW("Invalid parameter: value object or value is NULL: "
+                       "value-addr=%p, raw-value-addr=%p",
+                       integer_obj, val);
+               ret = BT_VALUE_STATUS_INVAL;
+               goto end;
+       }
+
+       if (!bt_value_is_integer(integer_obj)) {
+               BT_LOGW("Invalid parameter: value is not an integer value: addr=%p, "
+                       "type=%d", integer_obj, integer_obj->type);
                ret = BT_VALUE_STATUS_INVAL;
                goto end;
        }
@@ -714,17 +827,29 @@ enum bt_value_status bt_value_integer_set(struct bt_value *integer_obj,
        struct bt_value_integer *typed_integer_obj =
                BT_VALUE_TO_INTEGER(integer_obj);
 
-       if (!integer_obj || !bt_value_is_integer(integer_obj)) {
+       if (!integer_obj) {
+               BT_LOGW_STR("Invalid parameter: value object is NULL.");
+               ret = BT_VALUE_STATUS_INVAL;
+               goto end;
+       }
+
+       if (!bt_value_is_integer(integer_obj)) {
+               BT_LOGW("Invalid parameter: value is not an integer value: addr=%p, "
+                       "type=%d", integer_obj, integer_obj->type);
                ret = BT_VALUE_STATUS_INVAL;
                goto end;
        }
 
        if (integer_obj->is_frozen) {
+               BT_LOGW("Invalid parameter: value is frozen: addr=%p",
+                       integer_obj);
                ret = BT_VALUE_STATUS_FROZEN;
                goto end;
        }
 
        typed_integer_obj->value = val;
+       BT_LOGV("Set integer value's raw value: value-addr=%p, value=%" PRId64,
+               integer_obj, val);
 
 end:
        return ret;
@@ -737,7 +862,17 @@ enum bt_value_status bt_value_float_get(const struct bt_value *float_obj,
        struct bt_value_float *typed_float_obj =
                BT_VALUE_TO_FLOAT(float_obj);
 
-       if (!float_obj || !bt_value_is_float(float_obj) || !val) {
+       if (!float_obj || !val) {
+               BT_LOGW("Invalid parameter: value object or value is NULL: "
+                       "value-addr=%p, raw-value-addr=%p",
+                       float_obj, val);
+               ret = BT_VALUE_STATUS_INVAL;
+               goto end;
+       }
+
+       if (!bt_value_is_float(float_obj)) {
+               BT_LOGW("Invalid parameter: value is not a floating point number value: addr=%p, "
+                       "type=%d", float_obj, float_obj->type);
                ret = BT_VALUE_STATUS_INVAL;
                goto end;
        }
@@ -755,17 +890,29 @@ enum bt_value_status bt_value_float_set(struct bt_value *float_obj,
        struct bt_value_float *typed_float_obj =
                BT_VALUE_TO_FLOAT(float_obj);
 
-       if (!float_obj || !bt_value_is_float(float_obj)) {
+       if (!float_obj) {
+               BT_LOGW_STR("Invalid parameter: value object is NULL.");
+               ret = BT_VALUE_STATUS_INVAL;
+               goto end;
+       }
+
+       if (!bt_value_is_float(float_obj)) {
+               BT_LOGW("Invalid parameter: value is not a floating point number value: addr=%p, "
+                       "type=%d", float_obj, float_obj->type);
                ret = BT_VALUE_STATUS_INVAL;
                goto end;
        }
 
        if (float_obj->is_frozen) {
+               BT_LOGW("Invalid parameter: value is frozen: addr=%p",
+                       float_obj);
                ret = BT_VALUE_STATUS_FROZEN;
                goto end;
        }
 
        typed_float_obj->value = val;
+       BT_LOGV("Set floating point number value's raw value: value-addr=%p, value=%f",
+               float_obj, val);
 
 end:
        return ret;
@@ -778,7 +925,17 @@ enum bt_value_status bt_value_string_get(const struct bt_value *string_obj,
        struct bt_value_string *typed_string_obj =
                BT_VALUE_TO_STRING(string_obj);
 
-       if (!string_obj || !bt_value_is_string(string_obj) || !val) {
+       if (!string_obj || !val) {
+               BT_LOGW("Invalid parameter: value object or value is NULL: "
+                       "value-addr=%p, raw-value-addr=%p",
+                       string_obj, val);
+               ret = BT_VALUE_STATUS_INVAL;
+               goto end;
+       }
+
+       if (!bt_value_is_string(string_obj)) {
+               BT_LOGW("Invalid parameter: value is not a string value: addr=%p, "
+                       "type=%d", string_obj, string_obj->type);
                ret = BT_VALUE_STATUS_INVAL;
                goto end;
        }
@@ -796,17 +953,31 @@ enum bt_value_status bt_value_string_set(struct bt_value *string_obj,
        struct bt_value_string *typed_string_obj =
                BT_VALUE_TO_STRING(string_obj);
 
-       if (!string_obj || !bt_value_is_string(string_obj) || !val) {
+       if (!string_obj || !val) {
+               BT_LOGW("Invalid parameter: value object or value is NULL: "
+                       "value-addr=%p, raw-value-addr=%p",
+                       string_obj, val);
+               ret = BT_VALUE_STATUS_INVAL;
+               goto end;
+       }
+
+       if (!bt_value_is_string(string_obj)) {
+               BT_LOGW("Invalid parameter: value is not a string value: addr=%p, "
+                       "type=%d", string_obj, string_obj->type);
                ret = BT_VALUE_STATUS_INVAL;
                goto end;
        }
 
        if (string_obj->is_frozen) {
+               BT_LOGW("Invalid parameter: value is frozen: addr=%p",
+                       string_obj);
                ret = BT_VALUE_STATUS_FROZEN;
                goto end;
        }
 
        g_string_assign(typed_string_obj->gstr, val);
+       BT_LOGV("Set string value's raw value: value-addr=%p, raw-value-addr=%p",
+               string_obj, val);
 
 end:
        return ret;
@@ -818,11 +989,19 @@ int64_t bt_value_array_size(const struct bt_value *array_obj)
        struct bt_value_array *typed_array_obj =
                BT_VALUE_TO_ARRAY(array_obj);
 
-       if (!array_obj || !bt_value_is_array(array_obj)) {
+       if (!array_obj) {
+               BT_LOGW_STR("Invalid parameter: value object is NULL.");
                ret = (int64_t) BT_VALUE_STATUS_INVAL;
                goto end;
        }
 
+       if (!bt_value_is_array(array_obj)) {
+               BT_LOGW("Invalid parameter: value is not an array value: addr=%p, "
+                       "type=%d", array_obj, array_obj->type);
+               ret = BT_VALUE_STATUS_INVAL;
+               goto end;
+       }
+
        ret = (int64_t) typed_array_obj->garray->len;
 
 end:
@@ -835,14 +1014,30 @@ bool bt_value_array_is_empty(const struct bt_value *array_obj)
 }
 
 struct bt_value *bt_value_array_get(const struct bt_value *array_obj,
-               size_t index)
+               uint64_t index)
 {
        struct bt_value *ret;
        struct bt_value_array *typed_array_obj =
                BT_VALUE_TO_ARRAY(array_obj);
 
-       if (!array_obj || !bt_value_is_array(array_obj) ||
-                       index >= typed_array_obj->garray->len) {
+       if (!array_obj) {
+               BT_LOGW("Invalid parameter: value object is NULL: index=%" PRIu64,
+                       index);
+               ret = NULL;
+               goto end;
+       }
+
+       if (!bt_value_is_array(array_obj)) {
+               BT_LOGW("Invalid parameter: value is not an array value: addr=%p, "
+                       "type=%d", array_obj, array_obj->type);
+               ret = NULL;
+               goto end;
+       }
+
+       if (index >= typed_array_obj->garray->len) {
+               BT_LOGW("Invalid parameter: index is out of bounds: "
+                       "addr=%p, index=%" PRIu64,
+                       array_obj, index);
                ret = NULL;
                goto end;
        }
@@ -861,18 +1056,33 @@ enum bt_value_status bt_value_array_append(struct bt_value *array_obj,
        struct bt_value_array *typed_array_obj =
                BT_VALUE_TO_ARRAY(array_obj);
 
-       if (!array_obj || !bt_value_is_array(array_obj) || !element_obj) {
+       if (!array_obj || !element_obj) {
+               BT_LOGW("Invalid parameter: array value or element value is NULL: "
+                       "array-value-addr=%p, element-value-addr=%p",
+                       array_obj, element_obj);
+               ret = BT_VALUE_STATUS_INVAL;
+               goto end;
+       }
+
+       if (!bt_value_is_array(array_obj)) {
+               BT_LOGW("Invalid parameter: value is not an array value: addr=%p, "
+                       "type=%d", array_obj, array_obj->type);
                ret = BT_VALUE_STATUS_INVAL;
                goto end;
        }
 
        if (array_obj->is_frozen) {
+               BT_LOGW("Invalid parameter: value is frozen: addr=%p",
+                       array_obj);
                ret = BT_VALUE_STATUS_FROZEN;
                goto end;
        }
 
        g_ptr_array_add(typed_array_obj->garray, element_obj);
        bt_get(element_obj);
+       BT_LOGV("Appended element to array value: array-value-addr=%p, "
+               "element-value-addr=%p, new-size=%u",
+               array_obj, element_obj, typed_array_obj->garray->len);
 
 end:
        return ret;
@@ -887,7 +1097,6 @@ enum bt_value_status bt_value_array_append_bool(struct bt_value *array_obj,
        bool_obj = bt_value_bool_create_init(val);
        ret = bt_value_array_append(array_obj, bool_obj);
        bt_put(bool_obj);
-
        return ret;
 }
 
@@ -900,7 +1109,6 @@ enum bt_value_status bt_value_array_append_integer(
        integer_obj = bt_value_integer_create_init(val);
        ret = bt_value_array_append(array_obj, integer_obj);
        bt_put(integer_obj);
-
        return ret;
 }
 
@@ -913,7 +1121,6 @@ enum bt_value_status bt_value_array_append_float(struct bt_value *array_obj,
        float_obj = bt_value_float_create_init(val);
        ret = bt_value_array_append(array_obj, float_obj);
        bt_put(float_obj);
-
        return ret;
 }
 
@@ -926,7 +1133,6 @@ enum bt_value_status bt_value_array_append_string(struct bt_value *array_obj,
        string_obj = bt_value_string_create_init(val);
        ret = bt_value_array_append(array_obj, string_obj);
        bt_put(string_obj);
-
        return ret;
 }
 
@@ -939,7 +1145,6 @@ enum bt_value_status bt_value_array_append_empty_array(
        empty_array_obj = bt_value_array_create();
        ret = bt_value_array_append(array_obj, empty_array_obj);
        bt_put(empty_array_obj);
-
        return ret;
 }
 
@@ -951,24 +1156,42 @@ enum bt_value_status bt_value_array_append_empty_map(struct bt_value *array_obj)
        map_obj = bt_value_map_create();
        ret = bt_value_array_append(array_obj, map_obj);
        bt_put(map_obj);
-
        return ret;
 }
 
 enum bt_value_status bt_value_array_set(struct bt_value *array_obj,
-               size_t index, struct bt_value *element_obj)
+               uint64_t index, struct bt_value *element_obj)
 {
        enum bt_value_status ret = BT_VALUE_STATUS_OK;
        struct bt_value_array *typed_array_obj =
                BT_VALUE_TO_ARRAY(array_obj);
 
-       if (!array_obj || !bt_value_is_array(array_obj) || !element_obj ||
-                       index >= typed_array_obj->garray->len) {
+       if (!array_obj || !element_obj) {
+               BT_LOGW("Invalid parameter: array value or element value is NULL: "
+                       "index=%" PRIu64 ", array-value-addr=%p, element-value-addr=%p",
+                       index, array_obj, element_obj);
+               ret = BT_VALUE_STATUS_INVAL;
+               goto end;
+       }
+
+       if (!bt_value_is_array(array_obj)) {
+               BT_LOGW("Invalid parameter: value is not an array value: addr=%p, "
+                       "type=%d", array_obj, array_obj->type);
+               ret = BT_VALUE_STATUS_INVAL;
+               goto end;
+       }
+
+       if (index >= typed_array_obj->garray->len) {
+               BT_LOGW("Invalid parameter: index is out of bounds: "
+                       "addr=%p, index=%" PRIu64,
+                       array_obj, index);
                ret = BT_VALUE_STATUS_INVAL;
                goto end;
        }
 
        if (array_obj->is_frozen) {
+               BT_LOGW("Invalid parameter: value is frozen: addr=%p",
+                       array_obj);
                ret = BT_VALUE_STATUS_FROZEN;
                goto end;
        }
@@ -976,6 +1199,9 @@ enum bt_value_status bt_value_array_set(struct bt_value *array_obj,
        bt_put(g_ptr_array_index(typed_array_obj->garray, index));
        g_ptr_array_index(typed_array_obj->garray, index) = element_obj;
        bt_get(element_obj);
+       BT_LOGV("Set array value's element: array-value-addr=%p, "
+               "index=%" PRIu64 ", element-value-addr=%p",
+               array_obj, index, element_obj);
 
 end:
        return ret;
@@ -986,7 +1212,15 @@ int64_t bt_value_map_size(const struct bt_value *map_obj)
        int64_t ret;
        struct bt_value_map *typed_map_obj = BT_VALUE_TO_MAP(map_obj);
 
-       if (!map_obj || !bt_value_is_map(map_obj)) {
+       if (!map_obj) {
+               BT_LOGW_STR("Invalid parameter: value object is NULL.");
+               ret = (int64_t) BT_VALUE_STATUS_INVAL;
+               goto end;
+       }
+
+       if (!bt_value_is_map(map_obj)) {
+               BT_LOGW("Invalid parameter: value is not a map value: addr=%p, "
+                       "type=%d", map_obj, map_obj->type);
                ret = (int64_t) BT_VALUE_STATUS_INVAL;
                goto end;
        }
@@ -1009,7 +1243,16 @@ struct bt_value *bt_value_map_get(const struct bt_value *map_obj,
        struct bt_value *ret;
        struct bt_value_map *typed_map_obj = BT_VALUE_TO_MAP(map_obj);
 
-       if (!map_obj || !bt_value_is_map(map_obj) || !key) {
+       if (!map_obj || !key) {
+               BT_LOGW("Invalid parameter: value object or key is NULL: "
+                       "value-addr=%p, key-addr=%p", map_obj, key);
+               ret = NULL;
+               goto end;
+       }
+
+       if (!bt_value_is_map(map_obj)) {
+               BT_LOGW("Invalid parameter: value is not a map value: addr=%p, "
+                       "type=%d", map_obj, map_obj->type);
                ret = NULL;
                goto end;
        }
@@ -1031,7 +1274,16 @@ bool bt_value_map_has_key(const struct bt_value *map_obj, const char *key)
        GQuark quark;
        struct bt_value_map *typed_map_obj = BT_VALUE_TO_MAP(map_obj);
 
-       if (!map_obj || !bt_value_is_map(map_obj) || !key) {
+       if (!map_obj || !key) {
+               BT_LOGW("Invalid parameter: value object or key is NULL: "
+                       "value-addr=%p, key-addr=%p", map_obj, key);
+               ret = false;
+               goto end;
+       }
+
+       if (!bt_value_is_map(map_obj)) {
+               BT_LOGW("Invalid parameter: value is not a map value: addr=%p, "
+                       "type=%d", map_obj, map_obj->type);
                ret = false;
                goto end;
        }
@@ -1051,12 +1303,24 @@ enum bt_value_status bt_value_map_insert(struct bt_value *map_obj,
        enum bt_value_status ret = BT_VALUE_STATUS_OK;
        struct bt_value_map *typed_map_obj = BT_VALUE_TO_MAP(map_obj);
 
-       if (!map_obj || !bt_value_is_map(map_obj) || !key || !element_obj) {
+       if (!map_obj || !key || !element_obj) {
+               BT_LOGW("Invalid parameter: map value, key, or element value is NULL: "
+                       "map-value-addr=%p, key-addr=%p, element-value-addr=%p",
+                       map_obj, key, element_obj);
+               ret = BT_VALUE_STATUS_INVAL;
+               goto end;
+       }
+
+       if (!bt_value_is_map(map_obj)) {
+               BT_LOGW("Invalid parameter: value is not a map value: addr=%p, "
+                       "type=%d", map_obj, map_obj->type);
                ret = BT_VALUE_STATUS_INVAL;
                goto end;
        }
 
        if (map_obj->is_frozen) {
+               BT_LOGW("Invalid parameter: value is frozen: addr=%p",
+                       map_obj);
                ret = BT_VALUE_STATUS_FROZEN;
                goto end;
        }
@@ -1065,6 +1329,9 @@ enum bt_value_status bt_value_map_insert(struct bt_value *map_obj,
        g_hash_table_insert(typed_map_obj->ght,
                GUINT_TO_POINTER(quark), element_obj);
        bt_get(element_obj);
+       BT_LOGV("Inserted value into map value: map-value-addr=%p, "
+               "key=\"%s\", element-value-addr=%p",
+               map_obj, key, element_obj);
 
 end:
        return ret;
@@ -1079,7 +1346,6 @@ enum bt_value_status bt_value_map_insert_bool(struct bt_value *map_obj,
        bool_obj = bt_value_bool_create_init(val);
        ret = bt_value_map_insert(map_obj, key, bool_obj);
        bt_put(bool_obj);
-
        return ret;
 }
 
@@ -1092,7 +1358,6 @@ enum bt_value_status bt_value_map_insert_integer(struct bt_value *map_obj,
        integer_obj = bt_value_integer_create_init(val);
        ret = bt_value_map_insert(map_obj, key, integer_obj);
        bt_put(integer_obj);
-
        return ret;
 }
 
@@ -1105,7 +1370,6 @@ enum bt_value_status bt_value_map_insert_float(struct bt_value *map_obj,
        float_obj = bt_value_float_create_init(val);
        ret = bt_value_map_insert(map_obj, key, float_obj);
        bt_put(float_obj);
-
        return ret;
 }
 
@@ -1118,7 +1382,6 @@ enum bt_value_status bt_value_map_insert_string(struct bt_value *map_obj,
        string_obj = bt_value_string_create_init(val);
        ret = bt_value_map_insert(map_obj, key, string_obj);
        bt_put(string_obj);
-
        return ret;
 }
 
@@ -1131,7 +1394,6 @@ enum bt_value_status bt_value_map_insert_empty_array(struct bt_value *map_obj,
        array_obj = bt_value_array_create();
        ret = bt_value_map_insert(map_obj, key, array_obj);
        bt_put(array_obj);
-
        return ret;
 }
 
@@ -1144,7 +1406,6 @@ enum bt_value_status bt_value_map_insert_empty_map(struct bt_value *map_obj,
        empty_map_obj = bt_value_map_create();
        ret = bt_value_map_insert(map_obj, key, empty_map_obj);
        bt_put(empty_map_obj);
-
        return ret;
 }
 
@@ -1156,7 +1417,16 @@ enum bt_value_status bt_value_map_foreach(const struct bt_value *map_obj,
        GHashTableIter iter;
        struct bt_value_map *typed_map_obj = BT_VALUE_TO_MAP(map_obj);
 
-       if (!map_obj || !bt_value_is_map(map_obj) || !cb) {
+       if (!map_obj || !cb) {
+               BT_LOGW("Invalid parameter: map value or callback is NULL: "
+                       "value-addr=%p, cb-addr=%p", map_obj, cb);
+               ret = BT_VALUE_STATUS_INVAL;
+               goto end;
+       }
+
+       if (!bt_value_is_map(map_obj)) {
+               BT_LOGW("Invalid parameter: value is not a map value: addr=%p, "
+                       "type=%d", map_obj, map_obj->type);
                ret = BT_VALUE_STATUS_INVAL;
                goto end;
        }
@@ -1167,6 +1437,9 @@ enum bt_value_status bt_value_map_foreach(const struct bt_value *map_obj,
                const char *key_str = g_quark_to_string(GPOINTER_TO_UINT(key));
 
                if (!cb(key_str, element_obj, data)) {
+                       BT_LOGD("User cancelled the loop: key=\"%s\", "
+                               "value-addr=%p, data=%p",
+                               key_str, element_obj, data);
                        ret = BT_VALUE_STATUS_CANCELLED;
                        break;
                }
@@ -1196,6 +1469,10 @@ bool extend_map_element(const char *key,
        /* Replace in extended object */
        if (bt_value_map_insert(extend_data->extended_obj, key,
                        extension_obj_elem_copy)) {
+               BT_LOGE("Cannot replace value in extended value: key=\"%s\", "
+                       "extended-value-addr=%p, element-value-addr=%p",
+                       key, extend_data->extended_obj,
+                       extension_obj_elem_copy);
                goto error;
        }
 
@@ -1207,7 +1484,6 @@ error:
 
 end:
        BT_PUT(extension_obj_elem_copy);
-
        return ret;
 }
 
@@ -1217,13 +1493,33 @@ struct bt_value *bt_value_map_extend(struct bt_value *base_map_obj,
        struct bt_value *extended_obj = NULL;
        struct extend_map_element_data extend_data = { 0 };
 
-       if (!bt_value_is_map(base_map_obj) || !bt_value_is_map(extension_obj)) {
+       if (!base_map_obj || !extension_obj) {
+               BT_LOGW("Invalid parameter: base value or extension value is NULL: "
+                       "base-value-addr=%p, extension-value-addr=%p",
+                       base_map_obj, extension_obj);
+               goto error;
+       }
+
+       if (!bt_value_is_map(base_map_obj)) {
+               BT_LOGW("Invalid parameter: value is not a map value: addr=%p, "
+                       "type=%d", base_map_obj, base_map_obj->type);
                goto error;
        }
 
+       if (!bt_value_is_map(extension_obj)) {
+               BT_LOGW("Invalid parameter: value is not a map value: addr=%p, "
+                       "type=%d", extension_obj, extension_obj->type);
+               goto error;
+       }
+
+       BT_LOGD("Extending map value: base-value-addr=%p, extension-value-addr=%p",
+               base_map_obj, extension_obj);
+
        /* Create copy of base map object to start with */
        extended_obj = bt_value_copy(base_map_obj);
        if (!extended_obj) {
+               BT_LOGE("Cannot copy base value: base-value-addr=%p",
+                       base_map_obj);
                goto error;
        }
 
@@ -1235,13 +1531,19 @@ struct bt_value *bt_value_map_extend(struct bt_value *base_map_obj,
 
        if (bt_value_map_foreach(extension_obj, extend_map_element,
                        &extend_data)) {
+               BT_LOGE("Cannot iterate on the extension object's elements: ",
+                       "extension-value-addr=%p", extension_obj);
                goto error;
        }
 
        if (extend_data.got_error) {
+               BT_LOGE("Failed to successfully iterate on the extension object's elements: ",
+                       "extension-value-addr=%p", extension_obj);
                goto error;
        }
 
+       BT_LOGD("Extended map value: extended-value-addr=%p",
+               extended_obj);
        goto end;
 
 error:
@@ -1256,10 +1558,18 @@ struct bt_value *bt_value_copy(const struct bt_value *object)
        struct bt_value *copy_obj = NULL;
 
        if (!object) {
+               BT_LOGW_STR("Invalid parameter: value object is NULL.");
                goto end;
        }
 
+       BT_LOGD("Copying value object: addr=%p", object);
        copy_obj = copy_funcs[object->type](object);
+       if (copy_obj) {
+               BT_LOGD("Copied value object: copy-value-addr=%p",
+                       copy_obj);
+       } else {
+               BT_LOGE_STR("Failed to copy value object.");
+       }
 
 end:
        return copy_obj;
@@ -1271,10 +1581,18 @@ bool bt_value_compare(const struct bt_value *object_a,
        bool ret = false;
 
        if (!object_a || !object_b) {
+               BT_LOGW("Invalid parameter: value A or value B is NULL: "
+                       "value-a-addr=%p, value-b-addr=%p",
+                       object_a, object_b);
                goto end;
        }
 
        if (object_a->type != object_b->type) {
+               BT_LOGV("Values are different: type mismatch: "
+                       "value-a-addr=%p, value-b-addr=%p, "
+                       "value-a-type=%d, value-b-type=%d",
+                       object_a, object_b,
+                       object_a->type, object_b->type);
                goto end;
        }
 
This page took 0.036174 seconds and 4 git commands to generate.