refactor: use Node-API in comments (#194)

Author: toyobayashi

packages/emnapi/include/node/js_native_api_types.h

@@ -7,12 +7,11 @@
 #ifdef NAPI_EXPERIMENTAL
 #define NAPI_VERSION NAPI_VERSION_EXPERIMENTAL
 #else
-// The baseline version for N-API.
-// The NAPI_VERSION controls which version will be used by default when
-// compilling a native addon. If the addon developer specifically wants to use
-// functions available in a new version of N-API that is not yet ported in all
-// LTS versions, they can set NAPI_VERSION knowing that they have specifically
-// depended on that version.
+// The baseline version for Node-API.
+// NAPI_VERSION controls which version is used by default when compiling
+// a native addon. If the addon developer wants to use functions from a
+// newer Node-API version not yet available in all LTS versions, they can
+// set NAPI_VERSION to explicitly depend on that version.
 #define NAPI_VERSION 8
 #endif
 #endif
@@ -31,7 +30,7 @@
 
 // This file needs to be compatible with C compilers.
 // This is a public include file, and these includes have essentially
-// became part of it's API.
+// become part of its API.
 #include <stddef.h>  // NOLINT(modernize-deprecated-headers)
 #include <stdint.h>  // NOLINT(modernize-deprecated-headers)
 

packages/runtime/src/env.ts

@@ -284,7 +284,7 @@ export class NodeEnv extends Env {
             }
           }
         warn(
-          'Uncaught N-API callback exception detected, please run node with option --force-node-api-uncaught-exceptions-policy=true to handle those exceptions properly.',
+          'Uncaught Node-API callback exception detected, please run node with option --force-node-api-uncaught-exceptions-policy=true to handle those exceptions properly.',
           'DeprecationWarning',
           'DEP0168'
         )

packages/test/constructor/constructor-null.test.js

@@ -3,7 +3,7 @@
 const assert = require('assert')
 const { load } = require('../util')
 
-// Test passing NULL to object-related N-APIs.
+// Test passing NULL to object-related Node-APIs.
 module.exports = load('constructor').then(({ testNull }) => {
   const expectedResult = {
     envIsNull: 'Invalid argument',

packages/test/object/object_null.test.js

@@ -5,7 +5,7 @@ const assert = require('assert')
 const { load } = require('../util')
 
 module.exports = load('object').then(test_object => {
-// Test passing NULL to object-related N-APIs.
+// Test passing NULL to object-related Node-APIs.
   const { testNull } = test_object
 
   const expectedForProperty = {

packages/test/trap_in_thread/binding.c

@@ -25,7 +25,7 @@ static napi_value Abort(napi_env env, napi_callback_info info) {
   napi_threadsafe_function ts_fn;
   NODE_API_CALL(env, napi_get_cb_info(env, info, &argc, &argv, NULL, NULL));
   NODE_API_CALL(env, napi_create_string_utf8(env,
-      "N-API Thread-safe Function Test", NAPI_AUTO_LENGTH, &async_name));
+      "Node-API Thread-safe Function Test", NAPI_AUTO_LENGTH, &async_name));
   NODE_API_CALL(env, napi_create_threadsafe_function(env,
                                                      argv,
                                                      NULL,
@@ -49,7 +49,7 @@ static napi_value ReleaseInThread(napi_env env, napi_callback_info info) {
   pthread_t uv_threads;
   NODE_API_CALL(env, napi_get_cb_info(env, info, &argc, &argv, NULL, NULL));
   NODE_API_CALL(env, napi_create_string_utf8(env,
-      "N-API Thread-safe Function Test", NAPI_AUTO_LENGTH, &async_name));
+      "Node-API Thread-safe Function Test", NAPI_AUTO_LENGTH, &async_name));
   NODE_API_CALL(env, napi_create_threadsafe_function(env,
                                                      argv,
                                                      NULL,
@@ -74,7 +74,7 @@ static napi_value AbortInThread(napi_env env, napi_callback_info info) {
   pthread_t uv_threads;
   NODE_API_CALL(env, napi_get_cb_info(env, info, &argc, &argv, NULL, NULL));
   NODE_API_CALL(env, napi_create_string_utf8(env,
-      "N-API Thread-safe Function Test", NAPI_AUTO_LENGTH, &async_name));
+      "Node-API Thread-safe Function Test", NAPI_AUTO_LENGTH, &async_name));
   NODE_API_CALL(env, napi_create_threadsafe_function(env,
                                                      argv,
                                                      NULL,

packages/test/tsfn/binding.c

@@ -2,7 +2,7 @@
 // on a threading library for a new project it bears remembering that in the
 // future libuv may introduce API changes which may render it non-ABI-stable,
 // which, in turn, may affect the ABI stability of the project despite its use
-// of N-API.
+// of Node-API.
 #ifdef __EMSCRIPTEN__
 #include <emscripten.h>
 #endif
@@ -233,8 +233,11 @@ static napi_value StartThreadInternal(napi_env env,
 
   NODE_API_ASSERT(env, (ts_fn == NULL), "Existing thread-safe function");
   napi_value async_name;
-  NODE_API_CALL(env, napi_create_string_utf8(env,
-      "N-API Thread-safe Function Test", NAPI_AUTO_LENGTH, &async_name));
+  NODE_API_CALL(env,
+                napi_create_string_utf8(env,
+                                        "Node-API Thread-safe Function Test",
+                                        NAPI_AUTO_LENGTH,
+                                        &async_name));
   NODE_API_CALL(env,
       napi_get_value_uint32(env, argv[3], &ts_info.max_queue_size));
   NODE_API_CALL(env, napi_create_threadsafe_function(env,