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.

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>
  • Loading branch information
addaleax authored and MylesBorins committed Apr 16, 2018
1 parent f8c2585 commit 577f327
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/addons-napi/test_constructor/test_constructor.c
Expand Up @@ -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;
}
Expand Down

0 comments on commit 577f327

Please sign in to comment.