Skip to content

Commit

Permalink
lib/getrange.c: getrange(): Return early
Browse files Browse the repository at this point in the history
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 <[email protected]>
  • Loading branch information
alejandro-colomar authored and hallyn committed May 4, 2024
1 parent bbb2735 commit 7af7361
Showing 1 changed file with 4 additions and 6 deletions.
10 changes: 4 additions & 6 deletions lib/getrange.c
Expand Up @@ -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)
Expand All @@ -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;
}

0 comments on commit 7af7361

Please sign in to comment.