diff --git a/awesomerc.lua.in b/awesomerc.lua.in
index c7a3403e2..ab01665e3 100644
--- a/awesomerc.lua.in
+++ b/awesomerc.lua.in
@@ -158,11 +158,11 @@ for s = 1, screen.count() do
-- Create an imagebox widget which will contains an icon indicating which layout we're using.
-- We need one layoutbox per screen.
mylayoutbox[s] = awful.widget.layoutbox(s)
- mylayoutbox[s]:buttons(awful.util.table.join(
+ mylayoutbox[s].buttons = awful.util.table.join(
awful.button({ }, 1, function () awful.layout.inc(layouts, 1) end),
awful.button({ }, 3, function () awful.layout.inc(layouts, -1) end),
awful.button({ }, 4, function () awful.layout.inc(layouts, 1) end),
- awful.button({ }, 5, function () awful.layout.inc(layouts, -1) end)))
+ awful.button({ }, 5, function () awful.layout.inc(layouts, -1) end))
-- Create a taglist widget
mytaglist[s] = awful.widget.taglist(s, awful.widget.taglist.label.all, mytaglist.buttons)
@@ -191,11 +191,10 @@ end
-- }}}
-- {{{ Mouse bindings
-root.buttons(awful.util.table.join(
+root.buttons = awful.util.table.join(
awful.button({ }, 3, function () mymainmenu:toggle() end),
awful.button({ }, 4, awful.tag.viewnext),
- awful.button({ }, 5, awful.tag.viewprev)
-))
+ awful.button({ }, 5, awful.tag.viewprev))
-- }}}
-- {{{ Key bindings
@@ -318,7 +317,7 @@ for i = 1, keynumber do
end
-- Set keys
-root.keys(globalkeys)
+root.keys = globalkeys
-- }}}
-- {{{ Hooks
@@ -369,11 +368,10 @@ awful.hooks.manage.register(function (c, startup)
awful.titlebar.add(c, { modkey = modkey })
end
-- Add mouse bindings
- c:buttons(awful.util.table.join(
+ c.buttons = awful.util.table.join(
awful.button({ }, 1, function (c) client.focus = c; c:raise() end),
awful.button({ modkey }, 1, awful.mouse.client.move),
- awful.button({ modkey }, 3, awful.mouse.client.resize)
- ))
+ awful.button({ modkey }, 3, awful.mouse.client.resize))
-- New client may not receive focus
-- if they're not focusable, so set border anyway.
c.border_width = beautiful.border_width
@@ -404,7 +402,7 @@ awful.hooks.manage.register(function (c, startup)
client.focus = c
-- Set key bindings
- c:keys(clientkeys)
+ c.keys = clientkeys
-- Set the windows at the slave,
-- i.e. put it at the end of others instead of setting it master.
diff --git a/button.c b/button.c
index 888358d49..e06b8895b 100644
--- a/button.c
+++ b/button.c
@@ -63,49 +63,6 @@ luaA_button_new(lua_State *L)
return 1;
}
-/** Set a button array with a Lua table.
- * \param L The Lua VM state.
- * \param oidx The index of the object to store items into.
- * \param idx The index of the Lua table.
- * \param buttons The array button to fill.
- */
-void
-luaA_button_array_set(lua_State *L, int oidx, int idx, button_array_t *buttons)
-{
- luaA_checktable(L, idx);
-
- foreach(button, *buttons)
- luaA_object_unref_item(L, oidx, *button);
-
- button_array_wipe(buttons);
- button_array_init(buttons);
-
- lua_pushnil(L);
- while(lua_next(L, idx))
- if(luaA_toudata(L, -1, "button"))
- button_array_append(buttons, luaA_object_ref_item(L, oidx, -1));
- else
- lua_pop(L, 1);
-}
-
-/** Push an array of button as an Lua table onto the stack.
- * \param L The Lua VM state.
- * \param oidx The index of the object to get items from.
- * \param buttons The button array to push.
- * \return The number of elements pushed on stack.
- */
-int
-luaA_button_array_get(lua_State *L, int oidx, button_array_t *buttons)
-{
- lua_createtable(L, buttons->len, 0);
- for(int i = 0; i < buttons->len; i++)
- {
- luaA_object_push_item(L, oidx, buttons->tab[i]);
- lua_rawseti(L, -2, i + 1);
- }
- return 1;
-}
-
/** Button object.
* \param L The Lua VM state.
* \return The number of elements pushed on stack.
diff --git a/button.h b/button.h
index 03713917e..572b3dd2a 100644
--- a/button.h
+++ b/button.h
@@ -38,11 +38,7 @@ struct button_t
void *release;
};
-ARRAY_FUNCS(button_t *, button, DO_NOTHING)
LUA_OBJECT_FUNCS(button_t, button, "button")
-int luaA_button_array_get(lua_State *, int, button_array_t *);
-void luaA_button_array_set(lua_State *, int, int, button_array_t *);
-
#endif
// vim: filetype=c:expandtab:shiftwidth=4:tabstop=8:softtabstop=4:encoding=utf-8:textwidth=80
diff --git a/client.c b/client.c
index 1e0b774f7..f78666832 100644
--- a/client.c
+++ b/client.c
@@ -51,8 +51,6 @@ static int
luaA_client_gc(lua_State *L)
{
client_t *c = luaL_checkudata(L, 1, "client");
- button_array_wipe(&c->buttons);
- key_array_wipe(&c->keys);
xcb_get_wm_protocols_reply_wipe(&c->protocols);
p_delete(&c->class);
p_delete(&c->startup_id);
@@ -1552,6 +1550,14 @@ luaA_client_newindex(lua_State *L)
c->skiptb = luaA_checkboolean(L, 3);
hook_property(client, c, "skip_taskbar");
break;
+ case A_TK_KEYS:
+ luaA_object_unref_item(L, 1, c->keys);
+ c->keys = luaA_object_ref_item(L, 1, 3);
+ break;
+ case A_TK_BUTTONS:
+ luaA_object_unref_item(L, 1, c->buttons);
+ c->buttons = luaA_object_ref_item(L, 1, 3);
+ break;
default:
return 0;
}
@@ -1893,6 +1899,10 @@ luaA_client_index(lua_State *L)
}
}
break;
+ case A_TK_KEYS:
+ return luaA_object_push_item(L, 1, c->keys);
+ case A_TK_BUTTONS:
+ return luaA_object_push_item(L, 1, c->buttons);
default:
return 0;
}
@@ -1900,46 +1910,6 @@ luaA_client_index(lua_State *L)
return 1;
}
-/** Get or set mouse buttons bindings for a client.
- * \param L The Lua VM state.
- * \return The number of element pushed on stack.
- * \luastack
- * \lvalue A client.
- * \lparam An array of mouse button bindings objects, or nothing.
- * \return The array of mouse button bindings objects of this client.
- */
-static int
-luaA_client_buttons(lua_State *L)
-{
- client_t *client = luaA_client_checkudata(L, 1);
- button_array_t *buttons = &client->buttons;
-
- if(lua_gettop(L) == 2)
- luaA_button_array_set(L, 1, 2, buttons);
-
- return luaA_button_array_get(L, 1, buttons);
-}
-
-/** Get or set keys bindings for a client.
- * \param L The Lua VM state.
- * \return The number of element pushed on stack.
- * \luastack
- * \lvalue A client.
- * \lparam An array of key bindings objects, or nothing.
- * \return The array of key bindings objects of this client.
- */
-static int
-luaA_client_keys(lua_State *L)
-{
- client_t *c = luaA_client_checkudata(L, 1);
- key_array_t *keys = &c->keys;
-
- if(lua_gettop(L) == 2)
- luaA_key_array_set(L, 1, 2, keys);
-
- return luaA_key_array_get(L, 1, keys);
-}
-
/* Client module.
* \param L The Lua VM state.
* \return The number of pushed elements.
@@ -1996,8 +1966,6 @@ const struct luaL_reg awesome_client_meta[] =
{ "isvisible", luaA_client_isvisible },
{ "geometry", luaA_client_geometry },
{ "struts", luaA_client_struts },
- { "buttons", luaA_client_buttons },
- { "keys", luaA_client_keys },
{ "tags", luaA_client_tags },
{ "kill", luaA_client_kill },
{ "swap", luaA_client_swap },
diff --git a/client.h b/client.h
index 7d25b1b13..a7cb5cf3a 100644
--- a/client.h
+++ b/client.h
@@ -139,9 +139,9 @@ struct client_t
/** Titlebar */
wibox_t *titlebar;
/** Button bindings */
- button_array_t buttons;
+ void *buttons;
/** Key bindings */
- key_array_t keys;
+ void *keys;
/** Icon */
image_t *icon;
/** Size hints */
diff --git a/common/tokenize.gperf b/common/tokenize.gperf
index afe36afb8..9bee7c5c0 100644
--- a/common/tokenize.gperf
+++ b/common/tokenize.gperf
@@ -14,6 +14,7 @@ border_padding
border_width
bottom
button
+buttons
button_press
button_release
center
@@ -49,6 +50,7 @@ image
imagebox
instance
key
+keys
key_press
key_release
keysym
diff --git a/event.c b/event.c
index 5a6545a88..521c5d4ad 100644
--- a/event.c
+++ b/event.c
@@ -42,25 +42,34 @@
#include "common/atoms.h"
#include "common/xutil.h"
-#define DO_EVENT_HOOK_CALLBACK(type, prefix, xcbtype, xcbeventprefix, arraytype, match) \
+#define DO_EVENT_HOOK_CALLBACK(type, prefix, xcbtype, xcbeventprefix, match) \
static int \
event_##xcbtype##_callback(xcb_##xcbtype##_press_event_t *ev, \
- arraytype *arr, \
+ void *arr, \
int oud, \
int nargs, \
void *data) \
{ \
int abs_oud = oud < 0 ? ((lua_gettop(globalconf.L) + 1) + oud) : oud; \
int item_matching = 0, item_matched = 0; \
- foreach(item, *arr) \
- if(match(ev, *item, data)) \
+ if(oud) \
+ luaA_object_push_item(globalconf.L, abs_oud, arr); \
+ else \
+ luaA_object_push(globalconf.L, arr); \
+ lua_pushnil(globalconf.L); \
+ while(luaA_next(globalconf.L, -2)) \
+ { \
+ type *item = luaA_toudata(globalconf.L, -1, #prefix); \
+ if(item && match(ev, item, data)) \
{ \
- if(oud) \
- luaA_object_push_item(globalconf.L, abs_oud, *item); \
- else \
- prefix##_push(globalconf.L, *item); \
+ lua_insert(globalconf.L, -3); \
item_matching++; \
} \
+ else \
+ lua_pop(globalconf.L, 1); \
+ } \
+ /* remove the table */ \
+ lua_pop(globalconf.L, 1); \
for(item_matched = item_matching; item_matching > 0; item_matching--) \
{ \
type *item = luaL_checkudata(globalconf.L, -1, #prefix); \
@@ -71,14 +80,7 @@
{ \
for(int i = 0; i < nargs; i++) \
lua_pushvalue(globalconf.L, - nargs - item_matching); \
- if(oud) \
- { \
- luaA_object_push_item(globalconf.L, abs_oud, item); \
- luaA_object_push_item(globalconf.L, -1, item->press); \
- lua_remove(globalconf.L, -2); \
- } \
- else \
- prefix##_push_item(globalconf.L, item, item->press); \
+ luaA_object_push_item(globalconf.L, - nargs - 1, item->press); \
luaA_dofunction(globalconf.L, nargs, 0); \
} \
break; \
@@ -87,14 +89,7 @@
{ \
for(int i = 0; i < nargs; i++) \
lua_pushvalue(globalconf.L, - nargs - item_matching); \
- if(oud) \
- { \
- luaA_object_push_item(globalconf.L, abs_oud, item); \
- luaA_object_push_item(globalconf.L, -1, item->release); \
- lua_remove(globalconf.L, -2); \
- } \
- else \
- prefix##_push_item(globalconf.L, item, item->release); \
+ luaA_object_push_item(globalconf.L, - nargs - 1, item->release); \
luaA_dofunction(globalconf.L, nargs, 0); \
} \
break; \
@@ -122,8 +117,8 @@ event_key_match(xcb_key_press_event_t *ev, keyb_t *k, void *data)
&& (k->mod == XCB_BUTTON_MASK_ANY || k->mod == ev->state));
}
-DO_EVENT_HOOK_CALLBACK(button_t, button, button, XCB_BUTTON, button_array_t, event_button_match)
-DO_EVENT_HOOK_CALLBACK(keyb_t, key, key, XCB_KEY, key_array_t, event_key_match)
+DO_EVENT_HOOK_CALLBACK(button_t, button, button, XCB_BUTTON, event_button_match)
+DO_EVENT_HOOK_CALLBACK(keyb_t, key, key, XCB_KEY, event_key_match)
/** Handle an event with mouse grabber if needed
* \param x The x coordinate.
@@ -185,7 +180,7 @@ event_handle_button(void *data, xcb_connection_t *connection, xcb_button_press_e
}
wibox_push(globalconf.L, wibox);
- event_button_callback(ev, &wibox->buttons, -1, 1, NULL);
+ event_button_callback(ev, wibox->buttons, -1, 1, NULL);
/* then try to match a widget binding */
widget_t *w = widget_getbycoords(wibox->sw.orientation, &wibox->widgets,
@@ -196,13 +191,13 @@ event_handle_button(void *data, xcb_connection_t *connection, xcb_button_press_e
{
widget_push(globalconf.L, w);
wibox_push(globalconf.L, wibox);
- event_button_callback(ev, &w->buttons, -2, 2, NULL);
+ event_button_callback(ev, w->buttons, -2, 2, NULL);
}
}
else if((c = client_getbywin(ev->event)))
{
client_push(globalconf.L, c);
- event_button_callback(ev, &c->buttons, -1, 1, NULL);
+ event_button_callback(ev, c->buttons, -1, 1, NULL);
xcb_allow_events(globalconf.connection,
XCB_ALLOW_REPLAY_POINTER,
XCB_CURRENT_TIME);
@@ -211,7 +206,7 @@ event_handle_button(void *data, xcb_connection_t *connection, xcb_button_press_e
for(screen = 0; screen < nb_screen; screen++)
if(xutil_screen_get(connection, screen)->root == ev->event)
{
- event_button_callback(ev, &globalconf.buttons, 0, 0, NULL);
+ event_button_callback(ev, globalconf.buttons, 0, 0, NULL);
return 0;
}
@@ -603,15 +598,15 @@ event_handle_key(void *data __attribute__ ((unused)),
if((c = client_getbywin(ev->event)))
{
client_push(globalconf.L, c);
- if(!event_key_callback(ev, &c->keys, -1, 1, &keysym))
- if(!event_key_callback(ev, &globalconf.keys, 0, 0, &keysym))
+ if(!event_key_callback(ev, c->keys, -1, 1, &keysym))
+ if(!event_key_callback(ev, globalconf.keys, 0, 0, &keysym))
xcb_allow_events(globalconf.connection,
XCB_ALLOW_REPLAY_KEYBOARD,
XCB_CURRENT_TIME);
}
else
- event_key_callback(ev, &globalconf.keys, 0, 0, &keysym);
+ event_key_callback(ev, globalconf.keys, 0, 0, &keysym);
xcb_allow_events(globalconf.connection,
XCB_ALLOW_SYNC_KEYBOARD,
diff --git a/key.c b/key.c
index 9e696c10e..ef2d9ecb7 100644
--- a/key.c
+++ b/key.c
@@ -999,49 +999,6 @@ luaA_key_new(lua_State *L)
return 1;
}
-/** Set a key array with a Lua table.
- * \param L The Lua VM state.
- * \param oidx The index of the object to store items into.
- * \param idx The index of the Lua table.
- * \param keys The array key to fill.
- */
-void
-luaA_key_array_set(lua_State *L, int oidx, int idx, key_array_t *keys)
-{
- luaA_checktable(L, idx);
-
- foreach(key, *keys)
- luaA_object_unref_item(L, oidx, *key);
-
- key_array_wipe(keys);
- key_array_init(keys);
-
- lua_pushnil(L);
- while(lua_next(L, idx))
- if(luaA_toudata(L, -1, "key"))
- key_array_append(keys, luaA_object_ref_item(L, oidx, -1));
- else
- lua_pop(L, 1);
-}
-
-/** Push an array of key as an Lua table onto the stack.
- * \param L The Lua VM state.
- * \param oidx The index of the object to get items from.
- * \param keys The key array to push.
- * \return The number of elements pushed on stack.
- */
-int
-luaA_key_array_get(lua_State *L, int oidx, key_array_t *keys)
-{
- lua_createtable(L, keys->len, 0);
- for(int i = 0; i < keys->len; i++)
- {
- luaA_object_push_item(L, oidx, keys->tab[i]);
- lua_rawseti(L, -2, i + 1);
- }
- return 1;
-}
-
/** Push a modifier set to a Lua table.
* \param L The Lua VM state.
* \param modifiers The modifier.
diff --git a/key.h b/key.h
index b33f316d0..49efcbf10 100644
--- a/key.h
+++ b/key.h
@@ -39,15 +39,11 @@ typedef struct keyb_t
void *release;
} keyb_t;
-DO_ARRAY(keyb_t *, key, DO_NOTHING)
LUA_OBJECT_FUNCS(keyb_t, key, "key")
bool key_press_lookup_string(xcb_keysym_t, char *, ssize_t);
xcb_keysym_t key_getkeysym(xcb_keycode_t, uint16_t);
-void luaA_key_array_set(lua_State *, int, int, key_array_t *);
-int luaA_key_array_get(lua_State *, int, key_array_t *);
-
int luaA_pushmodifiers(lua_State *, uint16_t);
uint16_t luaA_tomodifiers(lua_State *L, int ud);
diff --git a/lib/awful/menu.lua.in b/lib/awful/menu.lua.in
index 9ad8a4a2a..ce8c8f33f 100644
--- a/lib/awful/menu.lua.in
+++ b/lib/awful/menu.lua.in
@@ -200,7 +200,7 @@ local function add_item(data, num, item_info)
label.bg_image = icon
end
- item:buttons(bindings)
+ item.buttons = bindings
function label.mouse_enter() item_enter(data, num, true) end
function item.mouse_enter() item_enter(data, num, true) end
@@ -210,7 +210,7 @@ local function add_item(data, num, item_info)
if type(item_info[2]) == "table" then
submenu = widget({ type = "imagebox" })
submenu.image = data.theme.submenu_icon and image(data.theme.submenu_icon)
- submenu:buttons(bindings)
+ submenu.buttons = bindings
function submenu.mouse_enter() item_enter(data, num, true) end
end
diff --git a/lib/awful/titlebar.lua.in b/lib/awful/titlebar.lua.in
index 662766ed0..eb71a197c 100644
--- a/lib/awful/titlebar.lua.in
+++ b/lib/awful/titlebar.lua.in
@@ -80,11 +80,10 @@ function add(c, args)
end
-- Redirect relevant events to the client the titlebar belongs to
- local bts = util.table.join(
+ title.buttons = util.table.join(
button({ }, 1, button_callback_focus_raise_move),
button({ args.modkey }, 1, button_callback_move),
button({ args.modkey }, 3, button_callback_resize))
- title:buttons(bts)
local appicon = capi.widget({ type = "imagebox" })
appicon.image = c.icon
@@ -173,8 +172,6 @@ end
-- @param theme The theme from beautifull. Used to get the image paths
-- @param state The state the button is associated to. Containse path the action and info about the image
local function button_new(c, name, modkey, theme, state)
- local bts = button({ }, 1, nil, state.action)
-
-- get the image path from the theme. Only return a button if we find an image
local img
img = "titlebar_" .. name .. "_button_" .. state.img
@@ -187,13 +184,9 @@ local function button_new(c, name, modkey, theme, state)
local bname = name .. "_" .. state.idx
local button = widget.button({ image = img })
if not button then return end
- local rbts = button:buttons()
- for k, v in pairs(rbts) do
- bts[#bts + 1] = v
- end
+ button.buttons = util.table.join(button.buttons, button({ }, 1, nil, state.action))
- button:buttons(bts)
button.visible = false
return button
end
diff --git a/lib/awful/widget/button.lua.in b/lib/awful/widget/button.lua.in
index 4dfa60c5c..bce6153da 100644
--- a/lib/awful/widget/button.lua.in
+++ b/lib/awful/widget/button.lua.in
@@ -32,7 +32,7 @@ function new(args)
args.type = "imagebox"
local w = capi.widget(args)
w.image = img_release
- w:buttons(button({}, 1, function () w.image = img_press end, function () w.image = img_release end))
+ w.buttons = button({}, 1, function () w.image = img_press end, function () w.image = img_release end)
function w.mouse_leave(s) w.image = img_release end
function w.mouse_enter(s) if capi.mouse.coords().buttons[1] then w.image = img_press end end
return w
diff --git a/lib/awful/widget/common.lua.in b/lib/awful/widget/common.lua.in
index a301b086d..d712d14b6 100644
--- a/lib/awful/widget/common.lua.in
+++ b/lib/awful/widget/common.lua.in
@@ -65,8 +65,8 @@ function list_update(w, buttons, label, data, widgets, objects)
data[o][kb] = capi.button(b.modifiers, b.button, press, release)
end
end
- w[k]:buttons(data[o])
- w[k + 1]:buttons(data[o])
+ w[k].buttons = data[o]
+ w[k + 1].buttons = data[o]
end
local text, bg, bg_image, icon = label(o)
diff --git a/lib/awful/widget/launcher.lua.in b/lib/awful/widget/launcher.lua.in
index 80a5715aa..1013bf693 100644
--- a/lib/awful/widget/launcher.lua.in
+++ b/lib/awful/widget/launcher.lua.in
@@ -21,12 +21,11 @@ function new(args)
if not w then return end
if args.command then
- b = util.table.join(w:buttons(), button({}, 1, nil, function () util.spawn(args.command) end))
+ w.buttons = util.table.join(w.buttons, button({}, 1, nil, function () util.spawn(args.command) end))
elseif args.menu then
- b = util.table.join(w:buttons(), button({}, 1, nil, function () args.menu:toggle() end))
+ w.buttons = util.table.join(w.buttons, button({}, 1, nil, function () args.menu:toggle() end))
end
- w:buttons(b)
return w
end
diff --git a/lib/naughty.lua.in b/lib/naughty.lua.in
index ce17024a1..c13cca7e9 100644
--- a/lib/naughty.lua.in
+++ b/lib/naughty.lua.in
@@ -312,7 +312,7 @@ function notify(args)
-- create textbox
local textbox = capi.widget({ type = "textbox", align = "flex" })
- textbox:buttons(util.table.join(button({ }, 1, run), button({ }, 3, die)))
+ textbox.buttons = util.table.join(button({ }, 1, run), button({ }, 3, die))
textbox:margin({ right = margin, left = margin, bottom = 2 * margin })
textbox.text = string.format('%s%s', font, title, text)
if hover_timeout then textbox.mouse_enter = hover_destroy end
@@ -328,7 +328,7 @@ function notify(args)
-- if we have an icon, use it
if icon then
iconbox = capi.widget({ type = "imagebox", align = "left" })
- iconbox:buttons(util.table.join(button({ }, 1, run), button({ }, 3, die)))
+ iconbox.buttons = util.table.join(button({ }, 1, run), button({ }, 3, die))
local img
if type(icon) == "string" then
img = capi.image(icon)
diff --git a/luaa.c b/luaa.c
index 910f5c11e..159cb197f 100644
--- a/luaa.c
+++ b/luaa.c
@@ -49,7 +49,8 @@ extern const struct luaL_reg awesome_dbus_lib[];
extern const struct luaL_reg awesome_hooks_lib[];
extern const struct luaL_reg awesome_keygrabber_lib[];
extern const struct luaL_reg awesome_mousegrabber_lib[];
-extern const struct luaL_reg awesome_root_lib[];
+extern const struct luaL_reg awesome_root_methods[];
+extern const struct luaL_reg awesome_root_meta[];
extern const struct luaL_reg awesome_button_methods[];
extern const struct luaL_reg awesome_button_meta[];
extern const struct luaL_reg awesome_image_methods[];
@@ -682,9 +683,8 @@ luaA_init(xdgHandle* xdg)
/* Export awesome lib */
luaA_openlib(L, "awesome", awesome_lib, awesome_lib);
- /* Export root lib */
- luaL_register(L, "root", awesome_root_lib);
- lua_pop(L, 1); /* luaL_register() leaves the table on stack */
+ /* Export root */
+ luaA_openlib(L, "root", awesome_root_methods, awesome_root_meta);
/* Export hooks lib */
luaL_register(L, "hooks", awesome_hooks_lib);
diff --git a/root.c b/root.c
index e3c79f38e..af2ebb707 100644
--- a/root.c
+++ b/root.c
@@ -22,7 +22,6 @@
#include
#include "structs.h"
-#include "button.h"
#include "wibox.h"
#include "common/xcursor.h"
#include "common/tokenize.h"
@@ -101,82 +100,6 @@ luaA_root_fake_input(lua_State *L)
return 0;
}
-/** Get or set global key bindings.
- * This binding will be available when you'll press keys on root window.
- * \param L The Lua VM state.
- * \return The number of element pushed on stack.
- * \luastack
- * \lparam An array of key bindings objects, or nothing.
- * \lreturn The array of key bindings objects of this client.
- */
-static int
-luaA_root_keys(lua_State *L)
-{
- if(lua_gettop(L) == 1)
- {
- luaA_checktable(L, 1);
-
- foreach(key, globalconf.keys)
- key_unref(globalconf.L, *key);
-
- key_array_wipe(&globalconf.keys);
- key_array_init(&globalconf.keys);
-
- lua_pushnil(L);
- while(lua_next(L, 1))
- key_array_append(&globalconf.keys, key_ref(L, -1));
-
- return 1;
- }
-
- lua_createtable(L, globalconf.keys.len, 0);
- for(int i = 0; i < globalconf.keys.len; i++)
- {
- key_push(L, globalconf.keys.tab[i]);
- lua_rawseti(L, -2, i + 1);
- }
-
- return 1;
-}
-
-/** Get or set global mouse bindings.
- * This binding will be available when you'll click on root window.
- * \param L The Lua VM state.
- * \return The number of element pushed on stack.
- * \luastack
- * \lparam An array of mouse button bindings objects, or nothing.
- * \lreturn The array of mouse button bindings objects.
- */
-static int
-luaA_root_buttons(lua_State *L)
-{
- if(lua_gettop(L) == 1)
- {
- luaA_checktable(L, 1);
-
- foreach(button, globalconf.buttons)
- button_unref(globalconf.L, *button);
-
- button_array_wipe(&globalconf.buttons);
- button_array_init(&globalconf.buttons);
-
- lua_pushnil(L);
- while(lua_next(L, 1))
- button_array_append(&globalconf.buttons, button_ref(L, -1));
-
- return 1;
- }
-
- lua_createtable(L, globalconf.buttons.len, 0);
- for(int i = 0; i < globalconf.buttons.len; i++)
- {
- button_push(L, globalconf.buttons.tab[i]);
- lua_rawseti(L, -2, i + 1);
- }
-
- return 1;
-}
-
/** Set the root cursor.
* \param L The Lua VM state.
* \return The number of element pushed on stack.
@@ -227,13 +150,58 @@ luaA_root_wiboxes(lua_State *L)
return 1;
}
-const struct luaL_reg awesome_root_lib[] =
+static int
+luaA_root_index(lua_State *L)
+{
+ size_t len;
+ const char *attr = luaL_checklstring(L, 2, &len);
+
+ switch(a_tokenize(attr, len))
+ {
+ case A_TK_KEYS:
+ return luaA_object_push(L, globalconf.keys);
+ case A_TK_BUTTONS:
+ return luaA_object_push(L, globalconf.buttons);
+ default:
+ return 0;
+ }
+}
+
+static int
+luaA_root_newindex(lua_State *L)
+{
+ size_t len;
+ const char *attr = luaL_checklstring(L, 2, &len);
+
+ switch(a_tokenize(attr, len))
+ {
+ case A_TK_KEYS:
+ luaA_object_unref(L, globalconf.keys);
+ globalconf.keys = luaA_object_ref(L, 3);
+ break;
+ case A_TK_BUTTONS:
+ luaA_object_unref(L, globalconf.buttons);
+ globalconf.buttons = luaA_object_ref(L, 3);
+ break;
+ default:
+ break;
+ }
+
+ return 0;
+}
+
+const struct luaL_reg awesome_root_methods[] =
{
- { "buttons", luaA_root_buttons },
- { "keys", luaA_root_keys },
{ "cursor", luaA_root_cursor },
{ "fake_input", luaA_root_fake_input },
{ "wiboxes", luaA_root_wiboxes },
+ { "__index", luaA_root_index },
+ { "__newindex", luaA_root_newindex },
+ { NULL, NULL }
+};
+
+const struct luaL_reg awesome_root_meta[] =
+{
{ NULL, NULL }
};
diff --git a/structs.h b/structs.h
index 4af2a50c5..80d122e21 100644
--- a/structs.h
+++ b/structs.h
@@ -68,9 +68,9 @@ struct awesome_t
/** True if xinerama is active */
bool xinerama_is_active;
/** Root window key bindings */
- key_array_t keys;
+ void *keys;
/** Root window mouse bindings */
- button_array_t buttons;
+ void *buttons;
/** Modifiers masks */
uint16_t numlockmask, shiftlockmask, capslockmask, modeswitchmask;
/** Check for XTest extension */
diff --git a/wibox.c b/wibox.c
index ad4f20c7f..a70c5fe0e 100644
--- a/wibox.c
+++ b/wibox.c
@@ -40,7 +40,6 @@ luaA_wibox_gc(lua_State *L)
wibox_t *wibox = luaL_checkudata(L, 1, "wibox");
p_delete(&wibox->cursor);
simplewindow_wipe(&wibox->sw);
- button_array_wipe(&wibox->buttons);
widget_node_array_wipe(&wibox->widgets);
return luaA_object_gc(L);
}
@@ -648,6 +647,8 @@ luaA_wibox_index(lua_State *L)
return luaA_object_push_item(L, 1, wibox->sw.shape.bounding);
case A_TK_SHAPE_CLIP:
return luaA_object_push_item(L, 1, wibox->sw.shape.clip);
+ case A_TK_BUTTONS:
+ return luaA_object_push_item(L, 1, wibox->buttons);
default:
return 0;
}
@@ -867,6 +868,9 @@ luaA_wibox_newindex(lua_State *L)
luaA_object_unref_item(L, 1, wibox->sw.shape.clip);
wibox->sw.shape.clip = luaA_object_ref_item(L, 1, 3);
wibox->need_shape_update = true;
+ case A_TK_BUTTONS:
+ luaA_object_unref_item(L, 1, wibox->buttons);
+ wibox->buttons = luaA_object_ref_item(L, 1, 3);
break;
default:
switch(wibox->type)
@@ -881,27 +885,6 @@ luaA_wibox_newindex(lua_State *L)
return 0;
}
-/** Get or set mouse buttons bindings to a wibox.
- * \param L The Lua VM state.
- * \luastack
- * \lvalue A wibox.
- * \lparam An array of mouse button bindings objects, or nothing.
- * \return The array of mouse button bindings objects of this wibox.
- */
-static int
-luaA_wibox_buttons(lua_State *L)
-{
- wibox_t *wibox = luaL_checkudata(L, 1, "wibox");
-
- if(lua_gettop(L) == 2)
- {
- luaA_button_array_set(L, 1, 2, &wibox->buttons);
- return 1;
- }
-
- return luaA_button_array_get(L, 1, &wibox->buttons);
-}
-
const struct luaL_reg awesome_wibox_methods[] =
{
{ "__call", luaA_wibox_new },
@@ -909,7 +892,6 @@ const struct luaL_reg awesome_wibox_methods[] =
};
const struct luaL_reg awesome_wibox_meta[] =
{
- { "buttons", luaA_wibox_buttons },
{ "geometry", luaA_wibox_geometry },
{ "__index", luaA_wibox_index },
{ "__newindex", luaA_wibox_newindex },
diff --git a/wibox.h b/wibox.h
index 27b842dd6..1351e5e71 100644
--- a/wibox.h
+++ b/wibox.h
@@ -68,7 +68,7 @@ struct wibox_t
/* Banned? used for titlebars */
bool isbanned;
/** Button bindings */
- button_array_t buttons;
+ void *buttons;
};
void wibox_unref_simplified(wibox_t **);
diff --git a/widget.c b/widget.c
index 3e74b7880..b95de8d99 100644
--- a/widget.c
+++ b/widget.c
@@ -44,7 +44,6 @@ luaA_widget_gc(lua_State *L)
widget_t *widget = luaL_checkudata(L, 1, "widget");
if(widget->destructor)
widget->destructor(widget);
- button_array_wipe(&widget->buttons);
return luaA_object_gc(L);
}
@@ -451,28 +450,6 @@ luaA_widget_new(lua_State *L)
return 1;
}
-/** Get or set mouse buttons bindings to a widget.
- * \param L The Lua VM state.
- *
- * \luastack
- * \lvalue A widget.
- * \lparam An array of mouse button bindings objects, or nothing.
- * \return The array of mouse button bindings objects of this widget.
- */
-static int
-luaA_widget_buttons(lua_State *L)
-{
- widget_t *widget = luaL_checkudata(L, 1, "widget");
-
- if(lua_gettop(L) == 2)
- {
- luaA_button_array_set(L, 1, 2, &widget->buttons);
- return 1;
- }
-
- return luaA_button_array_get(L, 1, &widget->buttons);
-}
-
/** Generic widget.
* \param L The Lua VM state.
* \return The number of elements pushed on stack.
@@ -501,6 +478,8 @@ luaA_widget_index(lua_State *L)
return luaA_object_push_item(L, 1, widget->mouse_enter);
case A_TK_MOUSE_LEAVE:
return luaA_object_push_item(L, 1, widget->mouse_leave);
+ case A_TK_BUTTONS:
+ return luaA_object_push_item(L, 1, widget->buttons);
default:
break;
}
@@ -535,6 +514,10 @@ luaA_widget_newindex(lua_State *L)
luaA_object_unref_item(L, 1, widget->mouse_leave);
widget->mouse_leave = luaA_object_ref_item(L, 1, 3);
return 0;
+ case A_TK_BUTTONS:
+ luaA_object_unref_item(L, 1, widget->buttons);
+ widget->buttons = luaA_object_ref_item(L, 1, 3);
+ break;
default:
return widget->newindex ? widget->newindex(L, token) : 0;
}
@@ -574,7 +557,6 @@ const struct luaL_reg awesome_widget_methods[] =
};
const struct luaL_reg awesome_widget_meta[] =
{
- { "buttons", luaA_widget_buttons },
{ "extents", luaA_widget_extents },
{ "__index", luaA_widget_index },
{ "__newindex", luaA_widget_newindex },
diff --git a/widget.h b/widget.h
index c7a68a729..aa8f5d299 100644
--- a/widget.h
+++ b/widget.h
@@ -51,7 +51,7 @@ struct widget_t
/** Misc private data */
void *data;
/** Button bindings */
- button_array_t buttons;
+ void *buttons;
/** True if the widget is visible */
bool isvisible;
};