mirrors/labwc
1
0
Fork 0
mirror of https://github.com/labwc/labwc.git synced 2025-11-09 13:30:01 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

Centralize freeing of action lists

Browse source 
Reduces some code duplication and makes it easier to extend struct action
This commit is contained in:
Consolatis 2022-01-23 13:46:46 +01:00 • committed by ARDiDo
parent 8e9643a855
commit f0a3f365e5
4 changed files with 14 additions and 17 deletions
Download patch file Download diff file Expand all files Collapse all files

10
src/action.c
View file

@ -2,6 +2,7 @@
#include <strings.h>
#include <wlr/util/log.h>
#include "common/spawn.h"
#include "common/zfree.h"
#include "labwc.h"
#include "menu/menu.h"
#include "action.h"
@ -76,6 +77,15 @@ action_create(const char *action_name)
return action;
}
void action_list_free(struct wl_list *action_list) {
struct action *action, *action_tmp;
wl_list_for_each_safe(action, action_tmp, action_list, link) {
wl_list_remove(&action->link);
zfree(action->arg);
zfree(action);
}
}
static void
show_menu(struct server *server, const char *menu)
{

13
src/config/rcxml.c
View file

@ -676,7 +676,6 @@ no_config:
void
rcxml_finish(void)
{
struct action *action, *action_tmp;
zfree(rc.font_name_activewindow);
zfree(rc.font_name_menuitem);
@ -686,11 +685,7 @@ rcxml_finish(void)
struct keybind *k, *k_tmp;
wl_list_for_each_safe(k, k_tmp, &rc.keybinds, link) {
wl_list_remove(&k->link);
wl_list_for_each_safe(action, action_tmp, &k->actions, link) {
wl_list_remove(&action->link);
zfree(action->arg);
zfree(action);
}
action_list_free(&k->actions);
zfree(k->keysyms);
zfree(k);
}
@ -698,11 +693,7 @@ rcxml_finish(void)
struct mousebind *m, *m_tmp;
wl_list_for_each_safe(m, m_tmp, &rc.mousebinds, link) {
wl_list_remove(&m->link);
wl_list_for_each_safe(action, action_tmp, &m->actions, link) {
wl_list_remove(&action->link);
zfree(action->arg);
zfree(action);
}
action_list_free(&m->actions);
zfree(m);
}

7
src/menu/menu.c
View file

@ -330,17 +330,12 @@ void
menu_finish(void)
{
struct menu *menu;
struct action *action, *action_tmp;
for (int i = 0; i < nr_menus; ++i) {
menu = menus + i;
struct menuitem *item, *next;
wl_list_for_each_safe(item, next, &menu->menuitems, link) {
wl_list_remove(&item->link);
wl_list_for_each_safe(action, action_tmp, &item->actions, link) {
wl_list_remove(&action->link);
zfree(action->arg);
zfree(action);
}
action_list_free(&item->actions);
free(item);
}
}