Skip delegate types with in, out or ref parameters

lemonmojo · lemonmojo · commit 97d3f8d388d9 · 2025-10-07T15:40:40.000+02:00
Fixes #110
diff --git a/Generator/Beyond.NET.CodeGenerator/Collectors/TypeCollector.cs b/Generator/Beyond.NET.CodeGenerator/Collectors/TypeCollector.cs
@@ -499,6 +499,25 @@ out string? unsupportedReason
                         unsupportedReason = "Unsupported delegate paramter type";
                         return false;
                     }
+
+                    if (delegateParameter.IsOut) {
+                        unsupportedReason = "Unsupported delegate paramter type: Has out parameters";
+                        return false;
+                    }
+
+                    if (delegateParameter.IsIn) {
+                        unsupportedReason = "Unsupported delegate paramter type: Has in parameters";
+                        return false;
+                    }
+
+                    if (!ExperimentalFeatureFlags.EnableByRefParametersInDelegates) {
+                        Type parameterType = delegateParameter.ParameterType;
+
+                        if (parameterType.IsByRef) {
+                            unsupportedReason = "Unsupported delegate paramter type: Has ref parameters";
+                            return false;
+                        }
+                    }
                 }
             }
         }
@@ -510,4 +529,4 @@ out string? unsupportedReason
 
         return true;
     }
-}
+}
diff --git a/Samples/Beyond.NET.Sample.Managed/Source/DelegatesTest.cs b/Samples/Beyond.NET.Sample.Managed/Source/DelegatesTest.cs
@@ -27,11 +27,12 @@ public enum StepMode
     Out,
 }
 
-// TODO: Delegates with ref parameters
+// TODO: Delegates with ref, out, in parameters
 public static class DelegatesTest
 {
     public delegate int TransformIntDelegate(int i);
-    // public delegate int TransformIntWithRefDelegate(ref int iRef);
+    public delegate int TransformIntWithRefDelegate(ref int iRef);
+    public delegate void TransformIntWithOutDelegate(int i, out int iOut);
     public delegate StepMode DebugEventHandler(object sender, StepMode stepMode);
 
     public static int TransformInt(
@@ -44,18 +45,26 @@ TransformIntDelegate intTransformer
         return result;
     }
 
-    // public static int TransformIntWithRef(
-    //     ref int iRef,
-    //     TransformIntWithRefDelegate intTransformer
-    // )
-    // {
-    //     int result = intTransformer(ref iRef);
-    //
-    //     return result;
-    // }
+    public static int TransformIntWithRef(
+        ref int iRef,
+        TransformIntWithRefDelegate intTransformer
+    )
+    {
+        int result = intTransformer(ref iRef);
+
+        return result;
+    }
+
+    public static void TransformIntWithOut(
+        int i,
+        TransformIntWithOutDelegate intTransformer
+    )
+    {
+        intTransformer(i, out i);
+    }
 
     public delegate Point PointTransformDelegate(Point point);
-    // public delegate Point PointTransformWithRefDelegate(ref Point pointRef);
+    public delegate Point PointTransformWithRefDelegate(ref Point pointRef);
 
     public static Point TransformPoint(
         Point point,
@@ -67,18 +76,18 @@ PointTransformDelegate pointTransformer
         return result;
     }
 
-    // public static Point TransformRefPoint(
-    //     ref Point pointRef,
-    //     PointTransformWithRefDelegate pointTransformer
-    // )
-    // {
-    //     Point result = pointTransformer(ref pointRef);
-    //
-    //     return result;
-    // }
+    public static Point TransformRefPoint(
+        ref Point pointRef,
+        PointTransformWithRefDelegate pointTransformer
+    )
+    {
+        Point result = pointTransformer(ref pointRef);
+
+        return result;
+    }
 
     public static StepMode TransformStepMode(StepMode mode)
     {
         return mode;
     }
-}
+}
diff --git a/Samples/Generated/Generated_C.h b/Samples/Generated/Generated_C.h
@@ -1,4 +1,4 @@
-// Number of generated types: 1638
+// Number of generated types: 1637
 // Number of generated members: 7782
 
 #pragma mark - BEGIN Header
@@ -2357,6 +2357,9 @@ typedef struct DNReadOnlySpanOfByte {
 // Unsupported Type "System.Nullable`1[[System.Int32, System.Private.CoreLib, Version=10.0.0.0, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]][]": Is Array of Generic Type
 // Unsupported Type "System.Nullable`1[[System.Guid, System.Private.CoreLib, Version=10.0.0.0, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]][]": Is Array of Generic Type
 // Unsupported Type "Beyond.NET.Sample.IAnimal[]": Is Array of Interface Type
+// Unsupported Type "Beyond.NET.Sample.DelegatesTest+TransformIntWithRefDelegate": Unsupported delegate paramter type: Has ref parameters
+// Unsupported Type "Beyond.NET.Sample.DelegatesTest+TransformIntWithOutDelegate": Unsupported delegate paramter type: Has out parameters
+// Unsupported Type "Beyond.NET.Sample.DelegatesTest+PointTransformWithRefDelegate": Unsupported delegate paramter type: Has ref parameters
 // Unsupported Type "Converter`2": Is Generic Delegate Type
 // Unsupported Type "System.Predicate`1[[System.String, System.Private.CoreLib, Version=10.0.0.0, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]]": Is Generic Delegate Type
 // Unsupported Type "System.Action`1[[System.String, System.Private.CoreLib, Version=10.0.0.0, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]]": Is Generic Delegate Type
@@ -2387,6 +2390,8 @@ typedef struct DNReadOnlySpanOfByte {
 // Unsupported Type "Beyond.NET.Sample.INotImplementedInterfaceWithStaticAbstractMembers": Static abstract members in interface
 // Unsupported Type "System.Span`1[[System.Byte, System.Private.CoreLib, Version=10.0.0.0, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]]&": Is unsupported Type
 // Unsupported Type "System.Nullable`1": Excluded
+// Unsupported Type "Beyond.NET.Sample.ByRefParametersDelegate": Unsupported delegate paramter type: Has ref parameters
+// Unsupported Type "Beyond.NET.Sample.OutParametersDelegate": Unsupported delegate paramter type: Has out parameters
 // Unsupported Type "System.ArraySegment`1[[System.Byte, System.Private.CoreLib, Version=10.0.0.0, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]]&": Is unsupported Type
 // Unsupported Type "System.Func`1[[System.Object, System.Private.CoreLib, Version=10.0.0.0, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]]": Is Generic Delegate Type
 // Unsupported Type "System.Func`2[[System.Object, System.Private.CoreLib, Version=10.0.0.0, Culture=neutral, PublicKeyToken=7cec85d7bea7798e],[System.Object, System.Private.CoreLib, Version=10.0.0.0, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]]": Is Generic Delegate Type
@@ -5594,6 +5599,7 @@ typedef void* Beyond_NET_Sample_Point_t;
 
 typedef void* Beyond_NET_Sample_DelegatesTest_t;
 
+
 typedef void* Beyond_NET_Sample_EventTests_t;
 
 typedef void* Beyond_NET_Sample_GenericTestClass_A1_t;
@@ -6429,10 +6435,6 @@ typedef System_Byte_Array_t _Nullable (*Beyond_NET_Sample_SpanTest_SpanToByteArr
 
 // TODO: (Beyond_NET_Sample_ByRefReturnValueDelegate) Unsupported delegate type. Reason: Has by ref return type
 
-// TODO: (Beyond_NET_Sample_ByRefParametersDelegate) Unsupported delegate type. Reason: Has by ref parameters
-
-// TODO: (Beyond_NET_Sample_OutParametersDelegate) Unsupported delegate type. Reason: Has out parameters
-
 typedef void* Beyond_NET_Sample_CharReturnerDelegate_t;
 typedef void (*Beyond_NET_Sample_CharReturnerDelegate_CDestructorFunction_t)(void* context);
 
@@ -45280,6 +45282,7 @@ Beyond_NET_Sample_DelegatesTest_Destroy(
 
 #pragma mark - END APIs of Beyond.NET.Sample.DelegatesTest
 
+
 #pragma mark - BEGIN APIs of Beyond.NET.Sample.EventTests
 Beyond_NET_Sample_EventTests_t _Nonnull /* Beyond.NET.Sample.EventTests */
 Beyond_NET_Sample_EventTests_Create(
@@ -63103,14 +63106,6 @@ Beyond_NET_Sample_SpanTest_SpanToByteArrayDelegate_Destroy(
 // TODO: (Beyond_NET_Sample_ByRefReturnValueDelegate_t) Unsupported delegate type. Reason: Has by ref return type
 #pragma mark - END APIs of Beyond.NET.Sample.ByRefReturnValueDelegate
 
-#pragma mark - BEGIN APIs of Beyond.NET.Sample.ByRefParametersDelegate
-// TODO: (Beyond_NET_Sample_ByRefParametersDelegate_t) Unsupported delegate type. Reason: Has by ref parameters
-#pragma mark - END APIs of Beyond.NET.Sample.ByRefParametersDelegate
-
-#pragma mark - BEGIN APIs of Beyond.NET.Sample.OutParametersDelegate
-// TODO: (Beyond_NET_Sample_OutParametersDelegate_t) Unsupported delegate type. Reason: Has out parameters
-#pragma mark - END APIs of Beyond.NET.Sample.OutParametersDelegate
-
 #pragma mark - BEGIN APIs of Beyond.NET.Sample.CharReturnerDelegate
 Beyond_NET_Sample_CharReturnerDelegate_t _Nonnull /* Beyond.NET.Sample.CharReturnerDelegate */
 Beyond_NET_Sample_CharReturnerDelegate_Create(
diff --git a/Samples/Generated/Generated_CS.cs b/Samples/Generated/Generated_CS.cs
@@ -1,6 +1,6 @@
 // <auto-generated />
 
-// Number of generated types: 1638
+// Number of generated types: 1637
 // Number of generated members: 7782
 
 // <Header>
@@ -3177,6 +3177,9 @@ internal static DNReadOnlySpanOfByte CopyToNativeReadOnlySpanOfByte(this ReadOnl
 // Unsupported Type "System.Nullable`1[[System.Int32, System.Private.CoreLib, Version=10.0.0.0, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]][]": Is Array of Generic Type
 // Unsupported Type "System.Nullable`1[[System.Guid, System.Private.CoreLib, Version=10.0.0.0, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]][]": Is Array of Generic Type
 // Unsupported Type "Beyond.NET.Sample.IAnimal[]": Is Array of Interface Type
+// Unsupported Type "Beyond.NET.Sample.DelegatesTest+TransformIntWithRefDelegate": Unsupported delegate paramter type: Has ref parameters
+// Unsupported Type "Beyond.NET.Sample.DelegatesTest+TransformIntWithOutDelegate": Unsupported delegate paramter type: Has out parameters
+// Unsupported Type "Beyond.NET.Sample.DelegatesTest+PointTransformWithRefDelegate": Unsupported delegate paramter type: Has ref parameters
 // Unsupported Type "Converter`2": Is Generic Delegate Type
 // Unsupported Type "System.Predicate`1[[System.String, System.Private.CoreLib, Version=10.0.0.0, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]]": Is Generic Delegate Type
 // Unsupported Type "System.Action`1[[System.String, System.Private.CoreLib, Version=10.0.0.0, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]]": Is Generic Delegate Type
@@ -3207,6 +3210,8 @@ internal static DNReadOnlySpanOfByte CopyToNativeReadOnlySpanOfByte(this ReadOnl
 // Unsupported Type "Beyond.NET.Sample.INotImplementedInterfaceWithStaticAbstractMembers": Static abstract members in interface
 // Unsupported Type "System.Span`1[[System.Byte, System.Private.CoreLib, Version=10.0.0.0, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]]&": Is unsupported Type
 // Unsupported Type "System.Nullable`1": Excluded
+// Unsupported Type "Beyond.NET.Sample.ByRefParametersDelegate": Unsupported delegate paramter type: Has ref parameters
+// Unsupported Type "Beyond.NET.Sample.OutParametersDelegate": Unsupported delegate paramter type: Has out parameters
 // Unsupported Type "System.ArraySegment`1[[System.Byte, System.Private.CoreLib, Version=10.0.0.0, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]]&": Is unsupported Type
 // Unsupported Type "System.Func`1[[System.Object, System.Private.CoreLib, Version=10.0.0.0, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]]": Is Generic Delegate Type
 // Unsupported Type "System.Func`2[[System.Object, System.Private.CoreLib, Version=10.0.0.0, Culture=neutral, PublicKeyToken=7cec85d7bea7798e],[System.Object, System.Private.CoreLib, Version=10.0.0.0, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]]": Is Generic Delegate Type
@@ -159307,12 +159312,24 @@ internal unsafe class Beyond_NET_Sample_StepMode
 
 internal unsafe class Beyond_NET_Sample_DelegatesTest
 {
+	// Unsupported Member "TransformIntWithRef": Has unsupported parameter(s): intTransformer: Unsupported delegate paramter type: Has ref parameters; 
+
+	// Unsupported Member "TransformIntWithOut": Has unsupported parameter(s): intTransformer: Unsupported delegate paramter type: Has out parameters; 
+
+	// Unsupported Member "TransformRefPoint": Has unsupported parameter(s): pointTransformer: Unsupported delegate paramter type: Has ref parameters; 
+
 	// Unsupported Member "TransformIntDelegate": Unsupported member type: NestedType
 
+	// Unsupported Member "TransformIntWithRefDelegate": Unsupported member type: NestedType
+
+	// Unsupported Member "TransformIntWithOutDelegate": Unsupported member type: NestedType
+
 	// Unsupported Member "DebugEventHandler": Unsupported member type: NestedType
 
 	// Unsupported Member "PointTransformDelegate": Unsupported member type: NestedType
 
+	// Unsupported Member "PointTransformWithRefDelegate": Unsupported member type: NestedType
+
 	[UnmanagedCallersOnly(EntryPoint = "Beyond_NET_Sample_DelegatesTest_TransformInt")]
 	internal static int /* System.Int32 */ Beyond_NET_Sample_DelegatesTest_TransformInt(int /* System.Int32 */ i, void* /* Beyond.NET.Sample.DelegatesTest.TransformIntDelegate */ intTransformer, void** /* System.Exception */ __outException)
 	{
@@ -159748,6 +159765,7 @@ private Beyond.NET.Sample.Point __InvokeByCallingCFunction(Beyond.NET.Sample.Poi
 }
 
 
+
 internal unsafe class Beyond_NET_Sample_EventTests
 {
 	// Unsupported Member "add_ValueChanged": Is Special Name
@@ -169884,18 +169902,6 @@ internal unsafe class Beyond_NET_Sample_ByRefReturnValueDelegate
 }
 
 
-internal unsafe class Beyond_NET_Sample_ByRefParametersDelegate
-{
-	// TODO: Unsupported delegate type. Reason: Has by ref parameters
-}
-
-
-internal unsafe class Beyond_NET_Sample_OutParametersDelegate
-{
-	// TODO: Unsupported delegate type. Reason: Has out parameters
-}
-
-
 internal unsafe class Beyond_NET_Sample_CharReturnerDelegate
 {
 	internal void* Context { get; }
diff --git a/Samples/Generated/Generated_Kotlin.kt b/Samples/Generated/Generated_Kotlin.kt
diff --git a/Samples/Generated/Generated_Swift.swift b/Samples/Generated/Generated_Swift.swift
