diff options
| author | Furkan Sahin <furkan-dev@proton.me> | 2025-01-07 13:21:56 +0100 |
|---|---|---|
| committer | Furkan Sahin <furkan-dev@proton.me> | 2025-01-07 13:21:56 +0100 |
| commit | 86d128b708d76a76587d4ce796b567ba4be36e66 (patch) | |
| tree | b5691def14719b8f49af16d35b29494b6ab35667 /swaybar/tray/item.c | |
| parent | 459f8dc51ab25e823390e15cbf1c3718b4239aa3 (diff) | |
Use has_prefix() instead of strncmp() throughout
This is safer than hardcoded string lengths.
Diffstat (limited to 'swaybar/tray/item.c')
| -rw-r--r-- | swaybar/tray/item.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/swaybar/tray/item.c b/swaybar/tray/item.c index ca6c03ad..12929743 100644 --- a/swaybar/tray/item.c +++ b/swaybar/tray/item.c @@ -15,6 +15,7 @@ #include "cairo_util.h" #include "list.h" #include "log.h" +#include "stringop.h" #include "wlr-layer-shell-unstable-v1-client-protocol.h" // TODO menu @@ -161,7 +162,7 @@ static int get_property_callback(sd_bus_message *msg, void *data, } if (strcmp(prop, "Status") == 0 || (sni->status && (sni->status[0] == 'N' ? - prop[0] == 'A' : strncmp(prop, "Icon", 4) == 0))) { + prop[0] == 'A' : has_prefix(prop, "Icon")))) { set_sni_dirty(sni); } cleanup: @@ -364,7 +365,7 @@ static void handle_click(struct swaybar_sni *sni, int x, int y, method = "ContextMenu"; } - if (strncmp(method, "Scroll", strlen("Scroll")) == 0) { + if (has_prefix(method, "Scroll")) { char dir = method[strlen("Scroll")]; char *orientation = (dir == 'U' || dir == 'D') ? "vertical" : "horizontal"; int sign = (dir == 'U' || dir == 'L') ? -1 : 1; |