Skip to content

Commit

Permalink
test: fix compiler warning in n-api test
Browse files Browse the repository at this point in the history
Missed in ca786c3. This does
not actually affect the outcome because returning `nullptr` or
`this` from a constructor has the same effect.

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>
addaleax committed Apr 13, 2017
1 parent 3900cf6 commit 8bd26d3
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/addons-napi/test_constructor/test_constructor.c
Original file line number Diff line number Diff line change
@@ -40,7 +40,7 @@ napi_value Echo(napi_env env, napi_callback_info info) {

napi_value New(napi_env env, napi_callback_info info) {
napi_value _this;
NAPI_CALL(env, napi_get_cb_info(env, info, NULL, NULL, NULL, &_this));
NAPI_CALL(env, napi_get_cb_info(env, info, NULL, NULL, &_this, NULL));

return _this;
}

0 comments on commit 8bd26d3

Please sign in to comment.