[easytag/wip/glist: 6/7] Refactor list handling in scan.c

[David King <davidk src gnome org>]

commit daa7ce91c67b172b208a46c4231c735c78d4dcbf
Author: David King <amigadave amigadave com>
Date:   Wed May 22 22:05:38 2013 +0100

    Refactor list handling in scan.c

 src/scan.c |   99 ++++++++++++++++++++---------------------------------------
 1 files changed, 34 insertions(+), 65 deletions(-)
---
diff --git a/src/scan.c b/src/scan.c
index a98503b..49031fd 100644
--- a/src/scan.c
+++ b/src/scan.c
@@ -3534,10 +3534,8 @@ Mask_Editor_List_Row_Selected (GtkTreeSelection* selection, gpointer data)
     /*
      * At some point, we might get called when no rows are selected?
      */
-    if (g_list_length(selectedRows) == 0)
+    if (!selectedRows)
     {
-        g_list_foreach(selectedRows, (GFunc) gtk_tree_path_free, NULL);
-        g_list_free(selectedRows);
         g_signal_handlers_unblock_by_func(G_OBJECT(MaskEditorEntry),
                                           G_CALLBACK(Mask_Editor_Entry_Changed),NULL);
         return;
@@ -3560,6 +3558,8 @@ Mask_Editor_List_Row_Selected (GtkTreeSelection* selection, gpointer data)
 
     g_signal_handlers_unblock_by_func(G_OBJECT(MaskEditorEntry),
                                       G_CALLBACK(Mask_Editor_Entry_Changed),NULL);
+
+    g_list_free_full (selectedRows, (GDestroyNotify)gtk_tree_path_free);
 }
 
 
@@ -3594,11 +3594,11 @@ Mask_Editor_List_Duplicate (void)
 {
     gchar *text = NULL;
     GList *selectedRows;
+    GList *l;
     GList *toInsert = NULL;
     GtkTreeSelection *selection;
     GtkTreeIter rowIter;
     GtkTreeModel *treeModel;
-    gboolean valid;
 
     g_return_if_fail (MaskEditorList != NULL);
 
@@ -3606,53 +3606,44 @@ Mask_Editor_List_Duplicate (void)
     selectedRows = gtk_tree_selection_get_selected_rows(selection, NULL);
     treeModel = gtk_tree_view_get_model(GTK_TREE_VIEW(MaskEditorList));
 
-    if (g_list_length(selectedRows) == 0)
+    if (!selectedRows)
     {
         Log_Print(LOG_ERROR,_("Copy: No row selected"));
-        g_list_foreach(selectedRows, (GFunc) gtk_tree_path_free, NULL);
-        g_list_free(selectedRows);
         return;
     }
 
     /* Loop through selected rows, duplicating them into a GList
      * We cannot directly insert because the paths in selectedRows
      * get out of date after an insertion */
-    while (selectedRows)
+    for (l = selectedRows; l != NULL; l = g_list_next (l))
     {
-        valid = gtk_tree_model_get_iter(treeModel, &rowIter, (GtkTreePath*) selectedRows->data);
-        if (valid)
+        if (gtk_tree_model_get_iter (treeModel, &rowIter,
+                                     (GtkTreePath*)l->data))
         {
             gtk_tree_model_get(treeModel, &rowIter, MASK_EDITOR_TEXT, &text, -1);
             toInsert = g_list_append(toInsert, text);
         }
-
-        selectedRows = selectedRows->next;
-        if (!selectedRows) break;
     }
 
     /* Duplicate the relevant entries, by looping through the list backwards
      * (to preserve original order) */
     toInsert = g_list_last(toInsert);
-    while (toInsert)
-    {
-        gtk_list_store_insert(GTK_LIST_STORE(treeModel), &rowIter, 0);
-        gtk_list_store_set(GTK_LIST_STORE(treeModel), &rowIter, MASK_EDITOR_TEXT, (gchar*) toInsert->data, 
-1);
-        g_free(toInsert->data);
 
-        toInsert = toInsert->prev;
-        if (!toInsert) break;
+    for (l = toInsert; l != NULL; l = g_list_previous (l))
+    {
+        gtk_list_store_insert_with_values (GTK_LIST_STORE(treeModel), &rowIter,
+                                           0, MASK_EDITOR_TEXT,
+                                           (gchar *)l->data, -1);
     }
+
     // Set focus to the last inserted line
     if (toInsert)
         Mask_Editor_List_Set_Row_Visible(treeModel,&rowIter);
 
     /* Free data no longer needed */
-    selectedRows = g_list_first(selectedRows);
     toInsert = g_list_first(toInsert);
-    g_list_foreach(selectedRows, (GFunc) gtk_tree_path_free, NULL);
-    g_list_free(selectedRows);
-    g_list_foreach(toInsert, (GFunc) g_free, NULL);
-    g_list_free(toInsert);
+    g_list_free_full (selectedRows, (GDestroyNotify)gtk_tree_path_free);
+    g_list_free_full (toInsert, (GDestroyNotify)g_free);
 }
 
 static void
@@ -3749,12 +3740,11 @@ Mask_Editor_List_Move_Up (void)
 {
     GtkTreeSelection *selection;
     GList *selectedRows;
-    GList *selectedRowsCopy;
+    GList *l;
     GtkTreeIter currentFile;
     GtkTreeIter nextFile;
     GtkTreePath *currentPath;
     GtkTreeModel *treemodel;
-    gboolean valid;
 
     g_return_if_fail (MaskEditorList != NULL);
 
@@ -3762,21 +3752,16 @@ Mask_Editor_List_Move_Up (void)
     treemodel = gtk_tree_view_get_model(GTK_TREE_VIEW(MaskEditorList));
     selectedRows = gtk_tree_selection_get_selected_rows(selection, NULL);
 
-    if (g_list_length(selectedRows) == 0)
+    if (!selectedRows)
     {
         Log_Print(LOG_ERROR,_("Move Up: No row selected"));
-        g_list_foreach(selectedRows, (GFunc)gtk_tree_path_free, NULL);
-        g_list_free(selectedRows);
         return;
     }
 
-    selectedRowsCopy = selectedRows;
-
-    while (selectedRows)
+    for (l = selectedRows; l != NULL; l = g_list_next (l))
     {
-        currentPath = (GtkTreePath*) selectedRows->data;
-        valid = gtk_tree_model_get_iter(treemodel, &currentFile, currentPath);
-        if (valid)
+        currentPath = (GtkTreePath *)l->data;
+        if (gtk_tree_model_get_iter(treemodel, &currentFile, currentPath))
         {
             /* Find the entry above the node... */
             if (gtk_tree_path_prev(currentPath))
@@ -3786,13 +3771,9 @@ Mask_Editor_List_Move_Up (void)
                 gtk_list_store_swap(GTK_LIST_STORE(treemodel), &currentFile, &nextFile);
             }
         }
-
-        selectedRows = selectedRows->next;
-        if (!selectedRows) break;
     }
 
-    g_list_foreach(selectedRowsCopy, (GFunc)gtk_tree_path_free, NULL);
-    g_list_free(selectedRowsCopy);
+    g_list_free_full (selectedRows, (GDestroyNotify)gtk_tree_path_free);
 }
 
 /*
@@ -3803,12 +3784,11 @@ Mask_Editor_List_Move_Down (void)
 {
     GtkTreeSelection *selection;
     GList *selectedRows;
-    GList *selectedRowsCopy;
+    GList *l;
     GtkTreeIter currentFile;
     GtkTreeIter nextFile;
     GtkTreePath *currentPath;
     GtkTreeModel *treemodel;
-    gboolean valid;
 
     g_return_if_fail (MaskEditorList != NULL);
 
@@ -3816,34 +3796,26 @@ Mask_Editor_List_Move_Down (void)
     treemodel = gtk_tree_view_get_model(GTK_TREE_VIEW(MaskEditorList));
     selectedRows = gtk_tree_selection_get_selected_rows(selection, NULL);
 
-    if (g_list_length(selectedRows) == 0)
+    if (!selectedRows)
     {
         Log_Print(LOG_ERROR,_("Move Down: No row selected"));
-        g_list_foreach(selectedRows, (GFunc)gtk_tree_path_free, NULL);
-        g_list_free(selectedRows);
         return;
     }
 
-    selectedRowsCopy = selectedRows;
-
-    while (selectedRows)
+    for (l = selectedRows; l != NULL; l = g_list_next (l))
     {
-        currentPath = (GtkTreePath*) selectedRows->data;
-        valid = gtk_tree_model_get_iter(treemodel, &currentFile, currentPath);
-        if (valid)
+        currentPath = (GtkTreePath *)l->data;
+
+        if (gtk_tree_model_get_iter(treemodel, &currentFile, currentPath))
         {
             /* Find the entry below the node and swap the two nodes by iter */
             gtk_tree_path_next(currentPath);
             if (gtk_tree_model_get_iter(treemodel, &nextFile, currentPath))
                 gtk_list_store_swap(GTK_LIST_STORE(treemodel), &currentFile, &nextFile);
         }
-
-        if (!selectedRows->next) break;
-        selectedRows = selectedRows->next;
     }
 
-    g_list_foreach(selectedRowsCopy, (GFunc)gtk_tree_path_free, NULL);
-    g_list_free(selectedRowsCopy);
+    g_list_free_full (selectedRows, (GDestroyNotify)gtk_tree_path_free);
 }
 
 /*
@@ -3962,7 +3934,6 @@ Mask_Editor_Entry_Changed (void)
     GList *selectedRows;
     GtkTreeIter row;
     const gchar* text;
-    gboolean valid;
 
     g_return_if_fail (MaskEditorList != NULL);
 
@@ -3970,22 +3941,20 @@ Mask_Editor_Entry_Changed (void)
     treemodel = gtk_tree_view_get_model(GTK_TREE_VIEW(MaskEditorList));
     selectedRows = gtk_tree_selection_get_selected_rows(selection, NULL);
 
-    if (g_list_length(selectedRows) == 0)
+    if (!selectedRows)
     {
-        g_list_foreach(selectedRows, (GFunc) gtk_tree_path_free, NULL);
-        g_list_free(selectedRows);
         return;
     }
 
     firstSelected = (GtkTreePath *)g_list_first(selectedRows)->data;
     text = gtk_entry_get_text(GTK_ENTRY(MaskEditorEntry));
 
-    valid = gtk_tree_model_get_iter(treemodel, &row, firstSelected);
-    if (valid)
+    if (gtk_tree_model_get_iter (treemodel, &row, firstSelected))
+    {
         gtk_list_store_set(GTK_LIST_STORE(treemodel), &row, MASK_EDITOR_TEXT, text, -1);
+    }
 
-    g_list_foreach(selectedRows, (GFunc) gtk_tree_path_free, NULL);
-    g_list_free(selectedRows);
+    g_list_free_full (selectedRows, (GDestroyNotify)gtk_tree_path_free);
 }
 
 /*