-
Notifications
You must be signed in to change notification settings - Fork 66
Navigation to *.class file (jdt:// scheme) is missing #42
Comments
why isn't the code embedded in this issue? |
Implementation notes:
|
@fbricon looks like we might just be able to wire the protocol handler up to the language server itself for obtaining the file contents? Not sure what "code embedded in this issue" refers to. |
Yes calling java/classFileContents with that jdt uri will return the source, you don't need to worry about opening/traversing the jar. For the embedded code, I'm talking about https://github.com/blog/2415-introducing-embedded-code-snippets. It seems to only work for code from the same repo. |
Read only text editor coming in atom/atom#16294 which will unblock this |
FYI, I plan on disabling sending those jdt:// uris unless clients advertise they support it (see eclipse-jdtls/eclipse.jdt.ls#649). @damieng you might want to keep an eye on that one. |
With jdt.ls 0.19.0 (or more recent), clients need to declare their support for jdt:// urls by sending extendedClientCapabilities.classFileContentsSupport=true in initializationOptions. See eclipse-jdtls/eclipse.jdt.ls#649 (comment) |
jdt.ls sends jdt:// uris for navigating to source code of classes in jar files. In vscode-java a provider is registered for the jdt scheme:
https://github.com/redhat-developer/vscode-java/blob/977973499608ccd5e64115876b250c5dae062023/src/extension.ts#L213-L221
https://github.com/redhat-developer/vscode-java/blob/56c5256f04b2d64208ff5346399cf2e1f94405aa/src/protocol.ts#L69-L71
The text was updated successfully, but these errors were encountered: