Skip to content
This repository was archived by the owner on Oct 15, 2020. It is now read-only.

Commit 2e0d64c

Browse files
committed
[fixup] additional cleanup
1 parent 83a16c1 commit 2e0d64c

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

test/addons-napi/test_new_target/binding.c

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ napi_value BaseClass(napi_env env, napi_callback_info info) {
1515
NAPI_ASSERT(env, !result, "this !== new.target");
1616

1717
// new.target !== undefined because we should be called as a new expression
18+
NAPI_ASSERT(env, newTargetArg != NULL, "newTargetArg != NULL");
1819
NAPI_CALL(env, napi_strict_equals(env, newTargetArg, undefined, &result));
1920
NAPI_ASSERT(env, !result, "new.target !== undefined");
2021

@@ -32,9 +33,8 @@ napi_value Constructor(napi_env env, napi_callback_info info) {
3233
napi_value undefined;
3334
NAPI_CALL(env, napi_get_undefined(env, &undefined));
3435

35-
NAPI_ASSERT(env, newTargetArg != NULL, "newTargetArg != NULL");
36-
3736
// new.target !== undefined because we should be called as a new expression
37+
NAPI_ASSERT(env, newTargetArg != NULL, "newTargetArg != NULL");
3838
NAPI_CALL(env, napi_strict_equals(env, newTargetArg, undefined, &result));
3939
NAPI_ASSERT(env, !result, "new.target !== undefined");
4040

@@ -46,7 +46,6 @@ napi_value Constructor(napi_env env, napi_callback_info info) {
4646
}
4747

4848
napi_value OrdinaryFunction(napi_env env, napi_callback_info info) {
49-
bool result;
5049
napi_value newTargetArg;
5150
NAPI_CALL(env, napi_get_new_target(env, info, &newTargetArg));
5251

0 commit comments

Comments
 (0)