1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

src: use Maybe<void> in ProcessEmitWarningSync

Browse Source 
Signed-off-by: Daeyeon Jeong <daeyeon.dev@gmail.com>
PR-URL: https://github.com/nodejs/node/pull/57250
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
This commit is contained in:
Daeyeon Jeong 2025-03-07 08:39:49 +09:00 committed by GitHub
parent 5401ac687c
commit 117c1ca9eb
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/node_process.h
View File

@ -37,7 +37,7 @@ inline v8::Maybe<bool> ProcessEmitWarning(Environment* env,
const char* fmt, const char* fmt,
Args&&... args); Args&&... args);
v8::Maybe<bool> ProcessEmitWarningSync(Environment* env, v8::Maybe<void> ProcessEmitWarningSync(Environment* env,
std::string_view message); std::string_view message);
v8::Maybe<bool> ProcessEmitExperimentalWarning(Environment* env, v8::Maybe<bool> ProcessEmitExperimentalWarning(Environment* env,
const std::string& warning); const std::string& warning);

9
src/node_process_events.cc
View File

@ -10,6 +10,7 @@ using v8::Function;
using v8::HandleScope; using v8::HandleScope;
using v8::Isolate; using v8::Isolate;
using v8::Just; using v8::Just;
using v8::JustVoid;
using v8::Local; using v8::Local;
using v8::Maybe; using v8::Maybe;
using v8::MaybeLocal; using v8::MaybeLocal;
@ -19,7 +20,7 @@ using v8::Object;
using v8::String; using v8::String;
using v8::Value; using v8::Value;
Maybe<bool> ProcessEmitWarningSync(Environment* env, std::string_view message) { Maybe<void> ProcessEmitWarningSync(Environment* env, std::string_view message) {
Isolate* isolate = env->isolate(); Isolate* isolate = env->isolate();
Local<Context> context = env->context(); Local<Context> context = env->context();
Local<String> message_string; Local<String> message_string;
@ -28,7 +29,7 @@ Maybe<bool> ProcessEmitWarningSync(Environment* env, std::string_view message) {
NewStringType::kNormal, NewStringType::kNormal,
static_cast<int>(message.size())) static_cast<int>(message.size()))
.ToLocal(&message_string)) { .ToLocal(&message_string)) {
return Nothing<bool>(); return Nothing<void>();
} }
Local<Value> argv[] = {message_string}; Local<Value> argv[] = {message_string};
@ -39,9 +40,9 @@ Maybe<bool> ProcessEmitWarningSync(Environment* env, std::string_view message) {
if (emit_function.As<Function>() if (emit_function.As<Function>()
->Call(context, v8::Undefined(isolate), arraysize(argv), argv) ->Call(context, v8::Undefined(isolate), arraysize(argv), argv)
.IsEmpty()) { .IsEmpty()) {
return Nothing<bool>(); return Nothing<void>();
} }
return Just(true); return JustVoid();
} }
MaybeLocal<Value> ProcessEmit(Environment* env, MaybeLocal<Value> ProcessEmit(Environment* env,