eclipse.jdt.ls icon indicating copy to clipboard operation
eclipse.jdt.ls copied to clipboard

add support for linkedEditingRange

Open yyoncho opened this issue 4 years ago • 2 comments

Very handy for renaming local variables/methods.

yyoncho avatar Oct 22 '21 17:10 yyoncho

lsp4j certainly supports it. The problem I see for Java, is that the API is only meant for linking content in the same document. However, renaming a public field, or method declaration can have implications for many other source files in the project.

It seems like it would only be safe for local variables, and private members of a class.

rgrunber avatar Oct 25 '21 16:10 rgrunber

FTR the clojure-lsp returns error if you try to call linkedEditingRange for something used outside of the current document which IMHO makes sense for this API(at least I dont see a reason not to do it like that).

yyoncho avatar Oct 25 '21 20:10 yyoncho