From 6954cd68d9fe36277c528b92f2cff44fdeea68be Mon Sep 17 00:00:00 2001 From: Denys Vlasenko Date: Wed, 16 Feb 2011 16:16:48 +0100 Subject: abrt-gtk: trivial simplification Signed-off-by: Denys Vlasenko --- src/gui-gtk/abrt-gtk.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/src/gui-gtk/abrt-gtk.c b/src/gui-gtk/abrt-gtk.c index c72f0043..be90883f 100644 --- a/src/gui-gtk/abrt-gtk.c +++ b/src/gui-gtk/abrt-gtk.c @@ -237,7 +237,6 @@ void sanitize_cursor(GtkTreePath *preferred_path) if (path) { /* Cursor exists already */ - gtk_tree_path_free(path); goto ret; } @@ -251,7 +250,6 @@ void sanitize_cursor(GtkTreePath *preferred_path) gtk_tree_view_get_cursor(GTK_TREE_VIEW(s_treeview), &path, /* GtkTreeViewColumn** */ NULL); if (path) /* yes */ { - gtk_tree_path_free(path); goto ret; } } @@ -261,14 +259,19 @@ void sanitize_cursor(GtkTreePath *preferred_path) if (gtk_tree_model_get_iter_first(GTK_TREE_MODEL(s_dumps_list_store), &iter)) { /* We have at least one element, put cursor on it */ + + /* Get path from iter pointing to 1st element */ path = gtk_tree_model_get_path(GTK_TREE_MODEL(s_dumps_list_store), &iter); + + /* Use it to set cursor */ gtk_tree_view_set_cursor(GTK_TREE_VIEW(s_treeview), path, /* GtkTreeViewColumn *focus_column */ NULL, /* start_editing */ false); - gtk_tree_path_free(path); } /* else we have no elements */ ret: + gtk_tree_path_free(path); + /* Without this, the *header* of the list gets the focus. Ugly. */ gtk_widget_grab_focus(s_treeview); } -- cgit