mirror of https://github.com/helix-editor/helix
606b957172
The lsp location type has the lsp's URI type and a range. We can replace
that with a custom type private to the lsp commands module that uses the
core URI type instead.
We can't entirely replace the type with a new Location type in core.
That type might look like:
pub struct Location {
uri: crate::Uri,
range: crate::Range,
}
But we can't convert every `lsp::Location` to this type because for
definitions, references and diagnostics language servers send documents
which we haven't opened yet, so we don't have the information to convert
an `lsp::Range` (line+col) to a `helix_core::Range` (char indexing).
This cleans up the picker definitions in this file so that they can all
use helpers like `jump_to_location` and `location_to_file_location` for
the picker preview. It also removes the only use of the deprecated
`PathOrId::from_path_buf` function, allowing us to drop the owned
variant of that type in the child commit.
|
||
|---|---|---|
| .. | ||
| doc_formatter | ||
| increment | ||
| syntax | ||
| auto_pairs.rs | ||
| chars.rs | ||
| comment.rs | ||
| config.rs | ||
| diagnostic.rs | ||
| diff.rs | ||
| doc_formatter.rs | ||
| fuzzy.rs | ||
| graphemes.rs | ||
| history.rs | ||
| indent.rs | ||
| lib.rs | ||
| line_ending.rs | ||
| macros.rs | ||
| match_brackets.rs | ||
| movement.rs | ||
| object.rs | ||
| position.rs | ||
| rope_reader.rs | ||
| search.rs | ||
| selection.rs | ||
| shellwords.rs | ||
| surround.rs | ||
| syntax.rs | ||
| test.rs | ||
| text_annotations.rs | ||
| textobject.rs | ||
| transaction.rs | ||
| uri.rs | ||
| wrap.rs | ||