Avoid unnecessary array allocation in JsonHelpers.Utf8GetString on ne…

…tstandard (dotnet#92304)
Artromskiy · Sep 20, 2023 · d411f50 · d411f50
1 parent 5883b72
commit d411f50
Showing 1 changed file with 15 additions and 4 deletions.
diff --git a/src/libraries/System.Text.Json/src/System/Text/Json/JsonHelpers.cs b/src/libraries/System.Text.Json/src/System/Text/Json/JsonHelpers.cs
@@ -93,11 +93,22 @@ public static void ReadWithVerify(this ref Utf8JsonReader reader)
         /// <returns></returns>
         public static string Utf8GetString(ReadOnlySpan<byte> bytes)
         {
-            return Encoding.UTF8.GetString(bytes
-#if !NETCOREAPP
-                        .ToArray()
+#if NETCOREAPP
+            return Encoding.UTF8.GetString(bytes);
+#else
+            if (bytes.Length == 0)
+            {
+                return string.Empty;
+            }
+
+            unsafe
+            {
+                fixed (byte* bytesPtr = bytes)
+                {
+                    return Encoding.UTF8.GetString(bytesPtr, bytes.Length);
+                }
+            }
 #endif
-                );
         }
 
         /// <summary>