Is it possible to display correct line numbers when using Eclipse Search panel to display References

[sdaimwood]

Hi there, I'm trying to display the results of a references query in the Search field of the eclipse workspace. The search display is currently displaying the linenumber - 1 for the reference's line. I've debugged with the code and clearly it is displaying the Location object's exact value as the symbols are sent back from the language server and processed in LSSearchQuery. However, the location is zero-indexed, which doesn't match up with Eclipse's displayed line numbers. Curious if I'm missing a workaround or if this is something that needs to be fixed in LSP4E.

Due to my business organization's limits, we're using v0.20.7 and Eclipse 4.23, I tried to see if something like this issue has been fixed in subsequent releases but couldn't find anything definite. Didn't want to create an issue in case it would be a duplicate.

See screenshots for where the references arrows are shown in the document and how the Search panel displays the linenumber - 1 as it's represented in the language server.

[sdaimwood]

Did some more debugging and found that if you modify the following lines:

lsp4e/org.eclipse.lsp4e/src/org/eclipse/lsp4e/operations/references/LSSearchQuery.java

Lines 131 to 133 in 9c92a32

	final var lineEntry = new LineElement(resource, document.getLineOfOffset(startOffset),
	lineInformation.getOffset(),
	document.get(lineInformation.getOffset(), lineInformation.getLength()));

to simply add +1 when constructing the LineElement:

 final var lineEntry = new LineElement(resource, document.getLineOfOffset(startOffset)+1, 
 		lineInformation.getOffset(), 
 		document.get(lineInformation.getOffset(), lineInformation.getLength())); 

that the display is correct, and the reference still goes to the correct location in the document.

[mickaelistria]

Thanks for the investigation. Please submit a pull request for this patch.

[sdaimwood]

Will do, thank you for the confirmation 👍