From 852e47a143e31e1a974a4d10017ba32626e7d6f8 Mon Sep 17 00:00:00 2001 From: TapTap Date: Tue, 11 Aug 2026 21:14:56 +0200 Subject: [PATCH] Satisfy cppcheck CLI and logger checks --- src/client/client_cli.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/client/client_cli.c b/src/client/client_cli.c index 641d437..b0ef08b 100644 --- a/src/client/client_cli.c +++ b/src/client/client_cli.c @@ -55,7 +55,7 @@ static bool parse_positive_int(const char* s, int* out_val) { /* Duplicate a string argument into *dest, freeing the old value. Returns true on success, false on * failure. */ -static bool set_string_option(char** dest, const char* value, const char* option_name) { +static int set_string_option(char** dest, const char* value, const char* option_name) { char* dup = str_dup(value); if (!dup) { fprintf(stderr, "Error: memory allocation failed for %s\n", option_name); @@ -67,7 +67,7 @@ static bool set_string_option(char** dest, const char* value, const char* option } /* Parse a string as a positive integer into *dest. Returns true on success, false on error. */ -static bool set_positive_int_option(int* dest, const char* value, const char* option_name) { +static int set_positive_int_option(int* dest, const char* value, const char* option_name) { if (!parse_positive_int(value, dest)) { fprintf(stderr, "Error: %s must be a positive integer\n", option_name); return -1; @@ -76,7 +76,7 @@ static bool set_positive_int_option(int* dest, const char* value, const char* op } /* Parse a string as a non-negative integer into *dest. Returns true on success, false on error. */ -static bool set_nonneg_int_option(int* dest, const char* value, const char* option_name) { +static int set_nonneg_int_option(int* dest, const char* value, const char* option_name) { if (!parse_nonneg_int(value, dest)) { fprintf(stderr, "Error: %s must be a non-negative integer\n", option_name); return -1;