-
Notifications
You must be signed in to change notification settings - Fork 116
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
Java 11 support #103
Comments
I am currently looking into making engima compatible with java 1.9 since the buggy dpi scaling in 1.8 makes enigma unuseable on a 4k monitor. |
We can look into it, I know the compiler tools that I host has 1 patch (somewhere on bit bucket irrc) |
Thanks, I would greatly appreciate that. |
Maybe you should just migrate the Procyon repo to here, seeing as it's not really maintained. MORE FORKS \o/ |
I think I have found the version of procyon we use, https://bitbucket.org/cuchaz/procyon/commits/all There are a number of commits from cuchaz that may or may not be relevant still. |
That's actually quite outdated compared to mainline! The only relevant commit for us is https://bitbucket.org/cuchaz/procyon/commits/a8cc9faf9b17bc4de56297c0873d6ab48f106d98 , I'm pretty sure. I'd just fork upstream to FabricMC and cherry-pick this, and hope it works. |
Working on this. |
There is an issue with Unsafe.defineClass being missing in Java 11+ specifically, but I'm pretty sure it does not affect Enigma, merely the standalone decompiler. |
A few things need to be done to support J11
Update the syntax pane library to 1.2.0 to fix Crash log Tested to work
Fix the decompiler using URLClassPath Crash log
Dracula causes warnings, Link to issue
Not the end of the world right now, but might be something to keep in mind. There may be more issues as well.
The text was updated successfully, but these errors were encountered: