all repos — openbox @ 9ea78a4eb0255e8cca6258e5ceb4bfd3e9e9d59e

openbox fork - make it a bit more like ryudo 

create ObParseInst structuer which can be then used to parse more than one file/memory thing at a time.
add function to parse memory block.
Dana Jansens danakj@orodu.net
commit

9ea78a4eb0255e8cca6258e5ceb4bfd3e9e9d59e

parent

8b67ce7030aa22595d77527cdc5b74b84d185a8b

2 files changed, 58 insertions(+), 22 deletions(-)

jump to
M parser/parse.c → parser/parse.c 
@@ -7,7 +7,9 @@ ParseCallback func;
     void *data;
 };
 
-static GHashTable *callbacks;
+struct _ObParseInst {
+    GHashTable *callbacks;
+};
 
 static void destfunc(struct Callback *c)
 {

@@ -15,22 +17,28 @@ g_free(c->tag);
     g_free(c);
 }
 
-void parse_startup()
+ObParseInst* parse_startup()
 {
-    callbacks = g_hash_table_new_full(g_str_hash, g_str_equal, NULL,
-                                      (GDestroyNotify)destfunc);
+    ObParseInst *i = g_new(ObParseInst, 1);
+    i->callbacks = g_hash_table_new_full(g_str_hash, g_str_equal, NULL,
+                                         (GDestroyNotify)destfunc);
+    return i;
 }
 
-void parse_shutdown()
+void parse_shutdown(ObParseInst *i)
 {
-    g_hash_table_destroy(callbacks);
+    if (i) {
+        g_hash_table_destroy(i->callbacks);
+        g_free(i);
+    }
 }
 
-void parse_register(const char *tag, ParseCallback func, void *data)
+void parse_register(ObParseInst *i, const char *tag,
+                    ParseCallback func, void *data)
 {
     struct Callback *c;
 
-    if ((c = g_hash_table_lookup(callbacks, tag))) {
+    if ((c = g_hash_table_lookup(i->callbacks, tag))) {
         g_warning("tag '%s' already registered", tag);
         return;
     }

@@ -39,7 +47,7 @@ c = g_new(struct Callback, 1);
     c->tag = g_strdup(tag);
     c->func = func;
     c->data = data;
-    g_hash_table_insert(callbacks, c->tag, c);
+    g_hash_table_insert(i->callbacks, c->tag, c);
 }
 
 gboolean parse_load_rc(xmlDocPtr *doc, xmlNodePtr *root)

@@ -66,7 +74,6 @@ 
 gboolean parse_load(const char *path, const char *rootname,
                     xmlDocPtr *doc, xmlNodePtr *root)
 {
-
     xmlLineNumbersDefault(1);
 
     if ((*doc = xmlParseFile(path))) {

@@ -79,8 +86,33 @@ } else {
             if (xmlStrcasecmp((*root)->name, (const xmlChar*)rootname)) {
                 xmlFreeDoc(*doc);
                 *doc = NULL;
-                g_warning("document %s is of wrong type. root *root is "
-                          "not 'openbox_config'", path);
+                g_warning("document %s is of wrong type. root node is "
+                          "not '%s'", path, rootname);
+            }
+        }
+    }
+    if (!*doc)
+        return FALSE;
+    return TRUE;
+}
+
+gboolean parse_load_mem(gpointer data, guint len, const char *rootname,
+                        xmlDocPtr *doc, xmlNodePtr *root)
+{
+    xmlLineNumbersDefault(1);
+
+    if ((*doc = xmlParseMemory(data, len))) {
+        *root = xmlDocGetRootElement(*doc);
+        if (!*root) {
+            xmlFreeDoc(*doc);
+            *doc = NULL;
+            g_warning("Given memory is an empty document");
+        } else {
+            if (xmlStrcasecmp((*root)->name, (const xmlChar*)rootname)) {
+                xmlFreeDoc(*doc);
+                *doc = NULL;
+                g_warning("document in given memory is of wrong type. root "
+                          "node is not '%s'", rootname);
             }
         }
     }

@@ -94,13 +126,13 @@ {
     xmlFree(doc);
 }
 
-void parse_tree(xmlDocPtr doc, xmlNodePtr node, void *nothing)
+void parse_tree(ObParseInst *i, xmlDocPtr doc, xmlNodePtr node)
 {
     while (node) {
-        struct Callback *c = g_hash_table_lookup(callbacks, node->name);
+        struct Callback *c = g_hash_table_lookup(i->callbacks, node->name);
 
         if (c)
-            c->func(doc, node, c->data);
+            c->func(i, doc, node, c->data);
 
         node = node->next;
     }
M parser/parse.h → parser/parse.h 
@@ -4,24 +4,28 @@ 
 #include <libxml/parser.h>
 #include <glib.h>
 
-typedef void (*ParseCallback)(xmlDocPtr doc, xmlNodePtr node, void *data);
+typedef struct _ObParseInst ObParseInst;
+
+typedef void (*ParseCallback)(ObParseInst *i, xmlDocPtr doc, xmlNodePtr node,
+                              gpointer data);
 
-void parse_startup();
-void parse_shutdown();
+ObParseInst* parse_startup();
+void parse_shutdown(ObParseInst *inst);
 
 /* Loads Openbox's rc, from $HOME or $PREFIX as a fallback */
 gboolean parse_load_rc(xmlDocPtr *doc, xmlNodePtr *root);
 
-/* callbacks - must call parse_startup to use these */
-
-void parse_register(const char *tag, ParseCallback func, void *data);
-void parse_tree(xmlDocPtr doc, xmlNodePtr node, void *nothing);
+void parse_register(ObParseInst *inst, const char *tag,
+                    ParseCallback func, gpointer data);
+void parse_tree(ObParseInst *inst, xmlDocPtr doc, xmlNodePtr node);
 
 
 /* open/close */
 
 gboolean parse_load(const char *path, const char *rootname,
                     xmlDocPtr *doc, xmlNodePtr *root);
+gboolean parse_load_mem(gpointer data, guint len, const char *rootname,
+                        xmlDocPtr *doc, xmlNodePtr *root);
 void parse_close(xmlDocPtr doc);