API: Consider Replacing Marshal.Read & Array Marshalling

[Sewer56]

Consider refactoring usage of Marshal.Read* and Marshal.Write* with either raw pointers or Unsafe.Read & Unsafe.Write.
For legacy reasons, the Marshal classes seem to include exception handling for bad addresses; and in .NET the inclusion of exception handling will prevent your code from being inlined.

SharpLab sample to demonstrate.

---

SpiralNeo/Info.SpiralFramework.Neo/HookDelegates.cs

Line 15 in 75a1c55

public delegate int ReadFile(IntPtr[] resultStructure, string path);

Marshalling arrays causes unnecessary heap allocations, consider using raw pointer + count, and then creating Span<T>.

[UnderMybrella]

Ahh, ty for that. I'm still getting familiar with C# so thanks for the improvement!