From d72b4409ce05e5604685fb8468068c8057315b2d Mon Sep 17 00:00:00 2001 From: Simon Ser Date: Tue, 6 Dec 2022 11:43:26 +0100 Subject: [PATCH] util/env: make env_parse_switch() return a size_t This function is guaranteed to never return a negative value. This is important because we use arr[env_parse_switch(...)] in a few places. --- include/util/env.h | 2 +- util/env.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/include/util/env.h b/include/util/env.h index 1edae0dd0..e271f4bfd 100644 --- a/include/util/env.h +++ b/include/util/env.h @@ -18,6 +18,6 @@ bool env_parse_bool(const char *option); * * switches is a NULL-terminated array. */ -ssize_t env_parse_switch(const char *option, const char **switches); +size_t env_parse_switch(const char *option, const char **switches); #endif diff --git a/util/env.c b/util/env.c index b0a9efdae..77911a227 100644 --- a/util/env.c +++ b/util/env.c @@ -19,7 +19,7 @@ bool env_parse_bool(const char *option) { return false; } -ssize_t env_parse_switch(const char *option, const char **switches) { +size_t env_parse_switch(const char *option, const char **switches) { const char *env = getenv(option); if (env) { wlr_log(WLR_INFO, "Loading %s option: %s", option, env);