n-api: create N-API version 7

Mark `napi_detach_arraybuffer` and `napi_is_detached_arraybuffer` as stable. Signed-off-by: Gabriel Schulhof <gabriel.schulhof@intel.com> PR-URL: nodejs#35199 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Jiawen Geng <technicalcute@gmail.com> Reviewed-By: Chengzhong Wu <legendecas@gmail.com> Reviewed-By: Michael Dawson <mdawson@devrus.com>
gabrielschulhof · Sep 25, 2020 · 2320140 · 2320140
1 parent d065960
commit 2320140
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 10 deletions.
diff --git a/doc/api/n-api.md b/doc/api/n-api.md
@@ -2951,10 +2951,9 @@ of the ECMAScript Language Specification.
 ### napi_detach_arraybuffer
 <!-- YAML
 added: v10.22.0
+napiVersion: 7
 -->
 
-> Stability: 1 - Experimental
-
 ```C
 napi_status napi_detach_arraybuffer(napi_env env,
                                     napi_value arraybuffer)
@@ -2977,10 +2976,9 @@ defined in [Section 24.1.1.3][] of the ECMAScript Language Specification.
 ### napi_is_detached_arraybuffer
 <!-- YAML
 added: v10.22.0
+napiVersion: 7
 -->
 
-> Stability: 1 - Experimental
-
 ```C
 napi_status napi_is_detached_arraybuffer(napi_env env,
                                          napi_value arraybuffer,

diff --git a/src/node_api.h b/src/node_api.h
@@ -10,7 +10,7 @@
 #define NAPI_VERSION 2147483647
 #else
 // The baseline version for N-API
-#define NAPI_VERSION 6
+#define NAPI_VERSION 7
 #endif
 #endif
 
@@ -747,7 +747,7 @@ napi_get_all_property_names(napi_env env,
 
 #endif  // NAPI_VERSION >= 6
 
-#ifdef NAPI_EXPERIMENTAL
+#if NAPI_VERSION >= 7
 // ArrayBuffer detaching
 NAPI_EXTERN napi_status
 napi_detach_arraybuffer(napi_env env,
@@ -757,7 +757,7 @@ NAPI_EXTERN napi_status
 napi_is_detached_arraybuffer(napi_env env,
                              napi_value value,
                              bool* result);
-#endif  // NAPI_EXPERIMENTAL
+#endif  // NAPI_VERSION >= 7
 
 EXTERN_C_END
 

diff --git a/src/node_version.h b/src/node_version.h
@@ -115,6 +115,6 @@
 
 // The NAPI_VERSION provided by this version of the runtime. This is the version
 // which the Node binary being built supports.
-#define NAPI_VERSION  6
+#define NAPI_VERSION  7
 
 #endif  // SRC_NODE_VERSION_H_
diff --git a/test/addons-napi/test_general/test.js b/test/addons-napi/test_general/test.js
@@ -34,7 +34,7 @@ assert.notStrictEqual(test_general.testGetPrototype(baseObject),
 
 // test version management functions
 // expected version is currently 4
-assert.strictEqual(test_general.testGetVersion(), 6);
+assert.strictEqual(test_general.testGetVersion(), 7);
 
 const [ major, minor, patch, release ] = test_general.testGetNodeVersion();
 assert.strictEqual(process.version.split('-')[0],

diff --git a/test/addons-napi/test_typedarray/test_typedarray.c b/test/addons-napi/test_typedarray/test_typedarray.c
@@ -1,4 +1,3 @@
-#define NAPI_EXPERIMENTAL
 #include <node_api.h>
 #include <string.h>
 #include "../common.h"