callisto-debian/gtk3
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

inspector: be careful about gsize vs guint64

Browse Source 
Since gsize doesn't work as a typename in a ui file,
we have to be careful not to pass pointers to wrongly
sized variables when getting the guint64 values out
of the model.

https://bugzilla.gnome.org/show_bug.cgi?id=742664
This commit is contained in:
Matthias Clasen
2015-01-09 14:44:26 -05:00
parent 9a38fb79e5
commit c2ff89bbe3
1 changed files with 11 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
gtk/inspector/resource-list.c
View File

@ -83,6 +83,7 @@ load_resources_recurse (GtkInspectorResourceList *sl,
gchar **names;
gint i;
GtkTreeIter iter;
guint64 stored_size;
names = g_resources_enumerate_children (path, 0, NULL);
for (i = 0; names[i]; i++)
@ -124,9 +125,10 @@ load_resources_recurse (GtkInspectorResourceList *sl,
*size_out += size;
}
stored_size = size;
gtk_tree_store_set (sl->priv->model, &iter,
COLUMN_COUNT, count,
COLUMN_SIZE, size,
COLUMN_SIZE, stored_size,
-1);
g_free (p);
@ -147,6 +149,7 @@ populate_details (GtkInspectorResourceList *rl,
gconstpointer data;
gint count;
gsize size;
guint64 stored_size;
GError *error = NULL;
gchar *markup;
@ -157,8 +160,9 @@ populate_details (GtkInspectorResourceList *rl,
COLUMN_PATH, &path,
COLUMN_NAME, &name,
COLUMN_COUNT, &count,
COLUMN_SIZE, &size,
COLUMN_SIZE, &stored_size,
-1);
size = stored_size;
if (g_str_has_suffix (path, "/"))
{
@ -340,10 +344,12 @@ size_data_func (GtkTreeViewColumn *col,
GtkTreeIter *iter,
gpointer data)
{
gint size;
gsize size;
guint64 stored_size;
gchar *text;
gtk_tree_model_get (model, iter, COLUMN_SIZE, &size, -1);
gtk_tree_model_get (model, iter, COLUMN_SIZE, &stored_size, -1);
size = stored_size;
text = g_format_size (size);
g_object_set (cell, "text", text, NULL);
g_free (text);