diff options
author | M Stoeckl <code@mstoeckl.com> | 2019-01-22 10:07:38 -0500 |
---|---|---|
committer | M Stoeckl <code@mstoeckl.com> | 2019-01-22 10:12:04 -0500 |
commit | 0af5b26e41c5141d4094652133c230d76bf82e56 (patch) | |
tree | 75c61907f094838e23899231ec5ec955c530f692 /swaybar | |
parent | Merge pull request #3494 from ianyfan/commands (diff) | |
download | sway-0af5b26e41c5141d4094652133c230d76bf82e56.tar.gz sway-0af5b26e41c5141d4094652133c230d76bf82e56.tar.zst sway-0af5b26e41c5141d4094652133c230d76bf82e56.zip |
Fix dead stores found by scan-build
In addition to removing unused code, two minor problems are fixed:
(1) `resize set` and `resize adjust` did not error when given
too many arguments.
(2) `orientation` was incorrectly overridden to be 'U' for
scroll events in the swaybar tray `handle_click` function.
Diffstat (limited to 'swaybar')
-rw-r--r-- | swaybar/tray/item.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/swaybar/tray/item.c b/swaybar/tray/item.c index 39d83b23..a43bbf6f 100644 --- a/swaybar/tray/item.c +++ b/swaybar/tray/item.c | |||
@@ -334,7 +334,7 @@ static void handle_click(struct swaybar_sni *sni, int x, int y, | |||
334 | 334 | ||
335 | if (strncmp(method, "Scroll", strlen("Scroll")) == 0) { | 335 | if (strncmp(method, "Scroll", strlen("Scroll")) == 0) { |
336 | char dir = method[strlen("Scroll")]; | 336 | char dir = method[strlen("Scroll")]; |
337 | char *orientation = (dir = 'U' || dir == 'D') ? "vertical" : "horizontal"; | 337 | char *orientation = (dir == 'U' || dir == 'D') ? "vertical" : "horizontal"; |
338 | int sign = (dir == 'U' || dir == 'L') ? -1 : 1; | 338 | int sign = (dir == 'U' || dir == 'L') ? -1 : 1; |
339 | 339 | ||
340 | sd_bus_call_method_async(sni->tray->bus, NULL, sni->service, sni->path, | 340 | sd_bus_call_method_async(sni->tray->bus, NULL, sni->service, sni->path, |