From d26cbedae0909005686c804fe59d64f41c3beda2 Mon Sep 17 00:00:00 2001 From: Anna Henningsen Date: Mon, 10 Apr 2017 23:57:18 +0200 Subject: [PATCH] test: fix compiler warning in n-api test Missed in ca786c3734f6e23e34cd60f13e6bdaab033c5739. This does not actually affect the outcome because returning `nullptr` or `this` from a constructor has the same effect. PR-URL: https://github.com/nodejs/node/pull/12318 Reviewed-By: Colin Ihrig Reviewed-By: James M Snell Reviewed-By: Michael Dawson --- test/addons-napi/test_constructor/test_constructor.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/addons-napi/test_constructor/test_constructor.c b/test/addons-napi/test_constructor/test_constructor.c index 23463235c315d7..0a73010d72f115 100644 --- a/test/addons-napi/test_constructor/test_constructor.c +++ b/test/addons-napi/test_constructor/test_constructor.c @@ -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; }