Use g_strsplit instead of homebrewing a solution

Lesson: don't reinvent the wheel.
Additionally, changed a `g_malloc` call to `g_malloc_n` for safety.

src/option_parser.c

@@ -150,7 +150,7 @@ bool string_parse_mouse_action_list(char **s, enum mouse_action **ret)
         while (s[len])
                 len++;
 
-        *ret = g_malloc((len + 1) * sizeof(enum mouse_action));
+        *ret = g_malloc_n((len + 1), sizeof(enum mouse_action));
         for (int i = 0; i < len; i++) {
                 if (!string_parse_mouse_action(s[i], *ret + i)) {
                         LOG_W("Unknown mouse action value: '%s'", s[i]);

src/utils.c

@@ -151,18 +151,10 @@ char **string_to_array(const char *string)
 {
         char **arr = NULL;
         if (string) {
-                char* dup = g_strdup(string);
+                arr = g_strsplit(string, ",", 0);
-                char* tmp = dup;
+                for (int i = 0; arr[i]; i++){
-                int num_tokens = 0;
+                        g_strstrip(arr[i]);
-                char *token = strsep(&tmp, ",");
-                while (token) {
-                        arr = g_realloc_n(arr, num_tokens + 2, sizeof(char*));
-                        arr[num_tokens] = g_strdup(g_strstrip(token));
-                        num_tokens++;
-                        token = strsep(&tmp, ",");
                 }
-                arr[num_tokens] = NULL;
-                g_free(dup);
         }
         return arr;
 }