Skip to content

Commit 7fb2297

Browse files
Incorporate trivial utility cleanups. (#310)
Signed-off-by: Samuel K. Gutierrez <[email protected]>
1 parent 608249b commit 7fb2297

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

src/qvi-utils.cc

+7-7
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
/* -*- Mode: C++; c-basic-offset:4; indent-tabs-mode:nil -*- */
22
/*
3-
* Copyright (c) 2020-2024 Triad National Security, LLC
3+
* Copyright (c) 2020-2025 Triad National Security, LLC
44
* All rights reserved.
55
*
66
* Copyright (c) 2020-2021 Lawrence Livermore National Security, LLC
@@ -105,15 +105,15 @@ qvi_atoi(
105105
const long val = strtol(str, &end, 10);
106106
int err = errno;
107107
// Did we get any digits?
108-
if (str == end) return QV_ERR_INVLD_ARG;
108+
if (qvi_unlikely(str == end)) return QV_ERR_INVLD_ARG;
109109
// In valid range?
110-
if (val > INT_MAX || (err == ERANGE && val == LONG_MAX)) {
110+
if (qvi_unlikely(val > INT_MAX || (err == ERANGE && val == LONG_MAX))) {
111111
return QV_ERR_INVLD_ARG;
112112
}
113-
if (val < INT_MIN || (err == ERANGE && val == LONG_MIN)) {
113+
if (qvi_unlikely(val < INT_MIN || (err == ERANGE && val == LONG_MIN))) {
114114
return QV_ERR_INVLD_ARG;
115115
}
116-
if (*end != '\0') return QV_ERR_INVLD_ARG;
116+
if (qvi_unlikely(*end != '\0')) return QV_ERR_INVLD_ARG;
117117

118118
*maybe_val = (int)val;
119119
return QV_SUCCESS;
@@ -132,13 +132,13 @@ int
132132
qvi_url(
133133
std::string &url
134134
) {
135-
static const cstr_t base = "tcp://127.0.0.1";
135+
static const std::string base = "tcp://127.0.0.1";
136136

137137
int port = 0;
138138
const int rc = qvi_port(&port);
139139
if (rc != QV_SUCCESS) return rc;
140140

141-
url = std::string(base) + ":" + std::to_string(port);
141+
url = base + ":" + std::to_string(port);
142142
return rc;
143143
}
144144

0 commit comments

Comments
 (0)