Revert "ges-launch: no need for a tmp string pointer"
authorMathieu Duponchelle <mathieu.duponchelle@opencreed.com>
Tue, 24 Mar 2015 12:01:39 +0000 (13:01 +0100)
committerMathieu Duponchelle <mathieu.duponchelle@opencreed.com>
Tue, 24 Mar 2015 12:01:39 +0000 (13:01 +0100)
This reverts commit 44a0924c1f6b07f0c91ee8bd03d3ae5d97da92d5.

There indeed is a need for a tmp string pointer.

tools/ges-launch.c

index 8147eb9..d30088a 100644 (file)
@@ -592,6 +592,7 @@ sanitize_argument (gchar * arg)
   char *equal_index = strstr (arg, "=");
   char *space_index = strstr (arg, " ");
   gchar *new_string = g_malloc (sizeof (gchar) * (strlen (arg) + 3));
+  gchar *tmp_string = new_string;
 
   if (!space_index)
     return g_strdup (arg);
@@ -602,16 +603,16 @@ sanitize_argument (gchar * arg)
   }
 
   for (arg = arg; *arg != '\0'; arg++) {
-    *new_string = *arg;
-    new_string += 1;
+    *tmp_string = *arg;
+    tmp_string += 1;
     if (*arg == '=') {
-      *new_string = '"';
-      new_string += 1;
+      *tmp_string = '"';
+      tmp_string += 1;
     }
   }
-  *new_string = '"';
-  new_string += 1;
-  *new_string = '\0';
+  *tmp_string = '"';
+  tmp_string += 1;
+  *tmp_string = '\0';
 
   return new_string;
 }