Commit 04b5d87b authored by Kirill Smelkov's avatar Kirill Smelkov

.

parent d876da33
...@@ -134,7 +134,7 @@ static error _pathError(const char *op, const string &path, int syserr) { ...@@ -134,7 +134,7 @@ static error _pathError(const char *op, const string &path, int syserr) {
char ebuf[128]; char ebuf[128];
char *estr = strerror_r(syserr, ebuf, sizeof(ebuf)); char *estr = strerror_r(syserr, ebuf, sizeof(ebuf));
return fmt::errorf("%s %s: %s", op, path.c_str(), estr); return fmt::errorf("%s %s: %s", op, v(path), estr);
} }
} // os:: } // os::
...@@ -190,12 +190,12 @@ namespace xstrconv { ...@@ -190,12 +190,12 @@ namespace xstrconv {
// parseHex64 decodes 16-character-wide hex-encoded string into uint64. // parseHex64 decodes 16-character-wide hex-encoded string into uint64.
tuple<uint64_t, error> parseHex64(const string& s) { tuple<uint64_t, error> parseHex64(const string& s) {
if (s.size() != 16) if (s.size() != 16)
return make_tuple(0, fmt::errorf("hex64 %s invalid", s.c_str())); return make_tuple(0, fmt::errorf("hex64 %s invalid", v(s)));
uint64_t v; uint64_t v;
int n = sscanf(s.c_str(), "%16" SCNx64, &v); // XXX verify int n = sscanf(s.c_str(), "%16" SCNx64, &v); // XXX verify
if (n != 1) if (n != 1)
return make_tuple(0, fmt::errorf("hex64 %s invalid", s.c_str())); return make_tuple(0, fmt::errorf("hex64 %s invalid", v(s)));
return make_tuple(v, nil); return make_tuple(v, nil);
} }
...@@ -205,7 +205,7 @@ tuple<int64_t, error> parseInt(const string& s) { ...@@ -205,7 +205,7 @@ tuple<int64_t, error> parseInt(const string& s) {
int64_t v; int64_t v;
int n = sscanf(s.c_str(), "%" SCNi64, &v); // XXX verify int n = sscanf(s.c_str(), "%" SCNi64, &v); // XXX verify
if (!(n == 1 && std::to_string(v) == s)) // XXX verify if (!(n == 1 && std::to_string(v) == s)) // XXX verify
return make_tuple(0, fmt::errorf("int %s invalid", s.c_str())); return make_tuple(0, fmt::errorf("int %s invalid", v(s)));
return make_tuple(v, nil); return make_tuple(v, nil);
} }
...@@ -214,7 +214,7 @@ tuple<uint64_t, error> parseUint(const string& s) { ...@@ -214,7 +214,7 @@ tuple<uint64_t, error> parseUint(const string& s) {
uint64_t v; uint64_t v;
int n = sscanf(s.c_str(), "%" SCNu64, &v); // XXX verify int n = sscanf(s.c_str(), "%" SCNu64, &v); // XXX verify
if (!(n == 1 && std::to_string(v) == s)) // XXX verify if (!(n == 1 && std::to_string(v) == s)) // XXX verify
return make_tuple(0, fmt::errorf("uint %s invalid", s.c_str())); return make_tuple(0, fmt::errorf("uint %s invalid", v(s)));
return make_tuple(v, nil); return make_tuple(v, nil);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment