rename: getdigits_safe => try_getdigits

This commit is contained in:
Justin M. Keyes
2019-09-13 17:23:02 -07:00
parent 427cf16e44
commit 0a24a2c314
3 changed files with 10 additions and 10 deletions

View File

@@ -5078,7 +5078,7 @@ chk_modeline(
} else {
e = s + 3;
}
if (getdigits_safe(&e, &vers) != OK) {
if (!try_getdigits(&e, &vers)) {
continue;
}

View File

@@ -1601,18 +1601,18 @@ char_u* skiptowhite_esc(char_u *p) {
/// It will be advanced past the read number.
/// @param[out] nr Number read from the string.
///
/// @return OK on success, FAIL on error/overflow
int getdigits_safe(char_u **pp, intmax_t *nr)
/// @return true on success, false on error/overflow
bool try_getdigits(char_u **pp, intmax_t *nr)
{
errno = 0;
*nr = strtoimax((char *)(*pp), (char **)pp, 10);
if ((*nr == INTMAX_MIN || *nr == INTMAX_MAX)
&& errno == ERANGE) {
return FAIL;
return false;
}
return OK;
return true;
}
/// Get a number from a string and skip over it.
@@ -1624,10 +1624,10 @@ int getdigits_safe(char_u **pp, intmax_t *nr)
intmax_t getdigits(char_u **pp)
{
intmax_t number;
int ret = getdigits_safe(pp, &number);
int ok = try_getdigits(pp, &number);
(void)ret; // Avoid "unused variable" warning in Release build
assert(ret == OK);
(void)ok; // Avoid "unused variable" warning in Release build
assert(ok);
return number;
}

View File

@@ -40,8 +40,8 @@ int socket_watcher_init(Loop *loop, SocketWatcher *watcher,
char *port = host_end + 1;
intmax_t iport;
int ret = getdigits_safe(&(char_u *){ (char_u *)port }, &iport);
if (ret == FAIL || iport < 0 || iport > UINT16_MAX) {
int ok = try_getdigits(&(char_u *){ (char_u *)port }, &iport);
if (!ok || iport < 0 || iport > UINT16_MAX) {
ELOG("Invalid port: %s", port);
return UV_EINVAL;
}