From: Alejandro Colomar <alx@kernel.org>
Date: Mon, 15 Apr 2024 09:57:49 +0000 (+0200)
Subject: lib/getrange.c: getrange(): Return early
X-Git-Tag: 4.15.2~44
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=7af7361fd6b3ba61678d909de10fdc085a2e9f3c;p=thirdparty%2Fshadow.git

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>
---

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':
-		/* <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;
 }