Use Unsafe.As for better perf

Fixes #2778
mono · Mar 2, 2024 · 9ec7f92 · 9ec7f92
1 parent d98115c
commit 9ec7f92
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 10 deletions.
diff --git a/binding/Directory.Build.targets b/binding/Directory.Build.targets
@@ -4,6 +4,10 @@
     <PackageReference Include="System.Memory" Version="4.5.5" />
   </ItemGroup>
 
+  <ItemGroup Condition=" ('$(TargetFramework)' == 'netstandard2.1') and (!$(MSBuildProjectName.Contains('.NativeAssets.'))) ">
+    <PackageReference Include="System.Runtime.CompilerServices.Unsafe" Version="4.5.3" />
+  </ItemGroup>
+
   <PropertyGroup Condition="'$(PackagingGroup)' == 'SkiaSharp'">
     <PackageDescription Condition="'$(PackageDescription)' == ''">SkiaSharp is a cross-platform 2D graphics API for .NET platforms based on Google's Skia Graphics Library.
 It provides a comprehensive 2D API that can be used across mobile, server and desktop models to render images.</PackageDescription>

diff --git a/binding/SkiaSharp/SKMatrix44.cs b/binding/SkiaSharp/SKMatrix44.cs
@@ -2,6 +2,7 @@
 
 using System;
 using System.Numerics;
+using System.Runtime.CompilerServices;
 
 namespace SkiaSharp
 {
@@ -474,17 +475,9 @@ public static implicit operator SKMatrix44 (SKMatrix matrix) =>
 				matrix.Persp0, matrix.Persp1, 0, matrix.Persp2);
 
 		public static implicit operator Matrix4x4 (SKMatrix44 matrix) =>
-			new Matrix4x4 (
-				matrix.m00, matrix.m10, matrix.m20, matrix.m30,
-				matrix.m01, matrix.m11, matrix.m21, matrix.m31,
-				matrix.m02, matrix.m12, matrix.m22, matrix.m32,
-				matrix.m03, matrix.m13, matrix.m23, matrix.m33);
+			Unsafe.As<SKMatrix44, Matrix4x4> (ref matrix);
 
 		public static implicit operator SKMatrix44 (Matrix4x4 matrix) =>
-			new SKMatrix44 (
-				matrix.M11, matrix.M21, matrix.M31, matrix.M41,
-				matrix.M12, matrix.M22, matrix.M32, matrix.M42,
-				matrix.M13, matrix.M23, matrix.M33, matrix.M43,
-				matrix.M14, matrix.M24, matrix.M34, matrix.M44);
+			Unsafe.As<Matrix4x4, SKMatrix44> (ref matrix);
 	}
 }