Eliminate illegal gtk_widget_set_has_window() calls.
authorAndriy Grytsenko <andrej@rep.kiev.ua>
Mon, 18 Aug 2014 11:28:44 +0000 (14:28 +0300)
committerAndriy Grytsenko <andrej@rep.kiev.ua>
Mon, 18 Aug 2014 11:28:44 +0000 (14:28 +0300)
The gtk_widget_set_has_window() is illegal outside of widget init()
function. As exception it may be allowed right after gtk_event_box_new()
because GtkEventBox has it set to TRUE by default.

plugins/cpu/cpu.c
plugins/launchtaskbar.c
plugins/monitors/monitors.c
plugins/pager.c

index 1cf3fcb..5dfb5f8 100644 (file)
@@ -236,9 +236,9 @@ static GtkWidget *cpu_constructor(LXPanel *panel, config_setting_t *settings)
 
     /* Allocate top level widget and set into Plugin widget pointer. */
     p = gtk_event_box_new();
+    gtk_widget_set_has_window(p, FALSE);
     lxpanel_plugin_set_data(p, c, cpu_destructor);
     gtk_container_set_border_width(GTK_CONTAINER(p), 1);
-    gtk_widget_set_has_window(p, FALSE);
 
     /* Allocate drawing area as a child of top level widget.  Enable button press events. */
     c->da = gtk_drawing_area_new();
index c26c716..53f4175 100644 (file)
@@ -818,7 +818,6 @@ static GtkWidget *_launchtaskbar_constructor(LXPanel *panel, config_setting_t *s
 
     gtk_container_set_border_width(GTK_CONTAINER(p), 0);
     gtk_container_set_border_width(GTK_CONTAINER(ltbp->lb_icon_grid), 0);
-    gtk_widget_set_has_window(p, FALSE);
 
     /* Read parameters from the configuration file. */
     config_setting_lookup_int(settings, "LaunchTaskBarMode", &ltbp->mode);
index 6276f85..db584c1 100644 (file)
@@ -622,7 +622,6 @@ monitors_constructor(LXPanel *panel, config_setting_t *settings)
     p = gtk_hbox_new(TRUE, 2);
     lxpanel_plugin_set_data(p, mp, monitors_destructor);
     gtk_container_set_border_width(GTK_CONTAINER(p), 1);
-    gtk_widget_set_has_window(p, FALSE);
 
     /* First time we use this plugin : only display CPU usage */
     mp->displayed_monitors[CPU_POSITION] = 1;
index e333783..7d723da 100644 (file)
@@ -67,7 +67,6 @@ static GtkWidget *pager_constructor(LXPanel *panel, config_setting_t *settings)
     w = wnck_pager_new(NULL);
     g_return_val_if_fail(w != NULL, 0);
     p = gtk_alignment_new(0, 0, 1.0, 1.0);
-    gtk_widget_set_has_window(p, FALSE);
 
     /* we cannot configure pager until it added into widgets hierarchy */
     g_signal_connect(p, "realize", G_CALLBACK(on_realize), panel);