Got some complaints from valgrind about leaks from calling g_object_get_property() without freeing the value afterwards. g_value_unset() should be used. The api docs mention that g_object_get() is what really should be used in most places except language bindings though, so maybe that should be changed too?

Here are the places the function call is used:

[kmaraas@rivendell gail]$ grep g_object_get_property *.c
gailbooleancell.c:  g_object_get_property (G_OBJECT(cell->renderer), "active", &new_value);
gailentry.c:      g_object_get_property (obj, "editable", &value);
gailentry.c:  g_object_get_property (G_OBJECT (entry), "editable", &value);
gailtreeview.c:              g_object_get_property (cur_renderer->data, *prop_list, &value);
gailwidget.c:  g_object_get_property (G_OBJECT (widget), "parent", &value);