Fix lint and test

nodejs · Jul 23, 2021 · e69fb2c · e69fb2c
1 parent f211f45
commit e69fb2c
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/napi-inl.h b/napi-inl.h
@@ -2114,11 +2114,14 @@ inline Value Function::Call(napi_value recv, size_t argc, const napi_value* args
   return Value(_env, result);
 }
 
-inline Value Function::Call(napi_value recv, size_t argc, const Value* args) const {
-  napi_value stackArgs[6];
+inline Value Function::Call(napi_value recv,
+                            size_t argc,
+                            const Value* args) const {
+  const size_t stackArgsCount = 6;
+  napi_value stackArgs[stackArgsCount];
   std::vector<napi_value> heapArgs;
   napi_value* argv;
-  if (argc <= std::size(stackArgs)) {
+  if (argc <= stackArgsCount) {
     argv = stackArgs;
   } else {
     heapArgs.resize(argc);

diff --git a/test/function.cc b/test/function.cc
@@ -230,7 +230,8 @@ Object InitFunction(Env env) {
   exports["callWithArgs"] = Function::New(env, CallWithArgs);
   exports["callWithVector"] = Function::New(env, CallWithVector);
   exports["callWithCStyleArray"] = Function::New(env, CallWithCStyleArray);
-  exports["callWithCStyleArrayUsingCppWrapper"] = Function::New(env, CallWithCStyleArrayUsingCppWrapper);
+  exports["callWithCStyleArrayUsingCppWrapper"] =
+      Function::New(env, CallWithCStyleArrayUsingCppWrapper);
   exports["callWithReceiverAndCStyleArray"] =
       Function::New(env, CallWithReceiverAndCStyleArray);
   exports["callWithReceiverAndCStyleArrayUsingCppWrapper"] =
@@ -266,7 +267,8 @@ Object InitFunction(Env env) {
   exports["callWithArgs"] = Function::New<CallWithArgs>(env);
   exports["callWithVector"] = Function::New<CallWithVector>(env);
   exports["callWithCStyleArray"] = Function::New<CallWithCStyleArray>(env);
-  exports["callWithCStyleArrayUsingCppWrapper"] = Function::New<CallWithCStyleArrayUsingCppWrapper>(env);
+  exports["callWithCStyleArrayUsingCppWrapper"] =
+      Function::New<CallWithCStyleArrayUsingCppWrapper>(env);
   exports["callWithReceiverAndCStyleArray"] =
       Function::New<CallWithReceiverAndCStyleArray>(env);
   exports["callWithReceiverAndCStyleArrayUsingCppWrapper"] =