-
Notifications
You must be signed in to change notification settings - Fork 34
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
Remove mixin hotswapping tutorial #108
Comments
It's a fairly commonly used feature, back when mixinextras caused a few issues with it I had many requests for it to be fixed. What JVMs doesn't it work on exactly? I've never had any issues with various Hotspot JVMs and almost no one uses OpenJ9. |
I wasnt aware of that, if thats the case then making it easier to use should be a higher priortiy. I have personally never used it before :D
Ss far as I know it doenst work on DCEVM/JBR. I would be happy to be wrong though. |
It's incredibly useful for debugging and when set up correctly doesn't fail or cause any issues. I will test on JBR |
Doesn't seem to work with |
Yeah I'm gonna say no on this, Mixin hotswapping is extremely useful and easier to set up than DCEVM in the view of new mod developers. |
I can confirm, I use hotswaps on mixins pretty often too. |
It works for me (I'm currently on JBR 21.0.3b465.3), though the added capabilities don't work for mixin classes. |
I'll close this then, will still look to improve in loom. |
This is an advanced feature that is not supported by all JVMs and is prone to error. I do plan on adding an option to loom to solve some of these issues. I dont think we should suggest for all users to set this when they are unlikely to ever use it.
The text was updated successfully, but these errors were encountered: