src: use MakeCallback() -> Call() in AsyncWorker

Change `AsyncWorker::OnOK()` and `AsyncWorker::OnError()` callbacks to **NOT** use `MakeCallback()`. An ordinary function call (`_callback::Call()`) is now correct. PR-URL: #361 Refs: https://nodejs.org/api/n-api.html#n_api_napi_make_callback Reviewed-By: Michael Dawson <[email protected]> Reviewed-By: Gabriel Schulhof <[email protected]> Reviewed-By: Nicola Del Gobbo <[email protected]>
nodejs · Oct 22, 2018 · fa3a615 · fa3a615
1 parent 2342415
commit fa3a615
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/napi-inl.h b/napi-inl.h
@@ -3588,11 +3588,11 @@ inline FunctionReference& AsyncWorker::Callback() {
 }
 
 inline void AsyncWorker::OnOK() {
-  _callback.MakeCallback(_receiver.Value(), std::initializer_list<napi_value>{});
+  _callback.Call(_receiver.Value(), std::initializer_list<napi_value>{});
 }
 
 inline void AsyncWorker::OnError(const Error& e) {
-  _callback.MakeCallback(_receiver.Value(), std::initializer_list<napi_value>{ e.Value() });
+  _callback.Call(_receiver.Value(), std::initializer_list<napi_value>{ e.Value() });
 }
 
 inline void AsyncWorker::SetError(const std::string& error) {