Merge pull request #119 from boingoing/fix_napi_get_current_env

boingoing · web-flow · commit 898016fd86da · 2017-02-28T18:52:41.000-08:00
Fix definition of napi_get_current_env
diff --git a/src/node_jsrtapi.cc b/src/node_jsrtapi.cc
@@ -316,8 +316,9 @@ napi_status napi_set_last_error(JsErrorCode jsError, void* engine_reserved = nul
 }
 
 //Stub for now
-napi_env napi_get_current_env() {
-  return nullptr;
+napi_status napi_get_current_env(napi_env* e) {
+  *e = nullptr;
+  return napi_ok;
 }
 
 napi_status napi_create_function(napi_env e, napi_callback cb, void* data, napi_value* result) {

Original file line number	Diff line number	Diff line change
`@@ -316,8 +316,9 @@ napi_status napi_set_last_error(JsErrorCode jsError, void* engine_reserved = nul`
`316`	`316`	`}`
`317`	`317`
`318`	`318`	`//Stub for now`
`319`		`-napi_env napi_get_current_env() {`
`320`		`- return nullptr;`
	`319`	`+napi_status napi_get_current_env(napi_env* e) {`
	`320`	`+ *e = nullptr;`
	`321`	`+ return napi_ok;`
`321`	`322`	`}`
`322`	`323`
`323`	`324`	`napi_status napi_create_function(napi_env e, napi_callback cb, void* data, napi_value* result) {`