Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Extensions.cs #294

Merged
merged 1 commit into from
Oct 22, 2019
Merged

Update Extensions.cs #294

merged 1 commit into from
Oct 22, 2019

Conversation

jiogao
Copy link
Contributor

@jiogao jiogao commented Oct 4, 2019

avoid the exception about generic method that same method name and different lenght of generic arguments.

avoid the exception about generic method that same method name and different lenght of generic arguments.
@liiir1985
Copy link
Collaborator

Do you have test case for this fix?

@jiogao
Copy link
Contributor Author

jiogao commented Oct 11, 2019

public class TestClass
{
public K Func<T, K>(T arg)
{
return default;
}
public void Func(T arg)
{
}
public void Func<T, K>(T arg0, K arg1)
{
}
}

//in hotfix dll
public static class ILRHotfixInit
{
public static void Init()
{
TestClass o = new TestClass();
o.Func("");
var res = o.Func<int, float>(1);
}
}

生成CLR代码之后就会报错
CLRBindings.Initialize(appdomain);
昨天传了一个可以测试的unity工程在群里了 :)

@liiir1985 liiir1985 merged commit 972f007 into Ourpalm:master Oct 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants