Remove redundant gtk_widget_show gtk4

Sun, 28 Aug 2022 01:17:08 -0500

author
Elliott Sales de Andrade <quantum.analyst@gmail.com>
date
Sun, 28 Aug 2022 01:17:08 -0500
branch
gtk4
changeset 41610
412ebdf7158c
parent 41609
40c35d54cc66
child 41611
518ed33dc53a

Remove redundant gtk_widget_show

These widgets were just created, or never hidden. I did leave the show on the video widgets, as I don't know where they eventually originate.

Testing Done:
Compiled only.

Reviewed at https://reviews.imfreedom.org/r/1668/

pidgin/gtkblist.c file | annotate | diff | comparison | revisions
pidgin/gtkmedia.c file | annotate | diff | comparison | revisions
pidgin/pidginaddchatdialog.c file | annotate | diff | comparison | revisions
pidgin/prefs/pidginproxyprefs.c file | annotate | diff | comparison | revisions
--- a/pidgin/gtkblist.c	Sun Aug 28 01:16:13 2022 -0500
+++ b/pidgin/gtkblist.c	Sun Aug 28 01:17:08 2022 -0500
@@ -3062,8 +3062,6 @@
 						);
 
 	gtkblist->treeview = gtk_tree_view_new_with_model(GTK_TREE_MODEL(gtkblist->treemodel));
-
-	gtk_widget_show(gtkblist->treeview);
 	gtk_widget_set_name(gtkblist->treeview, "pidgin_blist_treeview");
 
 	icon_theme = gtk_icon_theme_get_for_display(gdk_display_get_default());
--- a/pidgin/gtkmedia.c	Sun Aug 28 01:16:13 2022 -0500
+++ b/pidgin/gtkmedia.c	Sun Aug 28 01:17:08 2022 -0500
@@ -592,8 +592,6 @@
 			G_CALLBACK(destroy_parent_widget_cb),
 			volume_widget);
 
-	gtk_widget_show(volume_widget);
-
 	return volume_widget;
 }
 
@@ -624,8 +622,6 @@
 	g_object_set_data_full(G_OBJECT(win), "session-id",
 		g_strdup(_sid), g_free);
 
-	gtk_widget_show(keypad);
-
 	return keypad;
 }
 
@@ -664,7 +660,6 @@
 		gtkmedia->priv->hold =
 				gtk_toggle_button_new_with_mnemonic(_("_Hold"));
 		gtk_box_prepend(GTK_BOX(button_widget), gtkmedia->priv->hold);
-		gtk_widget_show(gtkmedia->priv->hold);
 		gtk_actionable_set_action_name(
 				GTK_ACTIONABLE(gtkmedia->priv->hold),
 				"win.Hold");
@@ -719,7 +714,6 @@
 		gtkmedia->priv->pause =
 				gtk_toggle_button_new_with_mnemonic(_("_Pause"));
 		gtk_box_prepend(GTK_BOX(button_widget), gtkmedia->priv->pause);
-		gtk_widget_show(gtkmedia->priv->pause);
 		gtk_actionable_set_action_name(
 				GTK_ACTIONABLE(gtkmedia->priv->pause),
 				"win.Pause");
@@ -738,7 +732,6 @@
 		gtkmedia->priv->mute =
 				gtk_toggle_button_new_with_mnemonic(_("_Mute"));
 		gtk_box_prepend(GTK_BOX(button_widget), gtkmedia->priv->mute);
-		gtk_widget_show(gtkmedia->priv->mute);
 		gtk_actionable_set_action_name(
 				GTK_ACTIONABLE(gtkmedia->priv->mute),
 				"win.Mute");
@@ -759,13 +752,14 @@
 				gtkmedia);
 	}
 
-	if (send_widget != NULL)
+	if (send_widget != NULL) {
 		gtkmedia->priv->send_widget = send_widget;
-	if (recv_widget != NULL)
+	}
+	if (recv_widget != NULL) {
 		gtkmedia->priv->recv_widget = recv_widget;
+	}
 	if (button_widget != NULL) {
 		gtkmedia->priv->button_widget = button_widget;
-		gtk_widget_show(GTK_WIDGET(button_widget));
 	}
 
 	if (purple_media_is_initiator(media, sid, NULL) == FALSE) {
@@ -783,8 +777,6 @@
 	} else if (type & PURPLE_MEDIA_AUDIO) {
 		gtk_window_set_icon_name(GTK_WINDOW(gtkmedia), "audio-call");
 	}
-
-	gtk_widget_show(gtkmedia->priv->display);
 }
 
 static void
@@ -915,8 +907,8 @@
 			purple_buddy_get_contact_alias(buddy) : screenname;
 	gtk_window_set_title(GTK_WINDOW(gtkmedia), alias);
 
-	if (purple_media_is_initiator(media, NULL, NULL) == TRUE)
-		gtk_widget_show(GTK_WIDGET(gtkmedia));
+	gtk_widget_set_visible(GTK_WIDGET(gtkmedia),
+	                       purple_media_is_initiator(media, NULL, NULL));
 
 	return TRUE;
 }
--- a/pidgin/pidginaddchatdialog.c	Sun Aug 28 01:16:13 2022 -0500
+++ b/pidgin/pidginaddchatdialog.c	Sun Aug 28 01:17:08 2022 -0500
@@ -178,8 +178,6 @@
 
 	g_hash_table_destroy(defaults);
 
-	gtk_widget_show(dialog->dynamic_box);
-
 	pidgin_add_chat_dialog_validate(dialog);
 }
 
--- a/pidgin/prefs/pidginproxyprefs.c	Sun Aug 28 01:16:13 2022 -0500
+++ b/pidgin/prefs/pidginproxyprefs.c	Sun Aug 28 01:17:08 2022 -0500
@@ -59,11 +59,9 @@
 	PidginProxyPrefs *prefs = data;
 	const char *proxy = value;
 
-	if (!purple_strequal(proxy, "none") && !purple_strequal(proxy, "envvar")) {
-		gtk_widget_show(prefs->options);
-	} else {
-		gtk_widget_hide(prefs->options);
-	}
+	gtk_widget_set_visible(prefs->options,
+	                       !purple_strequal(proxy, "none") &&
+	                       !purple_strequal(proxy, "envvar"));
 }
 
 static void

mercurial