Use both type_of and doc_text_for

Author: h-michael

crates/ra_lsp_server/src/main_loop/handlers.rs

@@ -514,29 +514,26 @@ pub fn handle_hover(
         Some(it) => it,
     };
     let mut result = Vec::new();
+    let file_id = params.text_document.try_conv_with(&world)?;
+    let file_range = FileRange {
+        file_id,
+        range: rr.reference_range,
+    };
+    if let Some(type_name) = world.analysis().type_of(file_range)? {
+        result.push(type_name);
+    }
     for nav in rr.resolves_to {
         if let Some(docs) = world.analysis().doc_text_for(nav)? {
             result.push(docs);
         }
     }
+
     let range = rr.reference_range.conv_with(&line_index);
     if result.len() > 0 {
         return Ok(Some(Hover {
             contents: HoverContents::Scalar(MarkedString::String(result.join("\n\n---\n"))),
             range: Some(range),
         }));
-    } else {
-        let file_id = params.text_document.try_conv_with(&world)?;
-        let file_range = FileRange {
-            file_id,
-            range: rr.reference_range,
-        };
-        if let Some(type_name) = world.analysis().type_of(file_range)? {
-            return Ok(Some(Hover {
-                contents: HoverContents::Scalar(MarkedString::String(type_name)),
-                range: Some(range),
-            }));
-        }
     }
     Ok(None)
 }