Merge pull request #526 from marek22k/fix-523-2

rewolff · web-flow · commit cfe22c90d815 · 2025-02-15T22:53:45.000+01:00
Remove redundant code
diff --git a/ui/utils.c b/ui/utils.c
@@ -80,7 +80,7 @@ int strtoint_or_err(
     if (str != NULL && *str != '\0') {
         errno = 0;
         num = strtol(str, &end, 0);
-        if (errno == 0 && str != end && end != NULL && *end == '\0' && num < INT_MAX) {
+        if (errno == 0 && str != end && end != NULL && *end == '\0') {
             return num;
         }
     }
@@ -99,7 +99,7 @@ unsigned long strtoulong_or_err(
     if (str != NULL && *str != '\0') {
         errno = 0;
         num = strtoul(str, &end, 0);
-        if (errno == 0 && str != end && end != NULL && *end == '\0' && num < UINT32_MAX) {
+        if (errno == 0 && str != end && end != NULL && *end == '\0') {
             return num;
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -80,7 +80,7 @@ int strtoint_or_err(`
`80`	`80`	`if (str != NULL && *str != '\0') {`
`81`	`81`	`errno = 0;`
`82`	`82`	`num = strtol(str, &end, 0);`
`83`		`- if (errno == 0 && str != end && end != NULL && *end == '\0' && num < INT_MAX) {`
	`83`	`+ if (errno == 0 && str != end && end != NULL && *end == '\0') {`
`84`	`84`	`return num;`
`85`	`85`	`}`
`86`	`86`	`}`
`@@ -99,7 +99,7 @@ unsigned long strtoulong_or_err(`
`99`	`99`	`if (str != NULL && *str != '\0') {`
`100`	`100`	`errno = 0;`
`101`	`101`	`num = strtoul(str, &end, 0);`
`102`		`- if (errno == 0 && str != end && end != NULL && *end == '\0' && num < UINT32_MAX) {`
	`102`	`+ if (errno == 0 && str != end && end != NULL && *end == '\0') {`
`103`	`103`	`return num;`
`104`	`104`	`}`
`105`	`105`	`}`