values: Use v_pointer for g_value_set_gtype()

... and g_value_get_gtype(). G_TYPE_GTYPE is a pointer type, so it's
values should use the v_pointer member. This is especially true, because
the value collectors from varargs in gvaluecollector.h use that, too.

This should only cause issues when sizeof(glong) != sizeof(gpointer),
and I'm not aware of any such platform. Maybe win64?
This commit is contained in:
Benjamin Otte 2012-01-01 21:51:40 +01:00
parent 86cd3018e0
commit 9d52243790

View File

@ -1187,7 +1187,7 @@ g_value_set_gtype (GValue *value,
{ {
g_return_if_fail (G_VALUE_HOLDS_GTYPE (value)); g_return_if_fail (G_VALUE_HOLDS_GTYPE (value));
value->data[0].v_long = v_gtype; value->data[0].v_pointer = GSIZE_TO_POINTER (v_gtype);
} }
@ -1206,7 +1206,7 @@ g_value_get_gtype (const GValue *value)
{ {
g_return_val_if_fail (G_VALUE_HOLDS_GTYPE (value), 0); g_return_val_if_fail (G_VALUE_HOLDS_GTYPE (value), 0);
return value->data[0].v_long; return GPOINTER_TO_SIZE (value->data[0].v_pointer);
} }
/** /**