Skip to content

Commit 577f327

Browse files
addaleaxMylesBorins
authored andcommittedApr 16, 2018
test: fix compiler warning in n-api test
Missed in ca786c3. This does not actually affect the outcome because returning `nullptr` or `this` from a constructor has the same effect. Backport-PR-URL: #19447 PR-URL: #12318 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
1 parent f8c2585 commit 577f327

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed
 

‎test/addons-napi/test_constructor/test_constructor.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ napi_value Echo(napi_env env, napi_callback_info info) {
4040

4141
napi_value New(napi_env env, napi_callback_info info) {
4242
napi_value _this;
43-
NAPI_CALL(env, napi_get_cb_info(env, info, NULL, NULL, NULL, &_this));
43+
NAPI_CALL(env, napi_get_cb_info(env, info, NULL, NULL, &_this, NULL));
4444

4545
return _this;
4646
}

0 commit comments

Comments
 (0)
Please sign in to comment.