summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAlejandro Colomar <alx@kernel.org>2024-04-15 11:57:49 +0200
committerSerge Hallyn <serge@hallyn.com>2024-05-04 17:22:57 -0500
commit7af7361fd6b3ba61678d909de10fdc085a2e9f3c (patch)
treed757a7feae73e7b055562031a971bb3f9d7937fd
parentbbb2735cc042e3ef5e4c9febc4965cdecd991f55 (diff)
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 <alx@kernel.org>
-rw-r--r--lib/getrange.c10
1 files changed, 4 insertions, 6 deletions
diff --git a/lib/getrange.c b/lib/getrange.c
index 0410cb81..e5ea1744 100644
--- a/lib/getrange.c
+++ b/lib/getrange.c
@@ -59,10 +59,10 @@ getrange(const char *range,
switch (*endptr) {
case '\0':
- /* <long> */
*has_max = true;
*max = *min;
- break;
+ return 0; /* <long> */
+
case '-':
endptr++;
if ('\0' == *endptr)
@@ -76,11 +76,9 @@ getrange(const char *range,
return -1;
*has_max = true;
- /* <long>-<long> */
- break;
+ return 0; /* <long>-<long> */
+
default:
return -1;
}
-
- return 0;
}