Merge pull request #9 from js6pak/pointers-contains-refs

nike4613 · web-flow · commit e4e31789148e · 2025-10-11T09:15:59.000-05:00
Fix IsReferenceOrContainsReferences for pointers
diff --git a/src/MonoMod.Backports/System/Memory,is_fx,lt_core_2.1,lt_std_2.1/SpanHelpers.Portable.cs b/src/MonoMod.Backports/System/Memory,is_fx,lt_core_2.1,lt_std_2.1/SpanHelpers.Portable.cs
@@ -131,7 +131,7 @@ public static IntPtr Add<T>(this IntPtr start, int index)
 
         private static bool IsReferenceOrContainsReferencesCore(Type type)
         {
-            if (type.GetTypeInfo().IsPrimitive) // This is hopefully the common case. All types that return true for this are value types w/out embedded references.
+            if (type.GetTypeInfo().IsPrimitive || type.GetTypeInfo().IsPointer) // This is hopefully the common case. All types that return true for this are value types w/out embedded references.
                 return false;
 
             if (!type.GetTypeInfo().IsValueType)

Original file line number	Diff line number	Diff line change
`@@ -131,7 +131,7 @@ public static IntPtr Add<T>(this IntPtr start, int index)`
`131`	`131`
`132`	`132`	`private static bool IsReferenceOrContainsReferencesCore(Type type)`
`133`	`133`	`{`
`134`		`- if (type.GetTypeInfo().IsPrimitive) // This is hopefully the common case. All types that return true for this are value types w/out embedded references.`
	`134`	`+ if (type.GetTypeInfo().IsPrimitive \|\| type.GetTypeInfo().IsPointer) // This is hopefully the common case. All types that return true for this are value types w/out embedded references.`
`135`	`135`	`return false;`
`136`	`136`
`137`	`137`	`if (!type.GetTypeInfo().IsValueType)`