src: make node::SignalWrap::OnSignal into lambda

PR-URL: #26184 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
nodejs · Feb 28, 2019 · 831aa9a · 831aa9a
1 parent 8a551b9
commit 831aa9a
Showing 1 changed file with 11 additions and 11 deletions.
diff --git a/src/signal_wrap.cc b/src/signal_wrap.cc
@@ -101,7 +101,17 @@ class SignalWrap : public HandleWrap {
       }
     }
 #endif
-    int err = uv_signal_start(&wrap->handle_, OnSignal, signum);
+    int err = uv_signal_start(
+        &wrap->handle_,
+        [](uv_signal_t* handle, int signum) {
+          SignalWrap* wrap = ContainerOf(&SignalWrap::handle_, handle);
+          Environment* env = wrap->env();
+          HandleScope handle_scope(env->isolate());
+          Context::Scope context_scope(env->context());
+          Local<Value> arg = Integer::New(env->isolate(), signum);
+          wrap->MakeCallback(env->onsignal_string(), 1, &arg);
+        },
+        signum);
     args.GetReturnValue().Set(err);
   }
 
@@ -112,16 +122,6 @@ class SignalWrap : public HandleWrap {
     args.GetReturnValue().Set(err);
   }
 
-  static void OnSignal(uv_signal_t* handle, int signum) {
-    SignalWrap* wrap = ContainerOf(&SignalWrap::handle_, handle);
-    Environment* env = wrap->env();
-    HandleScope handle_scope(env->isolate());
-    Context::Scope context_scope(env->context());
-
-    Local<Value> arg = Integer::New(env->isolate(), signum);
-    wrap->MakeCallback(env->onsignal_string(), 1, &arg);
-  }
-
   uv_signal_t handle_;
 };