From 8a5f74a24f726a839f3a0e154cfadec23040e14c Mon Sep 17 00:00:00 2001 From: Aleksey Kladov Date: Fri, 11 Jan 2019 16:01:48 +0300 Subject: use location link in goto def --- crates/ra_lsp_server/src/conv.rs | 28 ++++++++++++++++++-------- crates/ra_lsp_server/src/main_loop/handlers.rs | 9 +++++---- 2 files changed, 25 insertions(+), 12 deletions(-) (limited to 'crates/ra_lsp_server') diff --git a/crates/ra_lsp_server/src/conv.rs b/crates/ra_lsp_server/src/conv.rs index 0b9b18cbf..aad698da1 100644 --- a/crates/ra_lsp_server/src/conv.rs +++ b/crates/ra_lsp_server/src/conv.rs @@ -1,5 +1,5 @@ use languageserver_types::{ - self, CreateFile, DocumentChangeOperation, DocumentChanges, InsertTextFormat, Location, + self, CreateFile, DocumentChangeOperation, DocumentChanges, InsertTextFormat, Location, LocationLink, Position, Range, RenameFile, ResourceOp, SymbolKind, TextDocumentEdit, TextDocumentIdentifier, TextDocumentItem, TextDocumentPositionParams, Url, VersionedTextDocumentIdentifier, WorkspaceEdit, @@ -349,13 +349,25 @@ impl TryConvWith for &NavigationTarget { } } -impl TryConvWith for &RangeInfo { - type Ctx = ServerWorld; - type Output = Location; - fn try_conv_with(self, world: &ServerWorld) -> Result { - let line_index = world.analysis().file_line_index(self.info.file_id()); - to_location(self.info.file_id(), self.info.range(), &world, &line_index) - } +pub fn to_location_link( + target: &RangeInfo, + world: &ServerWorld, + // line index for original range file + line_index: &LineIndex, +) -> Result { + let url = target.info.file_id().try_conv_with(world)?; + let tgt_line_index = world.analysis().file_line_index(target.info.file_id()); + + let res = LocationLink { + origin_selection_range: Some(target.range.conv_with(line_index)), + target_uri: url.to_string(), + target_range: target.info.range().conv_with(&tgt_line_index), + target_selection_range: target + .info + .focus_range() + .map(|it| it.conv_with(&tgt_line_index)), + }; + Ok(res) } pub fn to_location( diff --git a/crates/ra_lsp_server/src/main_loop/handlers.rs b/crates/ra_lsp_server/src/main_loop/handlers.rs index e3bf55ae7..aad9d6568 100644 --- a/crates/ra_lsp_server/src/main_loop/handlers.rs +++ b/crates/ra_lsp_server/src/main_loop/handlers.rs @@ -9,7 +9,7 @@ use languageserver_types::{ SignatureInformation, SymbolInformation, TextDocumentIdentifier, TextEdit, WorkspaceEdit, }; use ra_ide_api::{ - FileId, FilePosition, FileRange, FoldKind, Query, RunnableKind, Severity, SourceChange, RangeInfo, + FileId, FilePosition, FileRange, FoldKind, Query, RunnableKind, Severity, RangeInfo, }; use ra_syntax::{TextUnit, AstNode}; use rustc_hash::FxHashMap; @@ -17,7 +17,7 @@ use serde_json::to_value; use std::io::Write; use crate::{ - conv::{to_location, Conv, ConvWith, MapConvWith, TryConvWith}, + conv::{to_location, to_location_link, Conv, ConvWith, MapConvWith, TryConvWith}, project_model::TargetKind, req::{self, Decoration}, server_world::ServerWorld, @@ -208,6 +208,7 @@ pub fn handle_goto_definition( params: req::TextDocumentPositionParams, ) -> Result> { let position = params.try_conv_with(&world)?; + let line_index = world.analysis().file_line_index(position.file_id); let nav_info = match world.analysis().goto_definition(position)? { None => return Ok(None), Some(it) => it, @@ -217,9 +218,9 @@ pub fn handle_goto_definition( .info .into_iter() .map(|nav| RangeInfo::new(nav_range, nav)) - .map(|nav| nav.try_conv_with(&world)) + .map(|nav| to_location_link(&nav, &world, &line_index)) .collect::>>()?; - Ok(Some(req::GotoDefinitionResponse::Array(res))) + Ok(Some(req::GotoDefinitionResponse::Link(res))) } pub fn handle_parent_module( -- cgit v1.2.3