all repos — openbox @ 0e69ae2b96fb4dd7435ebe15645d1384d11a3ef7

openbox fork - make it a bit more like ryudo 

* Change xml parsing to pass the parent node, rather than the first
child.
* Add x,y co-ordinates on showmenu action so we can place menus on keypress.
Scott Moynes smoynes@nexus.carleton.ca
commit

0e69ae2b96fb4dd7435ebe15645d1384d11a3ef7

parent

845833226b7d65019d41eb3afce6fad7ae3d30a2

7 files changed, 47 insertions(+), 13 deletions(-)

jump to
M openbox/config.c → openbox/config.c 
@@ -28,6 +28,8 @@ static void parse_focus(xmlDocPtr doc, xmlNodePtr node, void *d)
 {
     xmlNodePtr n;
 
+    node = node->xmlChildrenNode;
+    
     if ((n = parse_find_node("focusNew", node)))
         config_focus_new = parse_bool(doc, n);
     if ((n = parse_find_node("followMouse", node)))

@@ -44,6 +46,8 @@ static void parse_theme(xmlDocPtr doc, xmlNodePtr node, void *d)
 {
     xmlNodePtr n;
 
+    node = node->xmlChildrenNode;
+
     if ((n = parse_find_node("theme", node))) {
         g_free(config_theme);
         config_theme = parse_string(doc, n);

@@ -54,6 +58,8 @@ static void parse_desktops(xmlDocPtr doc, xmlNodePtr node, void *d)
 {
     xmlNodePtr n;
 
+    node = node->xmlChildrenNode;
+    
     if ((n = parse_find_node("number", node)))
         config_desktops_num = parse_int(doc, n);
     if ((n = parse_find_node("names", node))) {

@@ -78,6 +84,8 @@ static void parse_moveresize(xmlDocPtr doc, xmlNodePtr node, void *d)
 {
     xmlNodePtr n;
 
+    node = node->xmlChildrenNode;
+    
     if ((n = parse_find_node("opaqueMove", node)))
         config_opaque_move = parse_bool(doc, n);
     if ((n = parse_find_node("opaqueResize", node)))

@@ -87,6 +95,8 @@ 
 static void parse_dock(xmlDocPtr doc, xmlNodePtr node, void *d)
 {
     xmlNodePtr n;
+
+    node = node->xmlChildrenNode;
 
     if ((n = parse_find_node("position", node))) {
         if (parse_contains("TopLeft", doc, n))
M openbox/menu.c → openbox/menu.c 
@@ -33,20 +33,30 @@ gchar *id = NULL, *title = NULL, *label = NULL, *plugin;
     ObMenu *menu = NULL, *parent;
 
     if (newmenu == TRUE) {
-        if (!parse_attr_string("id", node->parent, &id))
+        if (!parse_attr_string("id", node, &id))
             goto parse_menu_fail;
-        if (!parse_attr_string("label", node->parent, &title))
+        if (!parse_attr_string("label", node, &title))
             goto parse_menu_fail;
-
         g_message("menu label %s", title);
 
-        menu = menu_new(title, id, data ? *((ObMenu**)data) : NULL);
-
+        if (parse_attr_string("plugin", node, &plugin)) {
+            PluginMenuCreateData data = {
+                .doc = doc,
+                .node = node,
+                .parent = menu
+            };
+            parent = plugin_create(plugin, &data);
+            g_free(plugin);
+        } else
+            menu = menu_new(title, id, data ? *((ObMenu**)data) : NULL);
+            
         if (data)
             *((ObMenu**)data) = menu;
     } else {
         menu = (ObMenu *)data;
     }
+
+    node = node->xmlChildrenNode;
     
     while (node) {
         if (!xmlStrcasecmp(node->name, (const xmlChar*) "menu")) {

@@ -59,7 +69,7 @@ parent = plugin_create(plugin, &data);
                 g_free(plugin);
             } else {
                 parent = menu;
-                parse_menu(doc, node->xmlChildrenNode, &parent);
+                parse_menu(doc, node, &parent);
                 menu_add_entry(menu, menu_entry_new_submenu(parent->label,
                                                             parent));
             }
M parser/parse.c → parser/parse.c 
@@ -100,7 +100,7 @@ while (node) {
         struct Callback *c = g_hash_table_lookup(callbacks, node->name);
 
         if (c)
-            c->func(doc, node->xmlChildrenNode, c->data);
+            c->func(doc, node, c->data);
 
         node = node->next;
     }
M plugins/keyboard/keyboard.c → plugins/keyboard/keyboard.c 
@@ -69,7 +69,7 @@ }
 
 static void parse_xml(xmlDocPtr doc, xmlNodePtr node, void *d)
 {
-    parse_key(doc, node, NULL);
+    parse_key(doc, node->xmlChildrenNode, NULL);
 }
 
 void plugin_setup_config()

@@ -247,6 +247,11 @@ 
                             if (act->func == action_cycle_windows &&
                                 !grabbed_key && grab_keyboard(TRUE)) {
                                 grabbed_key = p;
+                            }
+
+                            if (act->func == action_showmenu) {
+                                act->data.showmenu.x = e->data.x.e->xkey.x_root;
+                                act->data.showmenu.y = e->data.x.e->xkey.y_root;
                             }
 
                             act->data.any.c = focus_client;
M plugins/menu/fifo_menu.c → plugins/menu/fifo_menu.c 
@@ -133,6 +133,11 @@ g_free(FIFO_MENU_DATA(m)->fifo);
         FIFO_MENU_DATA(m)->fifo = NULL;
     }
 
+    if (FIFO_MENU_DATA(m)->buf != NULL) {
+        g_free(FIFO_MENU_DATA(m)->buf);
+        FIFO_MENU_DATA(m)->buf = NULL;
+    }
+ 
     g_free(m->plugin_data);
 
     menu_free(m->name);

@@ -162,9 +167,11 @@ 
     m = menu_new( (label != NULL ? label : ""),
                   (id != NULL ? id : PLUGIN_NAME),
                   data->parent);
-    menu_add_entry(data->parent, menu_entry_new_submenu(
-                       (label != NULL ? label : ""),
-                       m));
+
+    if (data->parent)
+        menu_add_entry(data->parent, menu_entry_new_submenu(
+                           (label != NULL ? label : ""),
+                           m));
 
     g_free(label);
     g_free(id);
M plugins/mouse/mouse.c → plugins/mouse/mouse.c 
@@ -31,6 +31,8 @@ char *contextstr;
     MouseAction mact;
     Action *action;
 
+    node = node->xmlChildrenNode;
+    
     if ((n = parse_find_node("dragThreshold", node)))
         threshold = parse_int(doc, n);
     if ((n = parse_find_node("doubleClickTime", node)))
M plugins/placement/history.c → plugins/placement/history.c 
@@ -181,7 +181,7 @@ 
     xmlIndentTreeOutput = 1;
     xmlSaveFormatFile(history_path, doc, 1);
 
-    xmlFree(doc);
+    xmlFreeDoc(doc);
 }
 
 static void load_history()

@@ -236,7 +236,7 @@ }
         g_free(name); g_free(class); g_free(role);
         node = parse_find_node("entry", node->next);
     }
-    xmlFree(doc);
+    xmlFreeDoc(doc);
 }
 
 void history_startup()