dont do the same thing 100 times
authorDana Jansens <danakj@orodu.net>
Sat, 21 Apr 2007 22:16:24 +0000 (22:16 +0000)
committerDana Jansens <danakj@orodu.net>
Sat, 21 Apr 2007 22:16:24 +0000 (22:16 +0000)
parser/parse.c

index 552221c..63d53ce 100644 (file)
@@ -87,11 +87,12 @@ gboolean parse_load_rc(const gchar *file, xmlDocPtr *doc, xmlNodePtr *root,
 
     *fileused = NULL;
 
-    for (it = xdg_config_dir_paths; !r && it; it = g_slist_next(it)) {
-        if (file) {
-            if ((r = parse_load(file, "openbox_config", doc, root)))
-                *fileused = g_strdup(file);
-        } else {
+    if (file) {
+        if ((r = parse_load(file, "openbox_config", doc, root)))
+            *fileused = g_strdup(file);
+    } else {
+        /* this won't run if the above code loaded a config */
+        for (it = xdg_config_dir_paths; !r && it; it = g_slist_next(it)) {
             gchar *path;
 
             path = g_build_filename(it->data, "openbox", "rc.xml", NULL);