-
Notifications
You must be signed in to change notification settings - Fork 886
LSP Server for Java #926
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
LSP Server for Java #926
Conversation
…to java-lsp-server
…s/incubator-netbeans into java-lsp-server
Wow, so much cool stuff here. |
Is the goal implementing an alternative to JDT-LS ? |
@mickaelistria surely the obvious answer is javac?! |
If standard JDK is the key feature, then how is it different from https://github.com/georgewfraser/java-language-server ? |
So, I think this is a good question.
This project is maintaining editor features that support developing in the Java language (and we could ask why, although that is probably out of the scope of this PR). So adding a fairly small layer that exports a subset of these features through the LSP does not feel to be very off. It is simply providing a way for those that prefer a different UI to use Java editor features this project already has. Not that this PR would be quite there.
The NB Java editor has an ability to use JDK's javac for parsing, so it can in some cases support editing of files with Java language features that are newer than the editor itself. And if a new NB feature is added supporting such a new language feature, the LSP server may use it right away. Yes, other server will get that as well, but as long as we maintain the stuff here, why not allow access to it using the LSP as well? This is niche, of course.
|
Until recently that project was also using parts of (pre-Apache) NetBeans' - eg. Personally I'd prefer there was an LS as part of the OpenJDK tools. But having it here seems next best step for an LS backed by javac and building on an editing infrastructure that's widely used. |
…old license by itself)
…y calling the correct superclass method instead of the synthetic bridge.
Should this be merged? |
I am definitely +1 on merging this :-). |
@@ -0,0 +1,18 @@ | |||
This is an extension for VS Code. Based on "lsp-sample" from: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing the license header (Travis CI fails)
OK, merging. |
No description provided.