Skip to content

Commit 84646ef

Browse files
addaleaxaddaleax
committed
src: allow std::string_view arguments to SPrintF() and friends
Modernize the code so there is no need to work with raw C strings anymore.
1 parent c6316f9 commit 84646ef

File tree

3 files changed

+28
-28
lines changed

3 files changed

+28
-28
lines changed

‎src/debug_utils-inl.h‎

Lines changed: 18 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -65,31 +65,32 @@ std::string ToBaseString(const T& value) {
6565
return ToStringHelper::BaseConvert<BASE_BITS>(value);
6666
}
6767

68-
inline std::string SPrintFImpl(const char* format) {
69-
const char* p = strchr(format, '%');
70-
if (p == nullptr) [[unlikely]]
71-
return format;
72-
CHECK_EQ(p[1], '%'); // Only '%%' allowed when there are no arguments.
68+
inline std::string SPrintFImpl(std::string_view format) {
69+
auto offset = format.find('%');
70+
if (offset == std::string_view::npos)
71+
return std::string(format);
72+
CHECK_LT(offset + 1, format.size());
73+
CHECK_EQ(format[offset + 1], '%'); // Only '%%' allowed when there are no arguments.
7374

74-
return std::string(format, p + 1) + SPrintFImpl(p + 2);
75+
return std::string(format.substr(0, offset + 1)) + SPrintFImpl(format.substr(offset + 2));
7576
}
7677

7778
template <typename Arg, typename... Args>
7879
std::string COLD_NOINLINE SPrintFImpl( // NOLINT(runtime/string)
79-
const char* format, Arg&& arg, Args&&... args) {
80-
const char* p = strchr(format, '%');
81-
CHECK_NOT_NULL(p); // If you hit this, you passed in too many arguments.
82-
std::string ret(format, p);
80+
std::string_view format, Arg&& arg, Args&&... args) {
81+
auto offset = format.find('%');
82+
CHECK_NE(offset, std::string_view::npos); // If you hit this, you passed in too many arguments.
83+
std::string ret(format.substr(0, offset));
8384
// Ignore long / size_t modifiers
84-
while (strchr("lz", *++p) != nullptr) {}
85-
switch (*p) {
85+
while (++offset < format.size() && (format[offset] == 'l' || format[offset] == 'z'));
86+
switch (offset == format.size() ? '\0' : format[offset]) {
8687
case '%': {
87-
return ret + '%' + SPrintFImpl(p + 1,
88+
return ret + '%' + SPrintFImpl(format.substr(offset + 1),
8889
std::forward<Arg>(arg),
8990
std::forward<Args>(args)...);
9091
}
9192
default: {
92-
return ret + '%' + SPrintFImpl(p,
93+
return ret + '%' + SPrintFImpl(format.substr(offset),
9394
std::forward<Arg>(arg),
9495
std::forward<Args>(args)...);
9596
}
@@ -120,17 +121,17 @@ std::string COLD_NOINLINE SPrintFImpl( // NOLINT(runtime/string)
120121
break;
121122
}
122123
}
123-
return ret + SPrintFImpl(p + 1, std::forward<Args>(args)...);
124+
return ret + SPrintFImpl(format.substr(offset + 1), std::forward<Args>(args)...);
124125
}
125126

126127
template <typename... Args>
127128
std::string COLD_NOINLINE SPrintF( // NOLINT(runtime/string)
128-
const char* format, Args&&... args) {
129+
std::string_view format, Args&&... args) {
129130
return SPrintFImpl(format, std::forward<Args>(args)...);
130131
}
131132

132133
template <typename... Args>
133-
void COLD_NOINLINE FPrintF(FILE* file, const char* format, Args&&... args) {
134+
void COLD_NOINLINE FPrintF(FILE* file, std::string_view format, Args&&... args) {
134135
FWrite(file, SPrintF(format, std::forward<Args>(args)...));
135136
}
136137

‎src/debug_utils.h‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,9 +33,9 @@ inline std::string ToString(const T& value);
3333
// - Supports %p and %s. %d, %i and %u are aliases for %s.
3434
// - Accepts any class that has a ToString() method for stringification.
3535
template <typename... Args>
36-
inline std::string SPrintF(const char* format, Args&&... args);
36+
inline std::string SPrintF(std::string_view format, Args&&... args);
3737
template <typename... Args>
38-
inline void FPrintF(FILE* file, const char* format, Args&&... args);
38+
inline void FPrintF(FILE* file, std::string_view format, Args&&... args);
3939
void NODE_EXTERN_PRIVATE FWrite(FILE* file, const std::string& str);
4040

4141
// Listing the AsyncWrap provider types first enables us to cast directly

‎src/node_errors.h‎

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -140,7 +140,7 @@ void OOMErrorHandler(const char* location, const v8::OOMDetails& details);
140140
#define V(code, type) \
141141
template <typename... Args> \
142142
inline v8::Local<v8::Object> code( \
143-
v8::Isolate* isolate, const char* format, Args&&... args) { \
143+
v8::Isolate* isolate, std::string_view format, Args&&... args) { \
144144
std::string message; \
145145
if (sizeof...(Args) == 0) { \
146146
message = format; \
@@ -165,17 +165,18 @@ void OOMErrorHandler(const char* location, const v8::OOMDetails& details);
165165
} \
166166
template <typename... Args> \
167167
inline void THROW_##code( \
168-
v8::Isolate* isolate, const char* format, Args&&... args) { \
168+
v8::Isolate* isolate, std::string_view format, Args&&... args) { \
169169
isolate->ThrowException( \
170170
code(isolate, format, std::forward<Args>(args)...)); \
171171
} \
172172
template <typename... Args> \
173173
inline void THROW_##code( \
174-
Environment* env, const char* format, Args&&... args) { \
174+
Environment* env, std::string_view format, Args&&... args) { \
175175
THROW_##code(env->isolate(), format, std::forward<Args>(args)...); \
176176
} \
177177
template <typename... Args> \
178-
inline void THROW_##code(Realm* realm, const char* format, Args&&... args) { \
178+
inline void THROW_##code( \
179+
Realm* realm, std::string_view format, Args&&... args) { \
179180
THROW_##code(realm->isolate(), format, std::forward<Args>(args)...); \
180181
}
181182
ERRORS_WITH_CODE(V)
@@ -258,10 +259,8 @@ PREDEFINED_ERROR_MESSAGES(V)
258259
// Errors with predefined non-static messages
259260
inline void THROW_ERR_SCRIPT_EXECUTION_TIMEOUT(Environment* env,
260261
int64_t timeout) {
261-
std::ostringstream message;
262-
message << "Script execution timed out after ";
263-
message << timeout << "ms";
264-
THROW_ERR_SCRIPT_EXECUTION_TIMEOUT(env, message.str().c_str());
262+
THROW_ERR_SCRIPT_EXECUTION_TIMEOUT(env, "Script execution timed out after %dms",
263+
timeout);
265264
}
266265

267266
inline void THROW_ERR_REQUIRE_ASYNC_MODULE(
@@ -283,7 +282,7 @@ inline void THROW_ERR_REQUIRE_ASYNC_MODULE(
283282
message += "\n Requiring ";
284283
message += +utf8.out();
285284
}
286-
THROW_ERR_REQUIRE_ASYNC_MODULE(env, message.c_str());
285+
THROW_ERR_REQUIRE_ASYNC_MODULE(env, message);
287286
}
288287

289288
inline v8::Local<v8::Object> ERR_BUFFER_TOO_LARGE(v8::Isolate* isolate) {

0 commit comments

Comments
 (0)