René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Exception thrown when calling a parameterless constructor that has overload with at least one parameter and `params`","articleBody":"### Environment\r\n\r\n-   Pythonnet version: master@c81c3c3db7fcc403d5f4e8298b320240829daab2\r\n-   Python version: 3.7, 3.9 (both x64)\r\n-   Operating System: Windows\r\n\r\n### Details\r\nThis is a regression introduced in c81c3c3db7fcc403d5f4e8298b320240829daab2 .\r\n\r\n-   If there is a parameterless constructor (and probably a method too) that has an overload with at least one parameter  `params` parameter, when called from Python, an InvalidIndexError will be raised from Python for .NET.\r\n\r\nCallstack from when called from within Unity\r\n```\r\nSystem.ArgumentOutOfRangeException : Index was out of range. Must be non-negative and less than the size of the collection.\r\nParameter name: index\r\n---\r\nat System.Collections.ArrayList.get_Item (System.Int32 index) [0x0001c] in \u003c437ba245d8404784b9fbab9b439ac908\u003e:0 \r\n  at Python.Runtime.MethodBinder.TryConvertArguments (System.Reflection.ParameterInfo[] pi, System.Boolean paramsArray, System.IntPtr args, System.Int32 pyArgCount, System.Collections.Generic.Dictionary`2[TKey,TValue] kwargDict, System.Collections.ArrayList defaultArgList, System.Boolean needsResolution, System.Int32\u0026 outs) [0x0008a] in \u003cd0b807c6e27c43ea84a0e3a3c643e888\u003e:0 \r\n  at Python.Runtime.MethodBinder.Bind (System.IntPtr inst, System.IntPtr args, System.IntPtr kw, System.Reflection.MethodBase info, System.Reflection.MethodInfo[] methodinfo) [0x0017c] in \u003cd0b807c6e27c43ea84a0e3a3c643e888\u003e:0 \r\n  at Python.Runtime.MethodBinder.Bind (System.IntPtr inst, System.IntPtr args, System.IntPtr kw, System.Reflection.MethodBase info) [0x00000] in \u003cd0b807c6e27c43ea84a0e3a3c643e888\u003e:0 \r\n  at Python.Runtime.ConstructorBinder.InvokeRaw (System.IntPtr inst, System.IntPtr args, System.IntPtr kw, System.Reflection.MethodBase info) [0x00074] in \u003cd0b807c6e27c43ea84a0e3a3c643e888\u003e:0 \r\n  at Python.Runtime.ConstructorBinder.InvokeRaw (System.IntPtr inst, System.IntPtr args, System.IntPtr kw) [0x00000] in \u003cd0b807c6e27c43ea84a0e3a3c643e888\u003e:0 \r\n  at Python.Runtime.ClassObject.tp_new (System.IntPtr tp, System.IntPtr args, System.IntPtr kw) [0x000b5] in \u003cd0b807c6e27c43ea84a0e3a3c643e888\u003e:0 \r\n  at Python.Runtime.NativeCall.Call_3 (System.IntPtr fp, System.IntPtr a1, System.IntPtr a2, System.IntPtr a3) [0x00006] in \u003cd0b807c6e27c43ea84a0e3a3c643e888\u003e:0 \r\n  at Python.Runtime.MetaType.tp_call (System.IntPtr tp, System.IntPtr args, System.IntPtr kw) [0x00024] in \u003cd0b807c6e27c43ea84a0e3a3c643e888\u003e:0 \r\n  at (wrapper managed-to-native) Python.Runtime.Runtime.PyRun_String(string,Python.Runtime.RunFlagType,intptr,intptr)\r\n```\r\n\r\nI have a patch to show how to reproduce it (although it crashes pytest)\r\n```\r\ndiff --git a/src/testing/constructortests.cs b/src/testing/constructortests.cs\r\nindex 8800c94..ecc61ce 100644\r\n--- a/src/testing/constructortests.cs\r\n+++ b/src/testing/constructortests.cs\r\n@@ -48,4 +48,22 @@ namespace Python.Test\r\n             value = v;\r\n         }\r\n     }\r\n+\r\n+    public class MultipleConstructorsTest\r\n+    {\r\n+        public string value;\r\n+        public Type[] type;\r\n+\r\n+        public MultipleConstructorsTest()\r\n+        {\r\n+            value = \"\";\r\n+            type = new Type[1] { null };\r\n+        }\r\n+\r\n+        public MultipleConstructorsTest(string s, params Type[] tp)\r\n+        {\r\n+            value = s;\r\n+            type = tp;\r\n+        }\r\n+    }\r\n }\r\ndiff --git a/src/tests/test_constructors.py b/src/tests/test_constructors.py\r\nindex 5e54896..c305377 100644\r\n--- a/src/tests/test_constructors.py\r\n+++ b/src/tests/test_constructors.py\r\n@@ -44,3 +44,12 @@ def test_subclass_constructor():\r\n     instance = Sub()\r\n     ob = SubclassConstructorTest(instance)\r\n     assert isinstance(ob.value, System.Exception)\r\n+\r\n+\r\n+def test_multiple_constructor():\r\n+    from Python.Test import MultipleConstructorsTest\r\n+    import System\r\n+\r\n+    # Test parameterless\r\n+    ob = MultipleConstructorsTest()\r\n+    assert ob.value == \"\"\r\n\r\n```\r\n","author":{"url":"https://github.com/BadSingleton","@type":"Person","name":"BadSingleton"},"datePublished":"2020-12-03T19:39:52.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":0},"url":"https://github.com/1302/pythonnet/issues/1302"}