Sun, 28 Aug 2022 01:17:08 -0500
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/
--- 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