all repos — openbox @ d6985fd99691dd552070563f8d05602f27da89f0

openbox fork - make it a bit more like ryudo

Allow specifying several mousebinds in one go, like for keybinds

Also fixes minor indent error
Mikael Magnusson mikachu@gmail.com
commit

d6985fd99691dd552070563f8d05602f27da89f0

parent

6273bf2e94a777760fee5346c290fb6672a0cb70

1 files changed, 15 insertions(+), 4 deletions(-)

jump to
M openbox/config.copenbox/config.c

@@ -567,6 +567,8 @@ }

nbut = obt_xml_find_node(n->children, "mousebind"); while (nbut) { + gchar **button, **buttons; + if (!obt_xml_attr_string(nbut, "button", &buttonstr)) goto next_nbut; if (obt_xml_attr_contains(nbut, "action", "press"))

@@ -582,17 +584,26 @@ mact = OB_MOUSE_ACTION_MOTION;

else goto next_nbut; + buttons = g_strsplit(buttonstr, " ", 0); nact = obt_xml_find_node(nbut->children, "action"); while (nact) { ObActionsAct *action; - if ((action = actions_parse(nact))) - mouse_bind(buttonstr, cx, mact, action); + /* actions_parse() creates one ref to the action, but we need + * exactly one ref per binding we use it for. */ + if ((action = actions_parse(nact))) { + for (button = buttons; *button; ++button) { + actions_act_ref(action); + mouse_bind(*button, cx, mact, action); + } + actions_act_unref(action); + } nact = obt_xml_find_node(nact->next, "action"); } - g_free(buttonstr); + g_strfreev(buttons); + g_free(buttonstr); next_nbut: - nbut = obt_xml_find_node(nbut->next, "mousebind"); + nbut = obt_xml_find_node(nbut->next, "mousebind"); } } g_free(modcxstr);