Don't use .copy, it's evil and mean

PassiveModding · Oct 3, 2024 · c3190b1 · c3190b1
1 parent ee8745f
commit c3190b1
Showing 1 changed file with 12 additions and 3 deletions.
diff --git a/Meddle/Meddle.Utils/Helpers/ImageUtils.cs b/Meddle/Meddle.Utils/Helpers/ImageUtils.cs
@@ -242,9 +242,18 @@ public static unsafe SKBitmap ToBitmap(this TextureResource resource)
         {
             bitmap.InstallPixels(bitmap.Info, (nint)data, rgba.Meta.Width * 4);
         }
-        bitmap.SetImmutable();
-
-        return bitmap.Copy() ?? throw new InvalidOperationException("Failed to copy bitmap.");
+        // bitmap.SetImmutable();
+        //
+        // return bitmap.Copy() ?? throw new InvalidOperationException("Failed to copy bitmap.");
+        var copy = new SKBitmap(rgba.Meta.Width, rgba.Meta.Height, SKColorType.Rgba8888, SKAlphaType.Unpremul);
+        var pixelBuf = copy.GetPixelSpan().ToArray();
+        bitmap.GetPixelSpan().CopyTo(pixelBuf);
+        fixed (byte* data = pixelBuf)
+        {
+            copy.InstallPixels(copy.Info, (nint)data, copy.Info.RowBytes);
+        }
+
+        return copy;
     }
 
     public static TexMeta FromResource(TextureResource resource)