src: replace NewFromUtf8 with OneByteString where appropriate

jasnell · RafaelGSS · commit 5dab48fd9fae · 2025-04-17T10:51:00.000-03:00
PR-URL: #57096 Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Chengzhong Wu <legendecas@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
diff --git a/src/cares_wrap.cc b/src/cares_wrap.cc
@@ -1660,11 +1660,9 @@ void CanonicalizeIP(const FunctionCallbackInfo<Value>& args) {
       uv_inet_pton(af = AF_INET6, *ip, result) != 0)
     return;
 
-  char canonical_ip[INET6_ADDRSTRLEN];
+  char canonical_ip[INET6_ADDRSTRLEN]{};
   CHECK_EQ(0, uv_inet_ntop(af, result, canonical_ip, sizeof(canonical_ip)));
-  Local<String> val = String::NewFromUtf8(isolate, canonical_ip)
-      .ToLocalChecked();
-  args.GetReturnValue().Set(val);
+  args.GetReturnValue().Set(OneByteString(isolate, canonical_ip));
 }
 
 void ConvertIpv6StringToBuffer(const FunctionCallbackInfo<Value>& args) {
diff --git a/src/node_binding.cc b/src/node_binding.cc
@@ -683,8 +683,7 @@ void GetLinkedBinding(const FunctionCallbackInfo<Value>& args) {
 
   Local<Object> module = Object::New(env->isolate());
   Local<Object> exports = Object::New(env->isolate());
-  Local<String> exports_prop =
-      String::NewFromUtf8Literal(env->isolate(), "exports");
+  Local<String> exports_prop = env->exports_string();
   module->Set(env->context(), exports_prop, exports).Check();
 
   if (mod->nm_context_register_func != nullptr) {
diff --git a/src/node_buffer.cc b/src/node_buffer.cc
@@ -71,7 +71,6 @@ using v8::Just;
 using v8::Local;
 using v8::Maybe;
 using v8::MaybeLocal;
-using v8::NewStringType;
 using v8::Nothing;
 using v8::Number;
 using v8::Object;
@@ -1310,12 +1309,9 @@ static void Btoa(const FunctionCallbackInfo<Value>& args) {
     written = simdutf::binary_to_base64(*stack_buf, out_len, buffer.out());
   }
 
-  auto value =
-      String::NewFromOneByte(env->isolate(),
-                             reinterpret_cast<const uint8_t*>(buffer.out()),
-                             NewStringType::kNormal,
-                             written)
-          .ToLocalChecked();
+  auto value = OneByteString(
+      env->isolate(), reinterpret_cast<const uint8_t*>(buffer.out()), written);
+
   return args.GetReturnValue().Set(value);
 }
 
@@ -1365,12 +1361,9 @@ static void Atob(const FunctionCallbackInfo<Value>& args) {
   }
 
   if (result.error == simdutf::error_code::SUCCESS) {
-    auto value =
-        String::NewFromOneByte(env->isolate(),
+    auto value = OneByteString(env->isolate(),
                                reinterpret_cast<const uint8_t*>(buffer.out()),
-                               NewStringType::kNormal,
-                               result.count)
-            .ToLocalChecked();
+                               result.count);
     return args.GetReturnValue().Set(value);
   }
 
diff --git a/src/node_i18n.cc b/src/node_i18n.cc
@@ -327,9 +327,7 @@ void ICUErrorName(const FunctionCallbackInfo<Value>& args) {
   Environment* env = Environment::GetCurrent(args);
   CHECK(args[0]->IsInt32());
   UErrorCode status = static_cast<UErrorCode>(args[0].As<Int32>()->Value());
-  args.GetReturnValue().Set(
-      String::NewFromUtf8(env->isolate(),
-                          u_errorName(status)).ToLocalChecked());
+  args.GetReturnValue().Set(OneByteString(env->isolate(), u_errorName(status)));
 }
 
 }  // anonymous namespace
diff --git a/src/spawn_sync.cc b/src/spawn_sync.cc
@@ -707,10 +707,10 @@ Local<Object> SyncProcessRunner::BuildResultObject() {
   }
 
   if (term_signal_ > 0)
-    js_result->Set(context, env()->signal_string(),
-                   String::NewFromUtf8(env()->isolate(),
-                                       signo_string(term_signal_))
-                       .ToLocalChecked())
+    js_result
+        ->Set(context,
+              env()->signal_string(),
+              OneByteString(env()->isolate(), signo_string(term_signal_)))
         .Check();
   else
     js_result->Set(context, env()->signal_string(),
