squiid/rust
1
0
Fork 0
Code Issues Pull requests Projects Activity

Fix the range of a hover request to be more in line with prior art

Browse source
This commit is contained in:
DJMcNab 2018-12-06 22:56:39 +00:00 committed by Aleksey Kladov
parent 66f656134f
commit 3d3026dc60
1 changed files with 8 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
crates/ra_lsp_server/src/main_loop/handlers.rs
View file

@ -506,12 +506,19 @@ pub fn handle_hover(
) -> Result<Option<Hover>> {
let position = params.try_conv_with(&world)?;
let line_index = world.analysis().file_line_index(position.file_id);
let file = world.analysis().file_syntax(position.file_id);
for (file_id, symbol) in world.analysis().approximately_resolve_symbol(position)? {
let range = symbol.node_range.conv_with(&line_index);
let comment = world.analysis.doc_comment_for(file_id, symbol)?;
if comment.is_some() {
let range = match ra_syntax::algo::find_leaf_at_offset(file.syntax(), position.offset)
.left_biased()
{
None => return Ok(None),
Some(it) => it.range(),
};
let range = range.conv_with(&line_index);
let contents = HoverContents::Scalar(MarkedString::String(comment.unwrap()));
return Ok(Some(Hover {