Skip to content

Java's eclipse-jdt-ls does not offer suggestions after a dot #725

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

Merged
merged 1 commit into from
Mar 14, 2024

Conversation

danirod
Copy link
Contributor

@danirod danirod commented Mar 6, 2024

I believe that there is an error in the way refresh_pattern is set for eclipse-jdt-ls, that prevents complete suggestions to be shown after a dot has been typed (such as using the dot to access the methods of an instance variable).

The refresh pattern was originally set in #490. Other issues in vim-lsp and vim-lsp-settings seem to be pointing to this:

As seen in the following case, while suggestions work after typing a dot in the import statement, typing a dot inside a method body does not present suggestions.

Screen.Recording.2024-03-06.at.14.38.53.mov

Patching the refresh_pattern for eclipse-jdt-ls and removing the dot as proposed in the patch causes dots to always trigger the completion menu, both on imports and on method bodies:

Screen.Recording.2024-03-06.at.14.40.41.mov

I know little about Vim patterns and their differences in regard to normal PCRE patterns and while the fix seems to work and is aligned with the experience of other people in issues linked above, I'm scared about unexpected consequences of changing this.

Pressing a dot after typing an instance variable in a Java method body
to see its members and other definitions as suggestions is not working
currently in eclipse-jdt-ls for Java.

While this is working for imports, methods do not get resolved. This
commit replaces a suspicious dot that may be related to this, which
causes the issue to get fixed.
@mattn mattn merged commit b93c195 into mattn:master Mar 14, 2024
@mattn
Copy link
Owner

mattn commented Mar 14, 2024

Thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants