n-api: create N-API version 7

Gabriel Schulhof · joesepi · commit dd80316e688e · 2020-10-22T18:30:32.000-04:00
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>
diff --git a/doc/api/n-api.md b/doc/api/n-api.md
@@ -3558,10 +3558,9 @@ added:
  - v13.0.0
  - v12.16.0
  - v10.22.0
+napiVersion: 7
 -->
 
-> Stability: 1 - Experimental
-
 ```c
 napi_status napi_detach_arraybuffer(napi_env env,
                                     napi_value arraybuffer)
@@ -3587,10 +3586,9 @@ added:
  - v13.3.0
  - v12.16.0
  - 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/js_native_api.h b/src/js_native_api.h
@@ -17,7 +17,7 @@
 // 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.
-#define NAPI_VERSION 6
+#define NAPI_VERSION 7
 #endif
 #endif
 
@@ -529,14 +529,17 @@ NAPI_EXTERN napi_status napi_get_instance_data(napi_env env,
                                                void** data);
 #endif  // NAPI_VERSION >= 6
 
-#ifdef NAPI_EXPERIMENTAL
+#if NAPI_VERSION >= 7
 // ArrayBuffer detaching
 NAPI_EXTERN napi_status napi_detach_arraybuffer(napi_env env,
                                                 napi_value arraybuffer);
 
 NAPI_EXTERN napi_status napi_is_detached_arraybuffer(napi_env env,
                                                      napi_value value,
                                                      bool* result);
+#endif  // NAPI_VERSION >= 7
+
+#ifdef NAPI_EXPERIMENTAL
 // Type tagging
 NAPI_EXTERN napi_status napi_type_tag_object(napi_env env,
                                              napi_value value,
diff --git a/src/node_version.h b/src/node_version.h
@@ -93,6 +93,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/js-native-api/test_general/test.js b/test/js-native-api/test_general/test.js
@@ -33,7 +33,7 @@ assert.notStrictEqual(test_general.testGetPrototype(baseObject),
                       test_general.testGetPrototype(extendedObject));
 
 // Test version management functions
-assert.strictEqual(test_general.testGetVersion(), 6);
+assert.strictEqual(test_general.testGetVersion(), 7);
 
 [
   123,
diff --git a/test/js-native-api/test_typedarray/test_typedarray.c b/test/js-native-api/test_typedarray/test_typedarray.c
@@ -1,4 +1,3 @@
-#define NAPI_EXPERIMENTAL
 #include <js_native_api.h>
 #include <string.h>
 #include <stdlib.h>

Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,7 @@ assert.notStrictEqual(test_general.testGetPrototype(baseObject),`
`33`	`33`	`test_general.testGetPrototype(extendedObject));`
`34`	`34`
`35`	`35`	`// Test version management functions`
`36`		`-assert.strictEqual(test_general.testGetVersion(), 6);`
	`36`	`+assert.strictEqual(test_general.testGetVersion(), 7);`
`37`	`37`
`38`	`38`	`[`
`39`	`39`	`123,`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,3 @@`
`1`		`-#define NAPI_EXPERIMENTAL`
`2`	`1`	`#include <js_native_api.h>`
`3`	`2`	`#include <string.h>`
`4`	`3`	`#include <stdlib.h>`