[easytag/wip/application-window: 2/2] Replace strncat() with g_string_append()



commit 85d4f4ae4f1bfcc0c9ba9f325e5e651a15dc43fd
Author: David King <amigadave amigadave com>
Date:   Sat Jan 4 15:34:45 2014 +0000

    Replace strncat() with g_string_append()

 src/cddb_dialog.c |   57 ++++++++++++++++++++++++++++++----------------------
 1 files changed, 33 insertions(+), 24 deletions(-)
---
diff --git a/src/cddb_dialog.c b/src/cddb_dialog.c
index 0bda4c6..0494849 100644
--- a/src/cddb_dialog.c
+++ b/src/cddb_dialog.c
@@ -1149,7 +1149,7 @@ static gchar *
 Cddb_Generate_Request_String_With_Fields_And_Categories_Options (EtCDDBDialog *self)
 {
     EtCDDBDialogPrivate *priv;
-    gchar string[256];
+    GString *string;
     gboolean cddbinallfields, cddbinartistfield, cddbintitlefield, cddbintracknamefield, cddbinotherfield;
     gboolean cddbinallcategories, cddbinbluescategory, cddbinclassicalcategory, cddbincountrycategory,
              cddbinfolkcategory, cddbinjazzcategory, cddbinmisccategory, cddbinnewagecategory,
@@ -1158,7 +1158,7 @@ Cddb_Generate_Request_String_With_Fields_And_Categories_Options (EtCDDBDialog *s
     priv = et_cddb_dialog_get_instance_private (self);
 
     /* Init. */
-    string[0] = 0;
+    string = g_string_sized_new (256);
 
     /* Fields. */
     cddbinallfields = gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (priv->search_all_toggle));
@@ -1167,13 +1167,19 @@ Cddb_Generate_Request_String_With_Fields_And_Categories_Options (EtCDDBDialog *s
     cddbintracknamefield = gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (priv->search_track_toggle));
     cddbinotherfield = gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (priv->search_other_toggle));
 
-    if (cddbinallfields)      strncat(string,"&allfields=YES",14);
-    else                      strncat(string,"&allfields=NO",13);
+    if (cddbinallfields)
+    {
+        g_string_append (string, "&allfields=YES");
+    }
+    else
+    {
+        g_string_append (string, "&allfields=NO");
+    }
 
-    if (cddbinartistfield)    strncat(string,"&fields=artist",14);
-    if (cddbintitlefield)     strncat(string,"&fields=title",13);
-    if (cddbintracknamefield) strncat(string,"&fields=track",13);
-    if (cddbinotherfield)     strncat(string,"&fields=rest",12);
+    if (cddbinartistfield) g_string_append (string, "&fields=artist");
+    if (cddbintitlefield) g_string_append (string, "&fields=title");
+    if (cddbintracknamefield) g_string_append (string, "&fields=track");
+    if (cddbinotherfield) g_string_append (string, "&fields=rest");
 
 
     // Categories (warning : there is one other CDDB catogories not used here ("data"))
@@ -1189,27 +1195,30 @@ Cddb_Generate_Request_String_With_Fields_And_Categories_Options (EtCDDBDialog *s
     cddbinrockcategory       = gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(priv->categories_rock_toggle));
     cddbinsoundtrackcategory = 
gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(priv->categories_soundtrack_toggle));
 
-    strncat(string,"&allcats=NO",11);
+    g_string_append (string, "&allcats=NO");
+
     if (cddbinallcategories)
     {
-        // All categories except "data"
-        strncat(string,"&cats=blues&cats=classical&cats=country&cats=folk&cats=jazz"
-                       "&cats=misc&cats=newage&cats=reggae&cats=rock&cats=soundtrack",119);
-    }else
+        /* All categories except "data". */
+        g_string_append (string,
+                         "&cats=blues&cats=classical&cats=country&cats=folk&cats=jazz"
+                         "&cats=misc&cats=newage&cats=reggae&cats=rock&cats=soundtrack");
+    }
+    else
     {
-        if (cddbinbluescategory)      strncat(string,"&cats=blues",11);
-        if (cddbinclassicalcategory)  strncat(string,"&cats=classical",15);
-        if (cddbincountrycategory)    strncat(string,"&cats=country",13);
-        if (cddbinfolkcategory)       strncat(string,"&cats=folk",10);
-        if (cddbinjazzcategory)       strncat(string,"&cats=jazz",10);
-        if (cddbinmisccategory)       strncat(string,"&cats=misc",10);
-        if (cddbinnewagecategory)     strncat(string,"&cats=newage",12);
-        if (cddbinreggaecategory)     strncat(string,"&cats=reggae",12);
-        if (cddbinrockcategory)       strncat(string,"&cats=rock",10);
-        if (cddbinsoundtrackcategory) strncat(string,"&cats=soundtrack",16);
+        if (cddbinbluescategory) g_string_append (string, "&cats=blues");
+        if (cddbinclassicalcategory) g_string_append (string, "&cats=classical");
+        if (cddbincountrycategory) g_string_append (string, "&cats=country");
+        if (cddbinfolkcategory) g_string_append (string, "&cats=folk");
+        if (cddbinjazzcategory) g_string_append (string, "&cats=jazz");
+        if (cddbinmisccategory) g_string_append (string, "&cats=misc");
+        if (cddbinnewagecategory) g_string_append (string, "&cats=newage");
+        if (cddbinreggaecategory) g_string_append (string, "&cats=reggae");
+        if (cddbinrockcategory) g_string_append (string, "&cats=rock");
+        if (cddbinsoundtrackcategory) g_string_append (string, "&cats=soundtrack");
     }
 
-    return g_strdup(string);
+    return g_string_free (string, FALSE);
 }
 
 


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