Throw error when parsing the method with default parameters in v2.5.0

Hello, I have written a little test, it will pass in v2.4.1 and failed in v2.5.0. But I can not find any breaking change in the release log, Is this a bug or am I missing something? Thank you!

Test case:

private delegate bool GFunction(string arg = null);

[Fact]
public void TestParseGFunction()
{
    static bool GetGFunction(string arg = null)
    {
        return arg is not null;
    };

    var interpreter = new Interpreter();
    interpreter.SetFunction("GFunction", (GFunction) GetGFunction);
    interpreter.SetVariable("arg", "arg");

    Assert.True((bool) interpreter.Eval("GFunction(arg)"));
    Assert.False((bool) interpreter.Eval("GFunction()"));
}

Failed information:

DynamicExpresso.Exceptions.ParseException
Argument list incompatible with delegate expression (at index 0).
   at DynamicExpresso.Parsing.Parser.ParseMethodGroupInvocation(MethodGroupExpression methodGroup, Int32 errorPos)
   at DynamicExpresso.Parsing.Parser.ParseUnary()
   at DynamicExpresso.Parsing.Parser.ParseMultiplicative()
   at DynamicExpresso.Parsing.Parser.ParseAdditive()
   at DynamicExpresso.Parsing.Parser.ParseTypeTesting()
   at DynamicExpresso.Parsing.Parser.ParseComparison()
   at DynamicExpresso.Parsing.Parser.ParseLogicalAnd()
   at DynamicExpresso.Parsing.Parser.ParseLogicalXor()
   at DynamicExpresso.Parsing.Parser.ParseLogicalOr()
   at DynamicExpresso.Parsing.Parser.ParseConditionalAnd()
   at DynamicExpresso.Parsing.Parser.ParseConditionalOr()
   at DynamicExpresso.Parsing.Parser.ParseConditional()
   at DynamicExpresso.Parsing.Parser.ParseAssignment()
   at DynamicExpresso.Parsing.Parser.ParseExpressionSegment()
   at DynamicExpresso.Parsing.Parser.ParseExpressionSegment(Type returnType)
   at DynamicExpresso.Parsing.Parser.Parse()
   at DynamicExpresso.Interpreter.ParseAsLambda(String expressionText, Type expressionType, Parameter[] parameters)
   at DynamicExpresso.Interpreter.Parse(String expressionText, Type expressionType, Parameter[] parameters)
   at DynamicExpresso.Interpreter.Eval(String expressionText, Type expressionType, Parameter[] parameters)
   at DynamicExpresso.Interpreter.Eval(String expressionText, Parameter[] parameters)
   at NetCasbin.UnitTest.UtilityTest.TestParseGFunction() in C:\Users\Sagilio\source\repos\OnWork\Casbin\Casbin.NET\NetCasbin.UnitTest\UtilityTest.cs:line 26

[metoule]

@Sagilio version 2.5.0 includes a fix for #82 to allow the registration of multiple delegates with the same name, to simulate function overloads. To do that, I changed the way delegates are resolved when set via SetFunction. I introduced a regression, for which I apologize: I didn’t realize that the Delegate. Method property and the Invoke method of the delegate type were different.

I fixed the issue via #145.

[metoule]

Yes, that seems like a side effect of my changes. I’ll have a look at it!