diff options
| author | Matthias Clasen <mclasen@redhat.com> | 2015-07-06 21:59:10 -0400 |
|---|---|---|
| committer | Matthias Clasen <mclasen@redhat.com> | 2015-07-06 21:59:10 -0400 |
| commit | 3cbe6d86aff9a58431c529d40f0a932ce575c4c2 (patch) | |
| tree | 99f84aa8f32992e1795cb2fec0fc256b29593fa1 /gtk/gtkplacessidebar.c | |
| parent | 06f91028408205fa55ef24c5553d058f5feba3e3 (diff) | |
| download | gtk+-3cbe6d86aff9a58431c529d40f0a932ce575c4c2.tar.gz | |
place sidebar: minor change
We generally prefer for loops to iterate over lists.
Diffstat (limited to 'gtk/gtkplacessidebar.c')
| -rw-r--r-- | gtk/gtkplacessidebar.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/gtk/gtkplacessidebar.c b/gtk/gtkplacessidebar.c index 5f92fac93b..54271df828 100644 --- a/gtk/gtkplacessidebar.c +++ b/gtk/gtkplacessidebar.c @@ -2317,9 +2317,9 @@ rename_entry_changed (GtkEntry *entry, gtk_label_set_label (GTK_LABEL (sidebar->rename_error), ""); return; } + rows = gtk_container_get_children (GTK_CONTAINER (sidebar->list_box)); - l = rows; - while (l != NULL && !found) + for (l = rows; l && !found; l = l->next) { g_object_get (l->data, "place-type", &type, @@ -2334,15 +2334,12 @@ rename_entry_changed (GtkEntry *entry, g_free (uri); g_free (name); - - l = l->next; } + g_list_free (rows); gtk_widget_set_sensitive (sidebar->rename_button, !found); gtk_label_set_label (GTK_LABEL (sidebar->rename_error), found ? _("This name is already taken") : ""); - - g_list_free (rows); } static void |