diff options
author | Alejandro Colomar <alx@kernel.org> | 2024-04-15 11:57:49 +0200 |
---|---|---|
committer | Serge Hallyn <serge@hallyn.com> | 2024-05-04 17:22:57 -0500 |
commit | 7af7361fd6b3ba61678d909de10fdc085a2e9f3c (patch) | |
tree | d757a7feae73e7b055562031a971bb3f9d7937fd | |
parent | bbb2735cc042e3ef5e4c9febc4965cdecd991f55 (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.c | 10 |
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; } |