fix(lsp.reference): single reference direct jump

[jamestrew]

follow up to #3099

clean up list_to_jump to work mostly with vim.lsp.util.locations_to_items output from the get-go

and closes #3118

[Conni2461]

i am going to review this tomorrow

[Conni2461]

works for both 0.9 and 0.10, can be merged after two nits are addressed, thanks. i like that solution more than #3120

thanks for fixing the issue :)

[jamestrew]

thanks for reviewing!

[lawrenceho]

Unfortunately this broke the jump to Java classes such as java.lang.String because the JDT URI is not a file name.
With item.filename being jdt:https://contents/java.base/java.lang/String.class..., the call vim.uri_from_fname(item.filename) in item_to_location would convert the JDT URI into an invalid URI by adding file:https:// in front of jdt:https:// and encode the already encoded URI again. The jump would fail as a result.

Error message:

Error executing vim.schedule lua callback: /usr/share/nvim/runtime/lua/vim/lsp/util.lua:1136: Cursor position outside buffer
stack traceback:
	[C]: in function 'nvim_win_set_cursor'
	/usr/share/nvim/runtime/lua/vim/lsp/util.lua:1136: in function 'jump_to_location'
	...nvim/lazy/telescope.nvim/lua/telescope/builtin/__lsp.lua:197: in function 'handler'
	/usr/share/nvim/runtime/lua/vim/lsp/client.lua:685: in function ''
	vim/_editor.lua: in function <vim/_editor.lua:0>

[jamestrew]

@lawrenceho that's unfortunate. I'll see what we can do to remedy this.

[lawrenceho]

Thank you @jamestrew. By the way, I can create a PR for this by adopting the same approach used in vim.uri_from_bufnr.