buffer: use Utf8LengthV2() instead of Utf8Length()

tniessen · targos · commit 8b5ada4b310b · 2025-05-31T13:23:21.000+02:00
`Utf8Length()` is deprecated and was replaced during the V8 update, but a subsequent commit introduced a new occurrence. Refs: #58070 Refs: #58048 PR-URL: #58156 Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: James M Snell <jasnell@gmail.com>
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`