Ensure that param-array matching works correctly

filmor · filmor · commit 001aeec43374 · 2020-12-10T18:02:56.000+01:00
We can match n Python parameters to exactly n+1 C# parameters of which the last one is a param-array. Fixes #1302.
diff --git a/src/runtime/methodbinder.cs b/src/runtime/methodbinder.cs
@@ -512,7 +512,7 @@ static object[] TryConvertArguments(ParameterInfo[] pi, bool paramsArray,
                 {
                     if(arrayStart == paramIndex)
                     {
-                        op = HandleParamsArray(args, arrayStart, pyArgCount, out isNewReference);                                                                 
+                        op = HandleParamsArray(args, arrayStart, pyArgCount, out isNewReference);
                     }
                     else
                     {
@@ -652,7 +652,7 @@ static bool MatchesArgumentCount(int positionalArgumentCount, ParameterInfo[] pa
             {
                 match = true;
             }
-            else if (positionalArgumentCount < parameters.Length)
+            else if (positionalArgumentCount < parameters.Length && (!paramsArray || positionalArgumentCount == parameters.Length - 1))
             {
                 // every parameter past 'positionalArgumentCount' must have either
                 // a corresponding keyword argument or a default parameter
@@ -677,7 +677,7 @@ static bool MatchesArgumentCount(int positionalArgumentCount, ParameterInfo[] pa
                         defaultArgList.Add(parameters[v].GetDefaultValue());
                         defaultsNeeded++;
                     }
-                    else if(!paramsArray)
+                    else if (!paramsArray)
                     {
                         match = false;
                     }
diff --git a/src/testing/constructortests.cs b/src/testing/constructortests.cs
@@ -48,4 +48,22 @@ public SubclassConstructorTest(Exception v)
             value = v;
         }
     }
+
+    public class MultipleConstructorsTest
+    {
+        public string value;
+        public Type[] type;
+
+        public MultipleConstructorsTest()
+        {
+            value = "";
+            type = new Type[1] { null };
+        }
+
+        public MultipleConstructorsTest(string s, params Type[] tp)
+        {
+            value = s;
+            type = tp;
+        }
+    }
 }
diff --git a/src/tests/test_constructors.py b/src/tests/test_constructors.py
@@ -44,3 +44,12 @@ class Sub(System.Exception):
     instance = Sub()
     ob = SubclassConstructorTest(instance)
     assert isinstance(ob.value, System.Exception)
+
+
+def test_multiple_constructor():
+    from Python.Test import MultipleConstructorsTest
+    import System
+
+    # Test parameterless
+    ob = MultipleConstructorsTest()
+    assert ob.value == ""

Original file line number	Diff line number	Diff line change
`@@ -512,7 +512,7 @@ static object[] TryConvertArguments(ParameterInfo[] pi, bool paramsArray,`
`512`	`512`	`{`
`513`	`513`	`if(arrayStart == paramIndex)`
`514`	`514`	`{`
`515`		`- op = HandleParamsArray(args, arrayStart, pyArgCount, out isNewReference);`
	`515`	`+ op = HandleParamsArray(args, arrayStart, pyArgCount, out isNewReference);`
`516`	`516`	`}`
`517`	`517`	`else`
`518`	`518`	`{`
`@@ -652,7 +652,7 @@ static bool MatchesArgumentCount(int positionalArgumentCount, ParameterInfo[] pa`
`652`	`652`	`{`
`653`	`653`	`match = true;`
`654`	`654`	`}`
`655`		`- else if (positionalArgumentCount < parameters.Length)`
	`655`	`+ else if (positionalArgumentCount < parameters.Length && (!paramsArray \|\| positionalArgumentCount == parameters.Length - 1))`
`656`	`656`	`{`
`657`	`657`	`// every parameter past 'positionalArgumentCount' must have either`
`658`	`658`	`// a corresponding keyword argument or a default parameter`
`@@ -677,7 +677,7 @@ static bool MatchesArgumentCount(int positionalArgumentCount, ParameterInfo[] pa`
`677`	`677`	`defaultArgList.Add(parameters[v].GetDefaultValue());`
`678`	`678`	`defaultsNeeded++;`
`679`	`679`	`}`
`680`		`- else if(!paramsArray)`
	`680`	`+ else if (!paramsArray)`
`681`	`681`	`{`
`682`	`682`	`match = false;`
`683`	`683`	`}`