n-api: remove compiler warning

addaleax · MylesBorins · commit 8d3162d9e68b · 2018-04-16T17:45:55.000-04:00
`TryCatch` without an `Isolate*` argument is deprecated, so add one. Backport-PR-URL: #19447 PR-URL: #13014 Reviewed-By: Timothy Gu <timothygu99@gmail.com> Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com> Reviewed-By: Jason Ginchereau <jasongin@microsoft.com>
diff --git a/src/node_api.cc b/src/node_api.cc
@@ -2783,7 +2783,7 @@ class Work {
       // report it as a fatal exception. (There is no JavaScript on the
       // callstack that can possibly handle it.)
       if (!env->last_exception.IsEmpty()) {
-        v8::TryCatch try_catch;
+        v8::TryCatch try_catch(env->isolate);
         env->isolate->ThrowException(
           v8::Local<v8::Value>::New(env->isolate, env->last_exception));
         node::FatalException(env->isolate, try_catch);
