fixup! buffer: add fast api for isAscii & isUtf8

Author: anonrig

src/node_buffer.cc

@@ -1175,24 +1175,24 @@ void Swap64(const FunctionCallbackInfo<Value>& args) {
   args.GetReturnValue().Set(args[0]);
 }
 
-bool FastIsUtf8(v8::Local<v8::Value>,
-                const v8::FastApiTypedArray<uint8_t>& buffer) {
-  uint8_t* buffer_data;
-  CHECK(buffer.getStorageIfAligned(&buffer_data));
+bool FastIsUtf8(v8::Local<v8::Value>, Local<Value> buffer) {
   TRACK_V8_FAST_API_CALL("buffer.isUtf8");
+  CHECK(buffer->IsTypedArray());
+  auto buffer_ = buffer.As<v8::TypedArray>()->Buffer();
+  auto buffer_data = buffer_->Data();
   return simdutf::validate_utf8(reinterpret_cast<const char*>(buffer_data),
-                                buffer.length());
+                                buffer_->ByteLength());
 }
 
 static v8::CFunction fast_is_utf8(v8::CFunction::Make(FastIsUtf8));
 
-bool FastIsAscii(v8::Local<v8::Value>,
-                 const v8::FastApiTypedArray<uint8_t>& buffer) {
-  uint8_t* buffer_data;
-  CHECK(buffer.getStorageIfAligned(&buffer_data));
+bool FastIsAscii(v8::Local<v8::Value>, Local<Value> buffer) {
   TRACK_V8_FAST_API_CALL("buffer.isAscii");
+  CHECK(buffer->IsTypedArray());
+  auto buffer_ = buffer.As<v8::TypedArray>()->Buffer();
+  auto buffer_data = buffer_->Data();
   return simdutf::validate_ascii(reinterpret_cast<const char*>(buffer_data),
-                                 buffer.length());
+                                 buffer_->ByteLength());
 }
 
 static v8::CFunction fast_is_ascii(v8::CFunction::Make(FastIsAscii));

src/node_external_reference.h

@@ -42,8 +42,6 @@ using CFunctionCallbackWithInt64 = void (*)(v8::Local<v8::Object> unused,
 using CFunctionCallbackWithBool = void (*)(v8::Local<v8::Object> unused,
                                            v8::Local<v8::Object> receiver,
                                            bool);
-using CFunctionFastIsUtf8 = bool (*)(
-    v8::Local<v8::Value>, const v8::FastApiTypedArray<uint8_t>& buffer);
 using CFunctionCallbackWithString =
     bool (*)(v8::Local<v8::Value>, const v8::FastOneByteString& input);
 using CFunctionCallbackWithStrings =
@@ -111,7 +109,6 @@ class ExternalReferenceRegistry {
   V(CFunctionCallbackValueReturnDoubleUnusedReceiver)                          \
   V(CFunctionCallbackWithInt64)                                                \
   V(CFunctionCallbackWithBool)                                                 \
-  V(CFunctionFastIsUtf8)                                                       \
   V(CFunctionCallbackWithString)                                               \
   V(CFunctionCallbackWithStrings)                                              \
   V(CFunctionCallbackWithTwoUint8Arrays)                                       \

test/parallel/test-buffer-isutf8-isascii-fast-api.js

@@ -24,11 +24,8 @@ assert.strictEqual(buffer.isAscii(nonAsciiBuffer), false);
 
 // Test detached buffers
 const detachedBuffer = new ArrayBuffer(10);
-try {
-  detachedBuffer.detach();
-} catch (_e) {
-  console.log('Skipping detached buffer tests - detach not supported');
-}
+// Let's detach the buffer if it's supported
+detachedBuffer.detach?.();
 
 if (detachedBuffer.detached) {
   const typedArray = new Uint8Array(detachedBuffer);