Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add inspection for the "Factory @Invoker must be static" mixin error #2346

Open
O7410 opened this issue Aug 1, 2024 · 0 comments
Open

Add inspection for the "Factory @Invoker must be static" mixin error #2346

O7410 opened this issue Aug 1, 2024 · 0 comments

Comments

@O7410
Copy link
Contributor

O7410 commented Aug 1, 2024

Minecraft Development for IntelliJ plugin version

2024.1-1.8.0

Description of the feature request

An inspection with the same message and a quick fix to make the method static
The error message gets triggered in mixin is these methods:
The method that decides the accessor type: AnnotatedMixinElementHandlerAccessor#attach
The method that triggers the error if it's a non static factory: AnnotatedMixinElementHandlerAccessor#registerFactoryForTarget
This is low priority since it's very easy to see what the problem is when you try to run it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants