Instance initializer is never called about mixin HOT 9 CLOSED

Initialisers in mixins have always been ignored. You must inject into the initialiser in the target class in order to call construct-time logic.

from mixin.

Except that when I try that, I get told that injecting is not allowed with multi-target mixins

from mixin.

It should be allowed if you set remap=false on the @Inject annotation. The reason it's not allowed on remapped injectors is that a single name in multiple classes which are obfuscated will obviously remap to multiple names in each target class so I can't squeeze it through the obfuscator. But <init> always has the same name and so you can set remap=false on it and it should be fine.

If that doesn't work let me know and I'll check the AP logic.

from mixin.

That sorta works -- now I'm getting an issue with the fact that the different targets take different constructor arguments:

Caused by: org.spongepowered.asm.mixin.injection.InvalidInjectionException: Invalid descriptor on callback: expected (Ljava/io/File;Ljava/net/Proxy;Ljava/io/File;Lorg/spongepowered/asm/mixin/injection/callback/CallbackInfo;)V but found (Lorg/spongepowered/asm/mixin/injection/callback/CallbackInfo;)V
    at org.spongep

(the output was cut off there in the terminal, not mispaste)

from mixin.

Hmm. Well if you don't actually need the actual arguments then I could make it so that injectors can be created which don't take the method args in their signature. That should let you inject with impugnity.

from mixin.

yeah, I don't care about the arguments so that sounds like what I'd need.

from mixin.

Ok I'll get right on that.

from mixin.

Ok, as of the commit above you can now make injector callbacks with only the CallbackInfo argument. You will need to update the mixin library version to 0.4.3 in spongecommon to get this change though.

from mixin.

Great, that seems to work. Thanks!

from mixin.