Allow static method injection

[limoka]

This PR enables static methods to be injected. This mostly allows to pass such methods to il2cpp delegates, which can be useful in some cases.

My main use of this is for interacting with Burst compiler. It expects us to pass delegates pointing to valid il2cpp methods to consider them for Burst compilation.

I have tested these changes on Core Keeper Unity 2021.3.14.

[js6pak]

I have tested these changes

Have you? As is the PR had no chance of working at all.

[js6pak]

What's this check for?

[limoka]

I don't remember exactly, but best guess would be that somewhere there was a static method with one of the parameters being a generic type, and likely that was causing issues.

[js6pak]

My test code:

public class Test : Il2CppSystem.Object
{
    public Test() : base(ClassInjector.DerivedConstructorPointer<Test>())
    {
        ClassInjector.DerivedConstructorBody(this);
    }

    public void InstanceFoo(int a, int b)
    {
        System.Console.WriteLine($"InstanceFoo {a} {b}");
    }

    public static void StaticFoo(int a, int b)
    {
        System.Console.WriteLine($"StaticFoo {a} {b}");
    }
}

ClassInjector.RegisterTypeInIl2Cpp<Test>();
var type = Il2CppType.Of<Test>();
var methods = type.GetMethods(BindingFlags.DeclaredOnly | BindingFlags.Public | BindingFlags.NonPublic | BindingFlags.Instance | BindingFlags.Static);
foreach (var methodInfo in methods)
{
    Log.LogWarning(methodInfo.Name);
    if (methodInfo.Name.EndsWith("Foo"))
    {
        var o = methodInfo.IsStatic ? null : new Test();
        methodInfo.Invoke(o, new Il2CppReferenceArray<Object>([1, 2]));
    }
}

[ds5678]

@limoka can you rebase?

[limoka]

@ds5678 rebased as you asked. Sorry it took a while, I was busy with IRL things.

[ds5678]

Why does it matter here if the method is static or not?

[limoka]

I mean Il2cpp Interop supports normal methods that are generic. If I don't include static part, those won't work.

This check is here so that only non generic static methods are included.
Most likely I didn't want to bother supporting generic static methods.

[ds5678]

Do we support generic instance methods?

[ds5678]

On the surface, this looks like it will break instance methods. However, I saw this elsewhere:

nativeParameterTypes.Add(typeof(Il2CppMethodInfo*));

so I think this line is actually fine.

[ds5678]

Do we support generic instance methods?

[ds5678]

Why is this needed?