use NAPI_THROW

vijos · Aug 29, 2019 · dedf9a8 · dedf9a8
1 parent 80d57a1
commit dedf9a8
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/sandbox/nodejs/sandbox-wrap.cc b/sandbox/nodejs/sandbox-wrap.cc
@@ -10,9 +10,9 @@ Sandbox::Options get_sandbox_options(const Napi::CallbackInfo &info) {
         return Sandbox::Options();
     }
     if (!info[0].IsObject()) {
-        Napi::TypeError::New(env, "invalid argument")
-            .ThrowAsJavaScriptException();
-        return Sandbox::Options();
+        NAPI_THROW(
+            Napi::TypeError::New(env, "invalid argument"),
+            Sandbox::Options());
     }
     Napi::Object js_options = info[0].ToObject();
     Sandbox::Options options;
@@ -46,7 +46,7 @@ SandboxWrap::SandboxWrap(const Napi::CallbackInfo &info)
       Sandbox(get_sandbox_options(info)) {
     Napi::Env env = info.Env();
     if (!Sandbox::init()) {
-        Napi::Error::New(env, "init failed").ThrowAsJavaScriptException();
+        NAPI_THROW_VOID(Napi::Error::New(env, "init failed"));
     }
 }