MinecraftDev icon indicating copy to clipboard operation
MinecraftDev copied to clipboard

Add mixin helper support for MixinExtras

Open shaksternano opened this issue 3 years ago • 3 comments
trafficstars

Would it be possible to add stuff like mixin target autocomplete and the go to target button for MixinExtras? https://github.com/LlamaLad7/MixinExtras

shaksternano avatar Jan 19 '22 15:01 shaksternano

I have discussed this with LlamaLad on discord, and for now I think this might be best as a separate plugin on top of mcdev which uses mcdev's mixin API, which should allow for this quite easily. MixinExtras is still quite new but if it gets popular then we may decide (with permission) to merge that separate plugin into mcdev.

Earthcomputer avatar Jan 19 '22 16:01 Earthcomputer

#1882 will partly address this by trying to detect whether an annotation is a mixin injector, which will facilitate completion and navigation for injectors that don't have proper support. However there will be no unresolved errors, as we can't be sure that a custom injector doesn't have custom resolution logic. To get full support for MixinExtras, Annotation handlers need to be written for these annotations as described above.

Earthcomputer avatar Aug 09 '22 19:08 Earthcomputer

Hi, seeing as this is an issue from 2022 I don't think there'll be much activity. But I'll just ask anyway. Is it possible to implement this? I think MixinExtras has gotten to a point where it is worth it to implement. Stuff like WrapOperation and WrapWithCondition shouldn't be too hard to implement, as they're very similar to Redirect calls. If this already exists I hope someone can send it. Please take it into consideration. Thanks :)

faluhub avatar Sep 15 '23 10:09 faluhub