-
Notifications
You must be signed in to change notification settings - Fork 408
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
Provide support for Java 21 #2863
Conversation
Please add some unnamed classes/ instance main method test cases. It seems to me completion doesn't work at all in a instance main method example, e.g.: public void main() {
Syste|
} |
the |
well JEP445 is not supported yet eclipse-jdt/eclipse.jdt.core#1106 *sigh* |
Has anyone been able to get String templates ( https://openjdk.org/jeps/430 ) working ? |
All preview features are currently missing eclipse-jdt/eclipse.jdt.core#890 |
Are we ok to merge as-is for releasing by Thursday ? As far as I could tell I was able to configure a Java 21 JVM with this change. |
We could use https://download.eclipse.org/eclipse/updates/4.30-I-builds/ instead of https://download.eclipse.org/eclipse/updates/4.29-P-builds/. See eclipse-jdt/eclipse.jdt.ui#806 |
After discussion, we've decided to hold off on this for after the 1.28.0 JDT-LS release. Once we release we can adopt the 4.30-I-builds. |
@rgrunber I have updated the PR. |
So I've uncovered today that ecj actually supports instance main methods (protected void main() in a class)- aka compiles them fine (no restrictions on that) and they are executed just fine on the console via |
The code to programmatically run/debug isn't handled by JDT-LS. It would be under https://github.com/microsoft/java-debug . The only problem I see is that locating the main class is done through the JDT search engine. I think that if the source parser can't recognize an unnamed classes without throwing syntax errors, then it also won't be able to index that content. It would still rely on language support being adopted. @snjeza It looks like we'll need to manually add commons.codec (required by m2e) to the target platform. It appears to have been removed in https://download.eclipse.org/eclipse/updates/4.30-I-builds/I20230923-1800/plugins/ .
|
@rgrunber |
@rgrunber I have updated the PR. |
test this please |
Thanks for the information. Will they be available on the final release? |
Signed-off-by: Snjezana Peco <[email protected]>
Fixes redhat-developer/vscode-java#3292
Requires redhat-developer/vscode-java#3314
Test vsix - VS Code 1.24.5