Repurposed Structures (Neoforge/Forge)

Repurposed Structures (Neoforge/Forge)

58M Downloads

Mixin conflict with Simply Improved Terrain

SmittyTheGod opened this issue · 5 comments

commented

There is a conflict between the two

commented

You’ll have to be more specific. Can you give me the latest.log file after getting conflict? What Mc version? Modloader?

commented

Yeah, sorry about that. I got help on the curseforge discord and was told to report this. 1.20.1 forge
latest.log

commented

Good find.

The reason I had used @Overwrite was because an @Inject performing the same task would bar the kind of interplay you're after entirely. That is, an @Inject that calls a modified copy of the code then cancels by setting the return value.

I would think it would work as-is, since the four calls to setBlock() are in the same order and perform the same duties. My goal was to preserve compatibility as much as possible by leaving as much of the code in the same place as possible, so I'm curious why it's not working as expected. it must be the local variable capture.

In any case, I think I can change it to a few targeted mixins with a bit of finagling.

commented

I raised my mixin's priority and got around the overwrite. Overwrite mixins, if done, should be lower priority as an overwrite and inject at same priority will explode. Update RS and it'll work with Simply Improved Terrain right now