buffer: use Utf8LengthV2() instead of Utf8Length()

tniessen · tniessen · commit caf58f257951 · 2025-05-04T12:36:58.000+01:00
`Utf8Length()` is deprecated and was replaced during the V8 update, but a subsequent commit introduced a new occurrence. Refs: #58070 Refs: #58048
diff --git a/src/node_buffer.cc b/src/node_buffer.cc
@@ -742,7 +742,7 @@ uint32_t FastByteLengthUtf8(
   Local<String> sourceStr = sourceValue.As<String>();
 
   if (!sourceStr->IsExternalOneByte()) {
-    return sourceStr->Utf8Length(isolate);
+    return sourceStr->Utf8LengthV2(isolate);
   }
   auto source = sourceStr->GetExternalOneByteStringResource();
   // For short inputs, the function call overhead to simdutf is maybe

Original file line number	Diff line number	Diff line change
`@@ -742,7 +742,7 @@ uint32_t FastByteLengthUtf8(`
`742`	`742`	`Local<String> sourceStr = sourceValue.As<String>();`
`743`	`743`
`744`	`744`	`if (!sourceStr->IsExternalOneByte()) {`
`745`		`- return sourceStr->Utf8Length(isolate);`
	`745`	`+ return sourceStr->Utf8LengthV2(isolate);`
`746`	`746`	`}`
`747`	`747`	`auto source = sourceStr->GetExternalOneByteStringResource();`
`748`	`748`	`// For short inputs, the function call overhead to simdutf is maybe`