-
Notifications
You must be signed in to change notification settings - Fork 373
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
Smart semicolon autocorrect #2710
Conversation
org.eclipse.jdt.ls.core/src/org/eclipse/jdt/ls/core/internal/handlers/SmartDetectionParams.java
Outdated
Show resolved
Hide resolved
d740d72
to
6202173
Compare
So the Java partitioning gets us better detection of where we are in the code ? I'd be fine with this. These classes for the most part are easy to move, if not for the fact that some are API 😐 , and would also involve jface.text -> o.e.text . |
Right. It is also faster. |
0fada23
to
850ac22
Compare
Signed-off-by: Snjezana Peco <snjezana.peco@redhat.com>
Signed-off-by: Snjezana Peco <snjezana.peco@redhat.com>
@rgrunber I have updated the PR. See redhat-developer/vscode-java#3159 (comment) |
Fixes redhat-developer/vscode-java#703
Requires redhat-developer/vscode-java#3159