From cecc7efece527e09afdfb59dc24cbf6086bd2654 Mon Sep 17 00:00:00 2001 From: Javier Date: Sun, 9 Feb 2014 00:50:39 +0100 Subject: improve client memory management warnings --- libtopmenu-client/topmenu-client.c | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) (limited to 'libtopmenu-client') diff --git a/libtopmenu-client/topmenu-client.c b/libtopmenu-client/topmenu-client.c index 63e27bc..771920e 100644 --- a/libtopmenu-client/topmenu-client.c +++ b/libtopmenu-client/topmenu-client.c @@ -82,6 +82,10 @@ void topmenu_client_connect_window_widget(GdkWindow *window, GtkWidget *widget) G_CALLBACK(handle_widget_button_event), plug, 0); gtk_widget_show(GTK_WIDGET(plug)); + // Gtk+ should keep a reference to plug as it is a "top level" widget. + // Otherwise I failed to parse the documentation. + g_warn_if_fail(G_OBJECT(plug)->ref_count == 1); + Window plug_xwin = gtk_plug_get_id(plug); Atom atom = XInternAtom(display, ATOM_TOPMENU_WINDOW, False); @@ -90,7 +94,8 @@ void topmenu_client_connect_window_widget(GdkWindow *window, GtkWidget *widget) XA_WINDOW, 32, PropModeReplace, (unsigned char*)&plug_xwin, 1); - g_object_set_data_full(G_OBJECT(window), OBJECT_DATA_KEY_PLUG, plug, (GDestroyNotify)>k_widget_destroy); + g_object_set_data_full(G_OBJECT(window), OBJECT_DATA_KEY_PLUG, plug, + (GDestroyNotify) >k_widget_destroy); } void topmenu_client_disconnect_window(GdkWindow *window) @@ -109,5 +114,6 @@ void topmenu_client_disconnect_window(GdkWindow *window) XDeleteProperty(display, xwin, atom); - g_object_unref(plug); + g_warn_if_fail(G_OBJECT(plug)->ref_count == 1); + gtk_widget_destroy(plug); } -- cgit v1.2.3