From 6e799ad40d8f313383dda1ac143eae14c41b9c89 Mon Sep 17 00:00:00 2001 From: Dana Jansens Date: Tue, 11 May 2010 19:01:46 -0400 Subject: add cleanup functions to the client list menus that removes everything from them when they are not being shown (saves iterating thru the list uselessly when a client is unmanaged) --- openbox/client_list_menu.c | 11 +++++++++++ 1 file changed, 11 insertions(+) (limited to 'openbox/client_list_menu.c') diff --git a/openbox/client_list_menu.c b/openbox/client_list_menu.c index b47ead02..f3df2a5a 100644 --- a/openbox/client_list_menu.c +++ b/openbox/client_list_menu.c @@ -121,6 +121,16 @@ static void desk_menu_destroy(ObMenu *menu, gpointer data) desktop_menus = g_slist_remove(desktop_menus, menu); } +static void self_cleanup(ObMenu *menu, gpointer data) +{ + menu_clear_entries(menu); + + while (desktop_menus) { + menu_free(desktop_menus->data); + desktop_menus = g_slist_delete_link(desktop_menus, desktop_menus); + } +} + static gboolean self_update(ObMenuFrame *frame, gpointer data) { ObMenu *menu = frame->menu; @@ -203,6 +213,7 @@ void client_list_menu_startup(gboolean reconfig) menu = menu_new(MENU_NAME, _("Desktops"), TRUE, NULL); menu_set_update_func(menu, self_update); + menu_set_cleanup_func(menu, self_cleanup); menu_set_execute_func(menu, self_execute); } -- cgit v1.2.3