diff options
author | Alejandro Colomar <alx@kernel.org> | 2024-04-15 11:47:21 +0200 |
---|---|---|
committer | Serge Hallyn <serge@hallyn.com> | 2024-05-04 17:22:57 -0500 |
commit | d7ab811a366b2969f687b88130deb0039604d5ce (patch) | |
tree | a3ad7fbb3f44a63b19327ea42359429994af37a4 | |
parent | 62a4daa2cd4dff4f085061bcf3a1268b2b31a0ce (diff) |
lib/getrange.c: getrange(): Don't else after return
Signed-off-by: Alejandro Colomar <alx@kernel.org>
-rw-r--r-- | lib/getrange.c | 19 |
1 files changed, 9 insertions, 10 deletions
diff --git a/lib/getrange.c b/lib/getrange.c index ac82fcf6..389f9042 100644 --- a/lib/getrange.c +++ b/lib/getrange.c @@ -66,17 +66,16 @@ getrange(const char *range, endptr++; if ('\0' == *endptr) return 0; /* <long>- */ - if (!isdigit (*endptr)) { + if (!isdigit(*endptr)) return -1; - } else { - errno = 0; - *max = strtoul_noneg(endptr, &endptr, 10); - if ('\0' != *endptr || 0 != errno) - return -1; - *has_max = true; - - /* <long>-<long> */ - } + + errno = 0; + *max = strtoul_noneg(endptr, &endptr, 10); + if ('\0' != *endptr || 0 != errno) + return -1; + *has_max = true; + + /* <long>-<long> */ break; default: return -1; |