From c42497aca00a234679ca5dc2884ab40afe0d84ad Mon Sep 17 00:00:00 2001 From: Konstantin Pospelov Date: Tue, 5 Mar 2019 22:38:26 +0300 Subject: [PATCH] bindsym: change xkb_rule_names initialization --- include/sway/config.h | 3 ++- sway/config.c | 5 +++-- sway/config/input.c | 17 +++++++---------- sway/input/keyboard.c | 6 ++---- 4 files changed, 14 insertions(+), 17 deletions(-) diff --git a/include/sway/config.h b/include/sway/config.h index a0a98fb6f..01c99cb4f 100644 --- a/include/sway/config.h +++ b/include/sway/config.h @@ -577,7 +577,8 @@ void merge_input_config(struct input_config *dst, struct input_config *src); struct input_config *store_input_config(struct input_config *ic); -struct xkb_rule_names input_config_get_rule_names(struct input_config *ic); +void input_config_fill_rule_names(struct input_config *ic, + struct xkb_rule_names *rules); void free_input_config(struct input_config *ic); diff --git a/sway/config.c b/sway/config.c index fed01eb43..d4a4a2f7d 100644 --- a/sway/config.c +++ b/sway/config.c @@ -337,7 +337,7 @@ static void config_defaults(struct sway_config *config) { if (!(config->ipc_policies = create_list())) goto cleanup; // The keysym to keycode translation - struct xkb_rule_names rules = {}; + struct xkb_rule_names rules = {0}; config->keysym_translation_state = keysym_translation_state_create(rules); @@ -989,7 +989,8 @@ static void translate_binding_list(list_t *bindings, list_t *bindsyms, void translate_keysyms(struct input_config *input_config) { keysym_translation_state_destroy(config->keysym_translation_state); - struct xkb_rule_names rules = input_config_get_rule_names(input_config); + struct xkb_rule_names rules = {0}; + input_config_fill_rule_names(input_config, &rules); config->keysym_translation_state = keysym_translation_state_create(rules); diff --git a/sway/config/input.c b/sway/config/input.c index 595aa029a..9c533e710 100644 --- a/sway/config/input.c +++ b/sway/config/input.c @@ -212,16 +212,13 @@ struct input_config *store_input_config(struct input_config *ic) { return ic; } -struct xkb_rule_names input_config_get_rule_names(struct input_config *ic) { - struct xkb_rule_names rules = { - .layout = ic->xkb_layout, - .model = ic->xkb_model, - .options = ic->xkb_options, - .rules = ic->xkb_rules, - .variant = ic->xkb_variant, - }; - - return rules; +void input_config_fill_rule_names(struct input_config *ic, + struct xkb_rule_names *rules) { + rules->layout = ic->xkb_layout; + rules->model = ic->xkb_model; + rules->options = ic->xkb_options; + rules->rules = ic->xkb_rules; + rules->variant = ic->xkb_variant; } void free_input_config(struct input_config *ic) { diff --git a/sway/input/keyboard.c b/sway/input/keyboard.c index 04b8b0ce8..396cc865e 100644 --- a/sway/input/keyboard.c +++ b/sway/input/keyboard.c @@ -482,11 +482,9 @@ void sway_keyboard_configure(struct sway_keyboard *keyboard) { struct wlr_input_device *wlr_device = keyboard->seat_device->input_device->wlr_device; - struct xkb_rule_names rules; + struct xkb_rule_names rules = {0}; if (input_config) { - rules = input_config_get_rule_names(input_config); - } else { - memset(&rules, 0, sizeof(rules)); + input_config_fill_rule_names(input_config, &rules); } if (!rules.layout) {