diff options
author | Brian Ashworth <[email protected]> | 2018-10-09 08:12:02 -0400 |
---|---|---|
committer | Brian Ashworth <[email protected]> | 2018-10-09 08:12:46 -0400 |
commit | d3f0e52784712696c7174d3adf8ec6cf3ac31b19 (patch) | |
tree | 313ca9e6eb03e9f5971d3fde90ace73b613ca9e4 /swaybar/config.c | |
parent | 1c969e86f50065985ddf35b7fef62c14aa7688a5 (diff) |
bar-bindsym: address ianyfan's comments
Diffstat (limited to 'swaybar/config.c')
-rw-r--r-- | swaybar/config.c | 16 |
1 files changed, 11 insertions, 5 deletions
diff --git a/swaybar/config.c b/swaybar/config.c index c646fe66..09d40c24 100644 --- a/swaybar/config.c +++ b/swaybar/config.c @@ -72,16 +72,22 @@ struct swaybar_config *init_config(void) { return config; } +static void free_binding(struct swaybar_binding *binding) { + if (!binding) { + return; + } + free(binding->command); + free(binding); +} + void free_config(struct swaybar_config *config) { free(config->status_command); free(config->font); free(config->mode); free(config->sep_symbol); - while (config->bindings->length) { - struct swaybar_binding *binding = config->bindings->items[0]; - list_del(config->bindings, 0); - free(binding->command); - free(binding); + for (int i = 0; i < config->bindings->length; i++) { + struct swaybar_binding *binding = config->bindings->items[i]; + free_binding(binding); } list_free(config->bindings); struct config_output *coutput, *tmp; |