[gnome-applets] gweather: simplify find_next_clicked
- From: Alberts Muktupāvels <muktupavels src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-applets] gweather: simplify find_next_clicked
- Date: Wed, 11 Mar 2020 11:43:47 +0000 (UTC)
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]