[gnome-applets] gweather: simplify find_next_clicked



commit 9a69b55a0b86d717432891d752aa3d955fd149c7
Author: Alberts Muktupāvels <alberts muktupavels gmail com>
Date:   Wed Mar 11 03:24:24 2020 +0200

    gweather: simplify find_next_clicked
    
    We are using default selection mode - GTK_SELECTION_SINGLE. There
    is no need to check selected row count and documentation says that
    gtk_tree_selection_get_selected would not work with
    GTK_SELECTION_MULTIPLE.
    
    CID 1491725.

 gweather/src/gweather-pref.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)
---
diff --git a/gweather/src/gweather-pref.c b/gweather/src/gweather-pref.c
index 5ef4b3d0b..eb795fb79 100644
--- a/gweather/src/gweather-pref.c
+++ b/gweather/src/gweather-pref.c
@@ -595,15 +595,13 @@ find_next_clicked (GtkButton *button, GWeatherPref *pref)
 
        selection = gtk_tree_view_get_selection (GTK_TREE_VIEW (tree));
 
-       if (gtk_tree_selection_count_selected_rows (selection) >= 1) {
-               gtk_tree_selection_get_selected (selection, &model, &iter);
+       if (gtk_tree_selection_get_selected (selection, &model, &iter)) {
                /* Select next or select parent */
                if (!gtk_tree_model_iter_next (model, &iter)) {
                        iter_parent = iter;
                        if (!gtk_tree_model_iter_parent (model, &iter, &iter_parent) || 
!gtk_tree_model_iter_next (model, &iter))
                                gtk_tree_model_get_iter_first (model, &iter);
                }
-
        } else {
                gtk_tree_model_get_iter_first (model, &iter);
        }


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]