[lttng-dev] [RFC PATCH babeltrace] Use glib portable macros to convert gpointer to int
Michael Jeanson
mjeanson at efficios.com
Thu Sep 15 00:52:21 UTC 2016
---
formats/ctf-text/ctf-text.c | 2 +-
formats/ctf/callbacks.c | 4 ++--
formats/ctf/events.c | 2 +-
formats/ctf/ir/field-types.c | 4 ++--
formats/ctf/metadata/ctf-parser.y | 2 +-
.../ctf/metadata/ctf-visitor-generate-io-struct.c | 12 ++++++------
lib/context.c | 4 ++--
lib/registry.c | 8 ++++----
lib/trace-collection.c | 8 ++++----
lib/trace-handle.c | 6 +++---
lib/values.c | 6 +++---
types/enum.c | 10 +++++-----
types/struct.c | 8 ++++----
types/types.c | 20 ++++++++++----------
types/variant.c | 12 ++++++------
15 files changed, 54 insertions(+), 54 deletions(-)
diff --git a/formats/ctf-text/ctf-text.c b/formats/ctf-text/ctf-text.c
index f615161..b5da014 100644
--- a/formats/ctf-text/ctf-text.c
+++ b/formats/ctf-text/ctf-text.c
@@ -136,7 +136,7 @@ struct ctf_callsite_dups *ctf_trace_callsite_lookup(struct ctf_trace *trace,
GQuark callsite_name)
{
return g_hash_table_lookup(trace->callsites,
- (gpointer) (unsigned long) callsite_name);
+ GUINT_TO_POINTER(callsite_name));
}
void bt_ctf_text_hook(void)
diff --git a/formats/ctf/callbacks.c b/formats/ctf/callbacks.c
index 9242a2b..a8c376f 100644
--- a/formats/ctf/callbacks.c
+++ b/formats/ctf/callbacks.c
@@ -115,13 +115,13 @@ int bt_ctf_iter_add_callback(struct bt_ctf_iter *iter,
if (event) {
/* find the event id */
event_id_ptr = g_hash_table_lookup(stream->event_quark_to_id,
- (gconstpointer) (unsigned long) event);
+ (gconstpointer) GUINT_TO_POINTER(event));
/* event not found in this stream class */
if (!event_id_ptr) {
fprintf(stderr, "[error] Event ID not found in stream class\n");
continue;
}
- event_id = (uint64_t)(unsigned long) *event_id_ptr;
+ event_id = GPOINTER_TO_UINT(*event_id_ptr);
/* find or create the bt_callback_chain for this event */
if (event_id >= bt_stream_cb->per_id_callbacks->len) {
diff --git a/formats/ctf/events.c b/formats/ctf/events.c
index d8e8d5d..0eb688a 100644
--- a/formats/ctf/events.c
+++ b/formats/ctf/events.c
@@ -690,7 +690,7 @@ int bt_ctf_get_event_decl_list(int handle_id, struct bt_context *ctx,
goto error;
handle = g_hash_table_lookup(ctx->trace_handles,
- (gpointer) (unsigned long) handle_id);
+ GUINT_TO_POINTER(handle_id));
if (!handle)
goto error;
diff --git a/formats/ctf/ir/field-types.c b/formats/ctf/ir/field-types.c
index f898d5a..4f2bc03 100644
--- a/formats/ctf/ir/field-types.c
+++ b/formats/ctf/ir/field-types.c
@@ -404,8 +404,8 @@ int add_structure_field(GPtrArray *fields,
field->name = name_quark;
field->type = field_type;
g_hash_table_insert(field_name_to_index,
- (gpointer) (unsigned long) name_quark,
- (gpointer) (unsigned long) fields->len);
+ GUINT_TO_POINTER(name_quark),
+ GUINT_TO_POINTER(fields->len));
g_ptr_array_add(fields, field);
end:
return ret;
diff --git a/formats/ctf/metadata/ctf-parser.y b/formats/ctf/metadata/ctf-parser.y
index 821f0f7..d0866ff 100644
--- a/formats/ctf/metadata/ctf-parser.y
+++ b/formats/ctf/metadata/ctf-parser.y
@@ -330,7 +330,7 @@ static int lookup_type(struct ctf_scanner_scope *s, const char *id)
{
int ret;
- ret = (int) (long) g_hash_table_lookup(s->types, id);
+ ret = GPOINTER_TO_INT(g_hash_table_lookup(s->types, id));
printf_debug("lookup %p %s %d\n", s, id, ret);
return ret;
}
diff --git a/formats/ctf/metadata/ctf-visitor-generate-io-struct.c b/formats/ctf/metadata/ctf-visitor-generate-io-struct.c
index 097a482..13b79f0 100644
--- a/formats/ctf/metadata/ctf-visitor-generate-io-struct.c
+++ b/formats/ctf/metadata/ctf-visitor-generate-io-struct.c
@@ -278,7 +278,7 @@ struct ctf_event_declaration *stream_event_lookup(struct ctf_stream_declaration
static
struct ctf_clock *trace_clock_lookup(struct ctf_trace *trace, GQuark clock_name)
{
- return g_hash_table_lookup(trace->parent.clocks, (gpointer) (unsigned long) clock_name);
+ return g_hash_table_lookup(trace->parent.clocks, GUINT_TO_POINTER(clock_name));
}
static
@@ -1907,7 +1907,7 @@ int ctf_event_visit(FILE *fd, int depth, struct ctf_node *node,
g_ptr_array_set_size(event->stream->events_by_id, event->id + 1);
g_ptr_array_index(event->stream->events_by_id, event->id) = event;
g_hash_table_insert(event->stream->event_quark_to_id,
- (gpointer) (unsigned long) event->name,
+ GUINT_TO_POINTER(event->name),
&event->id);
g_ptr_array_add(trace->event_declarations, event_decl);
return 0;
@@ -2491,7 +2491,7 @@ int ctf_clock_visit(FILE *fd, int depth, struct ctf_node *node, struct ctf_trace
goto error;
}
trace->parent.single_clock = clock;
- g_hash_table_insert(trace->parent.clocks, (gpointer) (unsigned long) clock->name, clock);
+ g_hash_table_insert(trace->parent.clocks, GUINT_TO_POINTER(clock->name), clock);
return 0;
error:
@@ -2526,7 +2526,7 @@ void ctf_clock_default(FILE *fd, int depth, struct ctf_trace *trace)
}
trace->parent.single_clock = clock;
- g_hash_table_insert(trace->parent.clocks, (gpointer) (unsigned long) clock->name, clock);
+ g_hash_table_insert(trace->parent.clocks, GUINT_TO_POINTER(clock->name), clock);
}
static
@@ -2683,12 +2683,12 @@ int ctf_callsite_visit(FILE *fd, int depth, struct ctf_node *node, struct ctf_tr
}
cs_dups = g_hash_table_lookup(trace->callsites,
- (gpointer) (unsigned long) callsite->name);
+ GUINT_TO_POINTER(callsite->name));
if (!cs_dups) {
cs_dups = g_new0(struct ctf_callsite_dups, 1);
BT_INIT_LIST_HEAD(&cs_dups->head);
g_hash_table_insert(trace->callsites,
- (gpointer) (unsigned long) callsite->name, cs_dups);
+ GUINT_TO_POINTER(callsite->name), cs_dups);
}
bt_list_add_tail(&callsite->node, &cs_dups->head);
return 0;
diff --git a/lib/context.c b/lib/context.c
index 3617bcd..56ac320 100644
--- a/lib/context.c
+++ b/lib/context.c
@@ -174,7 +174,7 @@ int bt_context_add_trace(struct bt_context *ctx, const char *path,
/* Add new handle to container */
g_hash_table_insert(ctx->trace_handles,
- (gpointer) (unsigned long) handle->id,
+ GUINT_TO_POINTER(handle->id),
handle);
return handle->id;
@@ -207,7 +207,7 @@ int bt_context_remove_trace(struct bt_context *ctx, int handle_id)
* automatically.
*/
if (!g_hash_table_remove(ctx->trace_handles,
- (gpointer) (unsigned long) handle_id)) {
+ GUINT_TO_POINTER(handle_id))) {
ret = -ENOENT;
goto end;
}
diff --git a/lib/registry.c b/lib/registry.c
index bdce0c6..bc2be95 100644
--- a/lib/registry.c
+++ b/lib/registry.c
@@ -73,7 +73,7 @@ struct bt_format *bt_lookup_format(bt_intern_str name)
return NULL;
return g_hash_table_lookup(format_registry,
- (gconstpointer) (unsigned long) name);
+ (gconstpointer) GUINT_TO_POINTER(name));
}
static void show_format(gpointer key, gpointer value, gpointer user_data)
@@ -81,7 +81,7 @@ static void show_format(gpointer key, gpointer value, gpointer user_data)
struct walk_data *data = user_data;
fprintf(data->fp, "%s%s", data->iter ? ", " : "",
- g_quark_to_string((GQuark) (unsigned long) key));
+ g_quark_to_string((GQuark) GPOINTER_TO_UINT(key)));
data->iter++;
}
@@ -116,7 +116,7 @@ int bt_register_format(struct bt_format *format)
format_refcount_inc();
g_hash_table_insert(format_registry,
- (gpointer) (unsigned long) format->name,
+ GUINT_TO_POINTER(format->name),
format);
return 0;
}
@@ -125,7 +125,7 @@ void bt_unregister_format(struct bt_format *format)
{
assert(bt_lookup_format(format->name));
g_hash_table_remove(format_registry,
- (gpointer) (unsigned long) format->name);
+ GUINT_TO_POINTER(format->name));
format_refcount_dec();
}
diff --git a/lib/trace-collection.c b/lib/trace-collection.c
index 035d2dc..213ece5 100644
--- a/lib/trace-collection.c
+++ b/lib/trace-collection.c
@@ -53,7 +53,7 @@ static void check_clock_match(gpointer key, gpointer value, gpointer user_data)
* by clock name.
*/
clock_b = g_hash_table_lookup(match->clocks,
- (gpointer) (unsigned long) clock_a->name);
+ GUINT_TO_POINTER(clock_a->name));
if (clock_b) {
match->clock_match = clock_b;
return;
@@ -64,7 +64,7 @@ static void check_clock_match(gpointer key, gpointer value, gpointer user_data)
* clocks.
*/
clock_b = g_hash_table_lookup(match->clocks,
- (gpointer) (unsigned long) clock_a->uuid);
+ GUINT_TO_POINTER(clock_a->uuid));
if (clock_b) {
match->clock_match = clock_b;
return;
@@ -87,7 +87,7 @@ static void clock_add(gpointer key, gpointer value, gpointer user_data)
struct ctf_clock *tc_clock;
tc_clock = g_hash_table_lookup(tc_clocks,
- (gpointer) (unsigned long) v);
+ GUINT_TO_POINTER(v));
if (!tc_clock) {
/*
* For now we only support CTF that has one
@@ -105,7 +105,7 @@ static void clock_add(gpointer key, gpointer value, gpointer user_data)
clock_match->tc->offset_first;
}
g_hash_table_insert(tc_clocks,
- (gpointer) (unsigned long) v,
+ GUINT_TO_POINTER(v),
value);
} else if (!t_clock->absolute) {
int64_t diff_ns;
diff --git a/lib/trace-handle.c b/lib/trace-handle.c
index b768a08..4c41d2f 100644
--- a/lib/trace-handle.c
+++ b/lib/trace-handle.c
@@ -60,7 +60,7 @@ const char *bt_trace_handle_get_path(struct bt_context *ctx, int handle_id)
return NULL;
handle = g_hash_table_lookup(ctx->trace_handles,
- (gpointer) (unsigned long) handle_id);
+ GUINT_TO_POINTER(handle_id));
if (!handle)
return NULL;
return handle->path;
@@ -77,7 +77,7 @@ int bt_trace_handle_get_timestamp_begin(struct bt_context *ctx,
return -1;
handle = g_hash_table_lookup(ctx->trace_handles,
- (gpointer) (unsigned long) handle_id);
+ GUINT_TO_POINTER(handle_id));
if (!handle) {
ret = -1;
goto end;
@@ -105,7 +105,7 @@ int bt_trace_handle_get_timestamp_end(struct bt_context *ctx,
return -1;
handle = g_hash_table_lookup(ctx->trace_handles,
- (gpointer) (unsigned long) handle_id);
+ GUINT_TO_POINTER(handle_id));
if (!handle) {
ret = -1;
goto end;
diff --git a/lib/values.c b/lib/values.c
index 345ef71..bd217bf 100644
--- a/lib/values.c
+++ b/lib/values.c
@@ -226,7 +226,7 @@ struct bt_value *bt_value_map_copy(const struct bt_value *map_obj)
g_hash_table_iter_init(&iter, typed_map_obj->ght);
while (g_hash_table_iter_next(&iter, &key, &element_obj)) {
- const char *key_str = g_quark_to_string((unsigned long) key);
+ const char *key_str = g_quark_to_string(GPOINTER_TO_UINT(key));
element_obj_copy = bt_value_copy(element_obj);
@@ -357,7 +357,7 @@ bool bt_value_map_compare(const struct bt_value *object_a,
while (g_hash_table_iter_next(&iter, &key, &element_obj_a)) {
struct bt_value *element_obj_b;
- const char *key_str = g_quark_to_string((unsigned long) key);
+ const char *key_str = g_quark_to_string(GPOINTER_TO_UINT(key));
element_obj_b = bt_value_map_get(object_b, key_str);
@@ -1156,7 +1156,7 @@ enum bt_value_status bt_value_map_foreach(const struct bt_value *map_obj,
g_hash_table_iter_init(&iter, typed_map_obj->ght);
while (g_hash_table_iter_next(&iter, &key, &element_obj)) {
- const char *key_str = g_quark_to_string((unsigned long) key);
+ const char *key_str = g_quark_to_string(GPOINTER_TO_UINT(key));
if (!cb(key_str, element_obj, data)) {
ret = BT_VALUE_STATUS_CANCELLED;
diff --git a/types/enum.c b/types/enum.c
index ec10a16..455e60a 100644
--- a/types/enum.c
+++ b/types/enum.c
@@ -270,7 +270,7 @@ GArray *bt_enum_quark_to_range_set(const struct declaration_enum *enum_declarati
GQuark q)
{
return g_hash_table_lookup(enum_declaration->table.quark_to_range_set,
- (gconstpointer) (unsigned long) q);
+ (gconstpointer) GUINT_TO_POINTER(q));
}
static
@@ -319,12 +319,12 @@ void bt_enum_signed_insert(struct declaration_enum *enum_declaration,
}
array = g_hash_table_lookup(enum_declaration->table.quark_to_range_set,
- (gconstpointer) (unsigned long) q);
+ (gconstpointer) GUINT_TO_POINTER(q));
if (!array) {
array = g_array_sized_new(FALSE, TRUE,
sizeof(struct enum_range), 1);
g_hash_table_insert(enum_declaration->table.quark_to_range_set,
- (gpointer) (unsigned long) q,
+ GUINT_TO_POINTER(q),
array);
}
g_array_set_size(array, array->len + 1);
@@ -354,12 +354,12 @@ void bt_enum_unsigned_insert(struct declaration_enum *enum_declaration,
}
array = g_hash_table_lookup(enum_declaration->table.quark_to_range_set,
- (gconstpointer) (unsigned long) q);
+ (gconstpointer) GUINT_TO_POINTER(q));
if (!array) {
array = g_array_sized_new(FALSE, TRUE,
sizeof(struct enum_range), 1);
g_hash_table_insert(enum_declaration->table.quark_to_range_set,
- (gpointer) (unsigned long) q,
+ GUINT_TO_POINTER(q),
array);
}
g_array_set_size(array, array->len + 1);
diff --git a/types/struct.c b/types/struct.c
index 7eaa424..e2f6a1e 100644
--- a/types/struct.c
+++ b/types/struct.c
@@ -196,8 +196,8 @@ void bt_struct_declaration_add_field(struct declaration_struct *struct_declarati
field->declaration = field_declaration;
/* Keep index in hash rather than pointer, because array can relocate */
g_hash_table_insert(struct_declaration->fields_by_name,
- (gpointer) (unsigned long) field->name,
- (gpointer) index);
+ GUINT_TO_POINTER(field->name),
+ GUINT_TO_POINTER(index));
/*
* Alignment of structure is the max alignment of declarations contained
* therein.
@@ -219,11 +219,11 @@ int bt_struct_declaration_lookup_field_index(struct declaration_struct *struct_d
gboolean found;
found = g_hash_table_lookup_extended(struct_declaration->fields_by_name,
- (gconstpointer) (unsigned long) field_name,
+ (gconstpointer) GUINT_TO_POINTER(field_name),
NULL, &index);
if (!found)
return -1;
- return (int) (unsigned long) index;
+ return GPOINTER_TO_INT(index);
}
/*
diff --git a/types/types.c b/types/types.c
index 41c6b8e..f58d2e4 100644
--- a/types/types.c
+++ b/types/types.c
@@ -56,7 +56,7 @@ struct bt_declaration *
struct declaration_scope *scope)
{
return g_hash_table_lookup(scope->typedef_declarations,
- (gconstpointer) (unsigned long) declaration_name);
+ (gconstpointer) GUINT_TO_POINTER(declaration_name));
}
struct bt_declaration *bt_lookup_declaration(GQuark declaration_name,
@@ -85,7 +85,7 @@ int bt_register_declaration(GQuark name, struct bt_declaration *declaration,
return -EEXIST;
g_hash_table_insert(scope->typedef_declarations,
- (gpointer) (unsigned long) name,
+ GUINT_TO_POINTER(name),
declaration);
bt_declaration_ref(declaration);
return 0;
@@ -97,7 +97,7 @@ struct bt_definition *
struct definition_scope *scope)
{
return g_hash_table_lookup(scope->definitions,
- (gconstpointer) (unsigned long) field_name);
+ (gconstpointer) GUINT_TO_POINTER(field_name));
}
/*
@@ -273,7 +273,7 @@ int bt_register_field_definition(GQuark field_name, struct bt_definition *defini
return -EEXIST;
g_hash_table_insert(scope->definitions,
- (gpointer) (unsigned long) field_name,
+ GUINT_TO_POINTER(field_name),
definition);
/* Don't keep reference on definition */
return 0;
@@ -340,7 +340,7 @@ struct declaration_struct *bt_lookup_struct_declaration_scope(GQuark struct_name
struct declaration_scope *scope)
{
return g_hash_table_lookup(scope->struct_declarations,
- (gconstpointer) (unsigned long) struct_name);
+ (gconstpointer) GUINT_TO_POINTER(struct_name));
}
struct declaration_struct *bt_lookup_struct_declaration(GQuark struct_name,
@@ -372,7 +372,7 @@ int bt_register_struct_declaration(GQuark struct_name,
return -EEXIST;
g_hash_table_insert(scope->struct_declarations,
- (gpointer) (unsigned long) struct_name,
+ GUINT_TO_POINTER(struct_name),
struct_declaration);
bt_declaration_ref(&struct_declaration->p);
@@ -389,7 +389,7 @@ struct declaration_untagged_variant *
struct declaration_scope *scope)
{
return g_hash_table_lookup(scope->variant_declarations,
- (gconstpointer) (unsigned long) variant_name);
+ (gconstpointer) GUINT_TO_POINTER(variant_name));
}
struct declaration_untagged_variant *
@@ -422,7 +422,7 @@ int bt_register_variant_declaration(GQuark variant_name,
return -EEXIST;
g_hash_table_insert(scope->variant_declarations,
- (gpointer) (unsigned long) variant_name,
+ GUINT_TO_POINTER(variant_name),
untagged_variant_declaration);
bt_declaration_ref(&untagged_variant_declaration->p);
@@ -440,7 +440,7 @@ struct declaration_enum *
struct declaration_scope *scope)
{
return g_hash_table_lookup(scope->enum_declarations,
- (gconstpointer) (unsigned long) enum_name);
+ (gconstpointer) GUINT_TO_POINTER(enum_name));
}
struct declaration_enum *
@@ -473,7 +473,7 @@ int bt_register_enum_declaration(GQuark enum_name,
return -EEXIST;
g_hash_table_insert(scope->enum_declarations,
- (gpointer) (unsigned long) enum_name,
+ GUINT_TO_POINTER(enum_name),
enum_declaration);
bt_declaration_ref(&enum_declaration->p);
diff --git a/types/variant.c b/types/variant.c
index 3972d78..f660e18 100644
--- a/types/variant.c
+++ b/types/variant.c
@@ -227,8 +227,8 @@ void bt_untagged_variant_declaration_add_field(struct declaration_untagged_varia
field->declaration = field_declaration;
/* Keep index in hash rather than pointer, because array can relocate */
g_hash_table_insert(untagged_variant_declaration->fields_by_tag,
- (gpointer) (unsigned long) field->name,
- (gpointer) index);
+ GUINT_TO_POINTER(field->name),
+ GUINT_TO_POINTER(index));
/*
* Alignment of variant is based on the alignment of its currently
* selected choice, so we leave variant alignment as-is (statically
@@ -244,13 +244,13 @@ bt_untagged_variant_declaration_get_field_from_tag(struct declaration_untagged_v
found = g_hash_table_lookup_extended(
untagged_variant_declaration->fields_by_tag,
- (gconstpointer) (unsigned long) tag, NULL, &index);
+ (gconstpointer) GUINT_TO_POINTER(tag), NULL, &index);
if (!found) {
return NULL;
}
- return &g_array_index(untagged_variant_declaration->fields, struct declaration_field, (unsigned long)index);
+ return &g_array_index(untagged_variant_declaration->fields, struct declaration_field, GPOINTER_TO_UINT(index));
}
/*
@@ -279,7 +279,7 @@ struct bt_definition *bt_variant_get_current_field(struct definition_variant *va
assert(tag_array->len == 1);
tag = g_array_index(tag_array, GQuark, 0);
if (!g_hash_table_lookup_extended(variant_declaration->untagged_variant->fields_by_tag,
- (gconstpointer) (unsigned long) tag,
+ (gconstpointer) GUINT_TO_POINTER(tag),
&orig_key,
&value)) {
/* Cannot find matching field. */
@@ -287,7 +287,7 @@ struct bt_definition *bt_variant_get_current_field(struct definition_variant *va
g_quark_to_string(tag));
return NULL;
}
- index = (unsigned long) value;
+ index = GPOINTER_TO_UINT(value);
variant->current_field = g_ptr_array_index(variant->fields, index);
return variant->current_field;
}
--
2.7.4
More information about the lttng-dev
mailing list