-
Notifications
You must be signed in to change notification settings - Fork 392
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
Incorrect completion in some cases #2626
Comments
I think I see your point though. Why has it been working for VS Code all this time ? @jdneo ? I have something like :
|
I imagine VSCode has some tweaks to "fix" such strange cases, intentionally or not. We had the same thing in LSP4E some time ago, with the behavior that was in place to better handle some lowly specified behavior was kept because "why not?", and removing it highlighted issues in some LSs. A similar story can have happened to VSCode. |
In eclipseide-jdtls, I'm trying to complete
syser
tosyserr
proposal, but insertion seems to leave one character behind, iesyser
is replaced withSystem.err.println();r
.Here is an example of client <-> server communication
The interesting part is the LS response:
"textEdit":{"range":{"start":{"line":7,"character":3},"end":{"line":7,"character":7}}
does define to replacesyse
range (since theend
is excluded) and thus leaves the trailingr
in the document.The text was updated successfully, but these errors were encountered: