]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
kconfig: gconf: remove unnecessary NULL checks for tree1 and tree2
authorMasahiro Yamada <masahiroy@kernel.org>
Tue, 24 Jun 2025 15:05:07 +0000 (00:05 +0900)
committerMasahiro Yamada <masahiroy@kernel.org>
Tue, 1 Jul 2025 11:07:34 +0000 (20:07 +0900)
The tree1 and tree2 variables are initialized earlier in
init_tree_model(), so the NULL checks are redundant.

Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
scripts/kconfig/gconf.c

index 432a467e3250a98c669afc987285e416e0a8a8a0..2ab000adccedb709461f1b0c05f7b1de24b63752 100644 (file)
@@ -678,8 +678,7 @@ void on_split_clicked(GtkButton * button, gpointer user_data)
        gtk_widget_show(tree1_w);
        gtk_window_get_default_size(GTK_WINDOW(main_wnd), &w, &h);
        gtk_paned_set_position(GTK_PANED(hpaned), w / 2);
-       if (tree2)
-               gtk_tree_store_clear(tree2);
+       gtk_tree_store_clear(tree2);
        display_list();
 
        /* Disable back btn, like in full mode. */
@@ -691,8 +690,7 @@ void on_full_clicked(GtkButton * button, gpointer user_data)
 {
        view_mode = FULL_VIEW;
        gtk_widget_hide(tree1_w);
-       if (tree2)
-               gtk_tree_store_clear(tree2);
+       gtk_tree_store_clear(tree2);
        display_tree(&rootmenu);
        gtk_widget_set_sensitive(back_btn, FALSE);
 }
@@ -1298,8 +1296,7 @@ static void display_tree(struct menu *menu)
 /* Display a part of the tree starting at current node (single/split view) */
 static void display_tree_part(void)
 {
-       if (tree2)
-               gtk_tree_store_clear(tree2);
+       gtk_tree_store_clear(tree2);
        if (view_mode == SINGLE_VIEW)
                display_tree(current);
        else if (view_mode == SPLIT_VIEW)
@@ -1312,8 +1309,7 @@ static void display_tree_part(void)
 /* Display the list in the left frame (split view) */
 static void display_list(void)
 {
-       if (tree1)
-               gtk_tree_store_clear(tree1);
+       gtk_tree_store_clear(tree1);
 
        tree = tree1;
        display_tree(&rootmenu);