Merge pull request #913 from throw-out/fix_optional_param_match fix #912

[unity]fix:reflection invoke optional param match
Tencent · Jul 15, 2022 · 7b5ccbb · 7b5ccbb
2 parents a8e8fdc + 9eeed32
commit 7b5ccbb
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/unity/Assets/Puerts/Runtime/Src/Wrapper/ReflectionWrap.cs b/unity/Assets/Puerts/Runtime/Src/Wrapper/ReflectionWrap.cs
@@ -81,7 +81,7 @@ public Parameters(ParameterInfo[] parameterInfos, GeneralGetterManager generalGe
             byRefValueSetFuncs = new GeneralSetter[parameterInfos.Length];
             paramIsByRef = new bool[parameterInfos.Length];
             isOut = new bool[parameterInfos.Length];
-            optionalParamPos = this.paramLength + 1;
+            optionalParamPos = this.paramLength; 
             for (int i = 0; i < parameterInfos.Length; i++)
             {
                 var parameterInfo = parameterInfos[i];
@@ -109,7 +109,7 @@ public Parameters(ParameterInfo[] parameterInfos, GeneralGetterManager generalGe
 
         public bool IsMatch(JSCallInfo jsCallInfo)
         {
-            if (hasParamsArray && optionalParamPos > paramLength)
+            if (hasParamsArray && optionalParamPos >= paramLength)
             {
                 if (jsCallInfo.Length < paramLength - 1)
                 {
@@ -120,7 +120,7 @@ public bool IsMatch(JSCallInfo jsCallInfo)
             {
                 return false;
             }
-            else if (jsCallInfo.Length < optionalParamPos - 1)
+            else if (jsCallInfo.Length < optionalParamPos)
             {
                 return false;
             }