From 7af7361fd6b3ba61678d909de10fdc085a2e9f3c Mon Sep 17 00:00:00 2001 From: Alejandro Colomar Date: Mon, 15 Apr 2024 11:57:49 +0200 Subject: [PATCH] lib/getrange.c: getrange(): Return early It's doesn't make much sense to break from a switch() just to return. Let's return early, to simplify. Signed-off-by: Alejandro Colomar --- lib/getrange.c | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/lib/getrange.c b/lib/getrange.c index 0410cb81f..e5ea17449 100644 --- a/lib/getrange.c +++ b/lib/getrange.c @@ -59,10 +59,10 @@ getrange(const char *range, switch (*endptr) { case '\0': - /* */ *has_max = true; *max = *min; - break; + return 0; /* */ + case '-': endptr++; if ('\0' == *endptr) @@ -76,11 +76,9 @@ getrange(const char *range, return -1; *has_max = true; - /* - */ - break; + return 0; /* - */ + default: return -1; } - - return 0; }