Use sequential struct layout in SimplexSolver

notgiven688 · Jan 11, 2025 · a32ffd0 · a32ffd0
1 parent 2b37b15
commit a32ffd0
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/Jitter2/Collision/NarrowPhase/SimplexSolver.cs b/src/Jitter2/Collision/NarrowPhase/SimplexSolver.cs
@@ -23,14 +23,14 @@
 
 using System.Diagnostics;
 using System.Runtime.CompilerServices;
+using System.Runtime.InteropServices;
 using Jitter2.LinearMath;
 
 #pragma warning disable CS0649 // Field is never assigned to, and will always have its default value
 
 namespace Jitter2.Collision;
 
-// This is a GJK-Implementation "by the book".
-
+[StructLayout(LayoutKind.Sequential)]
 public unsafe struct SimplexSolver
 {
     const Real Epsilon = (Real)1e-8;

diff --git a/src/Jitter2/Collision/NarrowPhase/SimplexSolverAB.cs b/src/Jitter2/Collision/NarrowPhase/SimplexSolverAB.cs
@@ -23,6 +23,7 @@
 
 using System.Diagnostics;
 using System.Runtime.CompilerServices;
+using System.Runtime.InteropServices;
 using Jitter2.LinearMath;
 
 using Vertex = Jitter2.Collision.MinkowskiDifference.Vertex;
@@ -31,8 +32,7 @@
 
 namespace Jitter2.Collision;
 
-// This is a GJK-Implementation "by the book".
-
+[StructLayout(LayoutKind.Sequential)]
 public unsafe struct SimplexSolverAB
 {
     const Real Epsilon = (Real)1e-8;