diff options
author | Brian Ashworth <bosrsf04@gmail.com> | 2018-10-09 08:12:02 -0400 |
---|---|---|
committer | Brian Ashworth <bosrsf04@gmail.com> | 2018-10-09 08:12:46 -0400 |
commit | d3f0e52784712696c7174d3adf8ec6cf3ac31b19 (patch) | |
tree | 313ca9e6eb03e9f5971d3fde90ace73b613ca9e4 /sway/commands/bar | |
parent | Implement bar bindsym (diff) | |
download | sway-d3f0e52784712696c7174d3adf8ec6cf3ac31b19.tar.gz sway-d3f0e52784712696c7174d3adf8ec6cf3ac31b19.tar.zst sway-d3f0e52784712696c7174d3adf8ec6cf3ac31b19.zip |
bar-bindsym: address ianyfan's comments
Diffstat (limited to 'sway/commands/bar')
-rw-r--r-- | sway/commands/bar/bindsym.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/sway/commands/bar/bindsym.c b/sway/commands/bar/bindsym.c index b0df9eff..4eea3e6a 100644 --- a/sway/commands/bar/bindsym.c +++ b/sway/commands/bar/bindsym.c | |||
@@ -33,12 +33,12 @@ struct cmd_results *bar_cmd_bindsym(int argc, char **argv) { | |||
33 | binding->button = 0; | 33 | binding->button = 0; |
34 | if (strncasecmp(argv[0], "button", strlen("button")) == 0 && | 34 | if (strncasecmp(argv[0], "button", strlen("button")) == 0 && |
35 | strlen(argv[0]) == strlen("button0")) { | 35 | strlen(argv[0]) == strlen("button0")) { |
36 | binding->button = argv[0][strlen("button")] - '1' + 1; | 36 | binding->button = argv[0][strlen("button")] - '0'; |
37 | } | 37 | } |
38 | if (binding->button == 0) { | 38 | if (binding->button < 1 || binding->button > 9) { |
39 | free_bar_binding(binding); | 39 | free_bar_binding(binding); |
40 | return cmd_results_new(CMD_FAILURE, "bar bindsym", | 40 | return cmd_results_new(CMD_FAILURE, "bar bindsym", |
41 | "Only button<n> is supported"); | 41 | "Only button<1-9> is supported"); |
42 | } | 42 | } |
43 | 43 | ||
44 | binding->command = join_args(argv + 1, argc - 1); | 44 | binding->command = join_args(argv + 1, argc - 1); |