[easytag/wip/musicbrainz-support-merge: 66/75] Added comments to musicbrainz_dialog.c

[David King <davidk src gnome org>]

commit 3b39c517c9bce4591d56c4a3deacad63bc27da5c
Author: Abhinav <abhijangda hotmail com>
Date:   Fri Aug 15 12:54:44 2014 +0530

    Added comments to musicbrainz_dialog.c

 src/musicbrainz_dialog.c |   47 ++++++++++++++++++++++++++++++++++++++++++++++
 1 files changed, 47 insertions(+), 0 deletions(-)
---
diff --git a/src/musicbrainz_dialog.c b/src/musicbrainz_dialog.c
index f1df377..d4ab3d7 100755
--- a/src/musicbrainz_dialog.c
+++ b/src/musicbrainz_dialog.c
@@ -479,15 +479,20 @@ manual_search_callback (GObject *source, GAsyncResult *res,
 
         if (mbDialog)
         {
+            /* if mbDialog is a valid Dialog */
             dlg = ET_MUSICBRAINZ_DIALOG (mbDialog);
             priv = ET_MUSICBRAINZ_DIALOG_GET_PRIVATE (dlg);
 
             if (priv->search)
             {
+                /* If there exists some previous search then enable 
+                 * all toolbar buttons */
                 et_music_brainz_dialog_toolbar_buttons_set_sensitive (TRUE);
             }
             else
             {
+                /* If no previous search exists, disable all 
+                 * toolbar buttons */
                 et_music_brainz_dialog_toolbar_buttons_set_sensitive (FALSE);
             }
         }
@@ -510,6 +515,8 @@ manual_search_callback (GObject *source, GAsyncResult *res,
                                     gtk_combo_box_text_get_active_text (combo_box));
     et_music_brainz_dialog_stop_set_sensitive (FALSE);
     et_music_brainz_dialog_toolbar_buttons_set_sensitive (TRUE);
+    
+    /* Set the current search as Manual Search */
     priv->search = ET_MB_SEARCH_TYPE_MANUAL;
     et_mb_set_search_manual (&priv->search, thread_data->text_to_search,
                              priv->mb_tree_root, thread_data->type);
@@ -610,12 +617,15 @@ btn_fetch_more_clicked (GtkWidget *btn, gpointer user_data)
     if (mb_dialog_priv->search->type == ET_MB_SEARCH_TYPE_AUTOMATIC && 
         level <= 2)
     {
+        /* if current search is automatic search and its level is 
+         * greater than 1 then fetch more results for these albums */
         et_mb_entity_view_fetch_more_at_current_level (ET_MB_ENTITY_VIEW (mb_dialog_priv->entityView));
         return;
     }
 
     if (level != 1)
     {
+        /* If Level is not equal to 1 then do nothing */
         return;
     }
 
@@ -730,6 +740,7 @@ btn_manual_find_clicked (GtkWidget *btn, gpointer user_data)
 
     if (type == -1)
     {
+        /* Do nothing is if no type of entity is selected */
         return;
     }
 
@@ -976,15 +987,20 @@ selected_find_callback (GObject *source, GAsyncResult *res,
 
         if (mbDialog)
         {
+            /* if mbDialog is a valid Dialog */
             dlg = ET_MUSICBRAINZ_DIALOG (mbDialog);
             mb_dialog_priv = ET_MUSICBRAINZ_DIALOG_GET_PRIVATE (dlg);
 
             if (mb_dialog_priv->search)
             {
+                /* If there exists some previous search then enable 
+                 * all toolbar buttons */
                 et_music_brainz_dialog_toolbar_buttons_set_sensitive (TRUE);
             }
             else
             {
+                /* If no previous search exists, disable all 
+                 * toolbar buttons */
                 et_music_brainz_dialog_toolbar_buttons_set_sensitive (FALSE);
             }
         }
@@ -1011,6 +1027,7 @@ selected_find_callback (GObject *source, GAsyncResult *res,
         btn_close_clicked (NULL, NULL);
     }
 
+    /* Set search as selected search */
     et_mb_set_selected_search (&mb_dialog_priv->search,
                                thread_data->n_nodes,
                                ((SelectedFindThreadData *)user_data)->list_iter);
@@ -1046,6 +1063,7 @@ selected_find_thread_func (GSimpleAsyncResult *res, GObject *obj,
 
     while (iter)
     {
+        /* Search for each album in hashtable */
         children = g_node_n_children (thread_data->node);
 
         if (!et_musicbrainz_search ((gchar *)iter->data, MB_ENTITY_KIND_ALBUM,
@@ -1060,6 +1078,8 @@ selected_find_thread_func (GSimpleAsyncResult *res, GObject *obj,
             return;
         }
 
+        /* Update n_nodes array containing number of nodes 
+         * downloaded for an album */
         g_array_index (thread_data->n_nodes, int, i) += g_node_n_children (thread_data->node) - children;
  
         if (g_cancellable_is_cancelled (cancellable))
@@ -1191,6 +1211,7 @@ btn_selected_find_clicked (GtkWidget *button, gpointer data)
         return;
     }
 
+    /* Create hash table for fast search and insert operations */
     hash_table = g_hash_table_new (g_str_hash, g_str_equal);
 
     for (l = iter_list; l != NULL; l = g_list_next (l))
@@ -1293,15 +1314,20 @@ discid_search_callback (GObject *source, GAsyncResult *res,
 
         if (mbDialog)
         {
+            /* if mbDialog is a valid Dialog */
             dlg = ET_MUSICBRAINZ_DIALOG (mbDialog);
             mb_dialog_priv = ET_MUSICBRAINZ_DIALOG_GET_PRIVATE (dlg);
 
             if (mb_dialog_priv->search)
             {
+                 /* If there exists some previous search then enable 
+                 * all toolbar buttons */
                 et_music_brainz_dialog_toolbar_buttons_set_sensitive (TRUE);
             }
             else
             {
+                /* If no previous search exists, disable all 
+                 * toolbar buttons */
                 et_music_brainz_dialog_toolbar_buttons_set_sensitive (FALSE);
             }
         }
@@ -1430,6 +1456,7 @@ btn_discid_search_clicked (GtkWidget *button, gpointer data)
     et_music_brainz_dialog_stop_set_sensitive (TRUE);
     et_music_brainz_dialog_toolbar_buttons_set_sensitive (FALSE);
 #else
+    /* Show this dialog if libdiscid doesn't exist */
     GtkWidget *msg_dialog;
     
     msg_dialog = gtk_message_dialog_new (GTK_WINDOW (mbDialog), GTK_DIALOG_MODAL, 
@@ -1481,15 +1508,20 @@ freedbid_search_callback (GObject *source, GAsyncResult *res,
 
         if (mbDialog)
         {
+            /* if mbDialog is a valid Dialog */
             dlg = ET_MUSICBRAINZ_DIALOG (mbDialog);
             mb_dialog_priv = ET_MUSICBRAINZ_DIALOG_GET_PRIVATE (dlg);
 
             if (mb_dialog_priv->search)
             {
+                /* If there exists some previous search then enable 
+                 * all toolbar buttons */
                 et_music_brainz_dialog_toolbar_buttons_set_sensitive (TRUE);
             }
             else
             {
+                /* If no previous search exists, disable all 
+                 * toolbar buttons */
                 et_music_brainz_dialog_toolbar_buttons_set_sensitive (FALSE);
             }
         }
@@ -1796,6 +1828,7 @@ btn_apply_changes_clicked (GtkWidget *btn, gpointer data)
         return;
     }
 
+    /* Function should continue only if a track has been selected */
     if (((EtMbEntity *)track_iter_list->data)->type !=
         MB_ENTITY_KIND_TRACK)
     {
@@ -1811,12 +1844,14 @@ btn_apply_changes_clicked (GtkWidget *btn, gpointer data)
 
     if (album_entity)
     {
+        /* If there is a parent album node of track nodes */
         mb5_release_get_title (album_entity->entity, album, 
                                sizeof (album));
     }
 
     if (gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (gtk_builder_get_object (builder, "chk_use_dlm"))))
     {
+        /* Use DLM Algorithm to find the best match for applying tags */
         for (list_iter1 = track_iter_list; list_iter1;
              list_iter1 = g_list_next (list_iter1))
         {
@@ -1870,6 +1905,7 @@ btn_apply_changes_clicked (GtkWidget *btn, gpointer data)
     }
     else
     {
+        /* nth Tag is applied to nth selected file */
         for (list_iter1 = track_iter_list, list_iter2 = file_iter_list;
              list_iter1 && list_iter2; list_iter1 = g_list_next (list_iter1),
              list_iter2 = g_list_next (list_iter2))
@@ -1947,6 +1983,8 @@ et_apply_track_tag_to_et_file (Mb5Recording recording, EtMbEntity *album_entity,
     
     if (album_entity)
     {
+        /* Find the release which has same id as that of 
+         * parent release node */
         gchar id[NAME_MAX_SIZE];
         
         mb5_release_get_id (album_entity->entity, id, sizeof (id));
@@ -1967,6 +2005,9 @@ et_apply_track_tag_to_et_file (Mb5Recording recording, EtMbEntity *album_entity,
 
     if (release || size == 1)
     {
+        /* If the required release in the recording is found or
+         * number of release in recording is 1 
+         * then use that release's tags only */
         if (!release)
         {
             release = mb5_release_list_item (release_list, 0);
@@ -2187,6 +2228,7 @@ et_initialize_tag_choice_dialog (EtMusicBrainzDialogPrivate *mb_dialog_priv)
                                                                             "tag_choice_dialog"));
     tag_choice_list = GTK_WIDGET (gtk_builder_get_object (builder,
                                                           "tag_choice_treeview"));
+    /* Create new list store */
     list_store = gtk_list_store_new (TAG_CHOICE_COLS_N, G_TYPE_STRING,
                                      G_TYPE_STRING, G_TYPE_STRING,
                                      G_TYPE_STRING, G_TYPE_STRING,
@@ -2198,6 +2240,7 @@ et_initialize_tag_choice_dialog (EtMusicBrainzDialogPrivate *mb_dialog_priv)
                              mb_dialog_priv->tag_choice_store);
     g_object_unref (list_store);
 
+    /* Insert columns into tree view */
     renderer = gtk_cell_renderer_text_new ();
     column = gtk_tree_view_column_new_with_attributes ("Title",
                                                        renderer, "text",
@@ -2432,6 +2475,7 @@ mb5_search_error_callback (GObject *source, GAsyncResult *res,
 
     if (mb_dialog_priv->exit_on_complete)
     {
+        /* If music brainz dialog has to be cleared on error */
         et_music_brainz_dialog_clear (&mbDialog);
     }
 }
@@ -2516,10 +2560,13 @@ et_musicbrainz_dialog_init (EtMusicBrainzDialog *dialog)
     gtk_box_pack_start (GTK_BOX (gtk_dialog_get_content_area (GTK_DIALOG (dialog))),
                         box, TRUE, TRUE, 2);
     gtk_widget_set_size_request (GTK_WIDGET (dialog), 700, 500);
+    /* Pack EtMbEntityView into central box */
     gtk_box_pack_start (GTK_BOX (gtk_builder_get_object (builder, "central_box")),
                         priv->entityView, TRUE, TRUE, 2);
     et_initialize_tag_choice_dialog (priv);
     cb_search = GTK_WIDGET (gtk_builder_get_object (builder, "cb_manual_search"));
+    
+    /* Set callback functions */
     g_signal_connect (gtk_bin_get_child (GTK_BIN (cb_search)), "activate",
                       G_CALLBACK (btn_manual_find_clicked), NULL);
     g_signal_connect (gtk_builder_get_object (builder, "btn_manual_find"),