LlamaLad7 / MixinExtras

Companion library to SpongePowered Mixin with many custom injectors for a more expressive experience.
MIT License
304 stars 17 forks source link

Deliberate conflict marker #90

Open Earthcomputer opened 1 month ago

Earthcomputer commented 1 month ago

Add an annotation (say, @Conflict), that can be added to an injector to mark it as conflicting with other injectors that target the same instruction. This tells MixinExtras to fail the injection in this case.

This has the following advantages:

LlamaLad7 commented 1 month ago

Would it cause conflicts only with other injectors marked as Conflicting?

Earthcomputer commented 1 month ago

My idea was to conflict with any injector which targets the same instruction, regardless of what that other injector is or how it's annotated. But more complicated options could certainly be added to the annotation