node-addon-api 4.3.0 → 5.1.0

package/package.json

@@ -15,6 +15,10 @@
       "name": "Alexander Floh",
       "url": "https://github.com/alexanderfloh"
     },
+    {
+      "name": "Ammar Faizi",
+      "url": "https://github.com/ammarfaizi2"
+    },
     {
       "name": "András Timár, Dr",
       "url": "https://github.com/timarandras"
@@ -71,6 +75,10 @@
       "name": "Daniel Bevenius",
       "url": "https://github.com/danbev"
     },
+    {
+      "name": "Dante Calderón",
+      "url": "https://github.com/dantehemerson"
+    },
     {
       "name": "Darshan Sen",
       "url": "https://github.com/RaisinTen"
@@ -95,14 +103,22 @@
       "name": "Doni Rubiagatra",
       "url": "https://github.com/rubiagatra"
     },
-    {
-      "name": "Ferdinand Holzer",
-      "url": "https://github.com/fholzer"
-    },
     {
       "name": "Eric Bickle",
       "url": "https://github.com/ebickle"
     },
+    {
+      "name": "extremeheat",
+      "url": "https://github.com/extremeheat"
+    },
+    {
+      "name": "Feng Yu",
+      "url": "https://github.com/F3n67u"
+    },
+    {
+      "name": "Ferdinand Holzer",
+      "url": "https://github.com/fholzer"
+    },
     {
       "name": "Gabriel Schulhof",
       "url": "https://github.com/gabrielschulhof"
@@ -143,6 +159,10 @@
       "name": "Jason Ginchereau",
       "url": "https://github.com/jasongin"
     },
+    {
+      "name": "Jenny",
+      "url": "https://github.com/egg-bread"
+    },
     {
       "name": "Jeroen Janssen",
       "url": "https://github.com/japj"
@@ -163,6 +183,10 @@
       "name": "joshgarde",
       "url": "https://github.com/joshgarde"
     },
+    {
+      "name": "Julian Mesa",
+      "url": "https://github.com/julianmesa-gitkraken"
+    },
     {
       "name": "Kasumi Hanazuki",
       "url": "https://github.com/hanazuki"
@@ -195,6 +219,10 @@
       "name": "Kyle Farnung",
       "url": "https://github.com/kfarnung"
     },
+    {
+      "name": "Kyle Kovacs",
+      "url": "https://github.com/nullromo"
+    },
     {
       "name": "legendecas",
       "url": "https://github.com/legendecas"
@@ -263,10 +291,22 @@
       "name": "pacop",
       "url": "https://github.com/pacop"
     },
+    {
+      "name": "Peter Šándor",
+      "url": "https://github.com/petersandor"
+    },
     {
       "name": "Philipp Renoth",
       "url": "https://github.com/DaAitch"
     },
+    {
+      "name": "rgerd",
+      "url": "https://github.com/rgerd"
+    },
+    {
+      "name": "Richard Lau",
+      "url": "https://github.com/richardlau"
+    },
     {
       "name": "Rolf Timmermans",
       "url": "https://github.com/rolftimmermans"
@@ -279,6 +319,10 @@
       "name": "Ryuichi Okumura",
       "url": "https://github.com/okuryu"
     },
+    {
+      "name": "Saint Gabriel",
+      "url": "https://github.com/chineduG"
+    },
     {
       "name": "Sampson Gao",
       "url": "https://github.com/sampsongao"
@@ -319,11 +363,19 @@
       "name": "Vlad Velmisov",
       "url": "https://github.com/Velmisov"
     },
+    {
+      "name": "Vladimir Morozov",
+      "url": "https://github.com/vmoroz"
+
+    },
     {
       "name": "WenheLI",
       "url": "https://github.com/WenheLI"
     },
-
+    {
+      "name": "Xuguang Mei",
+      "url": "https://github.com/meixg"
+    },
     {
       "name": "Yohei Kishimoto",
       "url": "https://github.com/morokosi"
@@ -335,6 +387,10 @@
     {
       "name": "Ziqiu Zhao",
       "url": "https://github.com/ZzqiZQute"
+    },
+    {
+      "name": "Feng Yu",
+      "url": "https://github.com/F3n67u"
     }
   ],
   "description": "Node.js API (Node-API)",
@@ -385,6 +441,7 @@
     "benchmark": "node benchmark",
     "pretest": "node-gyp rebuild -C test",
     "test": "node test",
+    "test:debug": "node-gyp rebuild -C test --debug && NODE_API_BUILD_CONFIG=Debug node ./test/index.js",
     "predev": "node-gyp rebuild -C test --debug",
     "dev": "node test",
     "predev:incremental": "node-gyp configure build -C test --debug",
@@ -394,6 +451,6 @@
     "lint:fix": "node tools/clang-format --fix && node tools/eslint-format --fix"
   },
   "pre-commit": "lint",
-  "version": "4.3.0",
+  "version": "5.1.0",
   "support": true
 }

package/tools/check-napi.js

@@ -4,16 +4,15 @@
 
 const fs = require('fs');
 const path = require('path');
-const child_process = require('child_process');
 
 // Read the output of the command, break it into lines, and use the reducer to
 // decide whether the file is an N-API module or not.
-function checkFile(file, command, argv, reducer) {
-  const child = child_process.spawn(command, argv, {
+function checkFile (file, command, argv, reducer) {
+  const child = require('child_process').spawn(command, argv, {
     stdio: ['inherit', 'pipe', 'inherit']
   });
   let leftover = '';
-  let isNapi = undefined;
+  let isNapi;
   child.stdout.on('data', (chunk) => {
     if (isNapi === undefined) {
       chunk = (leftover + chunk.toString()).split(/[\r\n]+/);
@@ -27,11 +26,11 @@ function checkFile(file, command, argv, reducer) {
   child.on('close', (code, signal) => {
     if ((code === null && signal !== null) || (code !== 0)) {
       console.log(
-        command + ' exited with code: '  + code + ' and signal: ' + signal);
+        command + ' exited with code: ' + code + ' and signal: ' + signal);
     } else {
       // Green if it's a N-API module, red otherwise.
       console.log(
-          '\x1b[' + (isNapi ? '42' : '41') + 'm' +
+        '\x1b[' + (isNapi ? '42' : '41') + 'm' +
           (isNapi ? '    N-API' : 'Not N-API') +
           '\x1b[0m: ' + file);
     }
@@ -39,7 +38,7 @@ function checkFile(file, command, argv, reducer) {
 }
 
 // Use nm -a to list symbols.
-function checkFileUNIX(file) {
+function checkFileUNIX (file) {
   checkFile(file, 'nm', ['-a', file], (soFar, line) => {
     if (soFar === undefined) {
       line = line.match(/([0-9a-f]*)? ([a-zA-Z]) (.*$)/);
@@ -54,7 +53,7 @@ function checkFileUNIX(file) {
 }
 
 // Use dumpbin /imports to list symbols.
-function checkFileWin32(file) {
+function checkFileWin32 (file) {
   checkFile(file, 'dumpbin', ['/imports', file], (soFar, line) => {
     if (soFar === undefined) {
       line = line.match(/([0-9a-f]*)? +([a-zA-Z0-9]) (.*$)/);
@@ -68,16 +67,16 @@ function checkFileWin32(file) {
 
 // Descend into a directory structure and pass each file ending in '.node' to
 // one of the above checks, depending on the OS.
-function recurse(top) {
+function recurse (top) {
   fs.readdir(top, (error, items) => {
     if (error) {
-      throw ("error reading directory " + top + ": " + error);
+      throw new Error('error reading directory ' + top + ': ' + error);
     }
     items.forEach((item) => {
       item = path.join(top, item);
       fs.stat(item, ((item) => (error, stats) => {
         if (error) {
-          throw ("error about " + item + ": " + error);
+          throw new Error('error about ' + item + ': ' + error);
         }
         if (stats.isDirectory()) {
           recurse(item);
@@ -86,9 +85,9 @@ function recurse(top) {
             // artefacts of node-addon-api having identified a version of
             // Node.js that ships with a correct implementation of N-API.
             path.basename(item) !== 'nothing.node') {
-          process.platform === 'win32' ?
-              checkFileWin32(item) :
-              checkFileUNIX(item);
+          process.platform === 'win32'
+            ? checkFileWin32(item)
+            : checkFileUNIX(item);
         }
       })(item));
     });

package/tools/clang-format.js

@@ -20,7 +20,10 @@ function main (args) {
   }
 
   const clangFormatPath = path.dirname(require.resolve('clang-format'));
-  const options = ['--binary=node_modules/.bin/clang-format', '--style=file'];
+  const binary = process.platform === 'win32'
+    ? 'node_modules\\.bin\\clang-format.cmd'
+    : 'node_modules/.bin/clang-format';
+  const options = ['--binary=' + binary, '--style=file'];
   if (fix) {
     options.push(FORMAT_START);
   } else {