Use &str instread of Option<String>

crates/ra_ide_api/src/display.rs

@@ -78,7 +78,7 @@ where
 pub(crate) fn doc_text_for(nav: NavigationTarget) -> Option<String> {
     match (nav.description(), nav.docs()) {
         (Some(desc), docs) => Some(rust_code_markup_with_doc(desc, docs)),
-        (None, Some(docs)) => Some(docs),
+        (None, Some(docs)) => Some(docs.to_string()),
         _ => None,
     }
 }

crates/ra_ide_api/src/display/navigation_target.rs

@@ -54,12 +54,12 @@ impl NavigationTarget {
         self.full_range
     }
 
-    pub fn docs(&self) -> Option<String> {
-        self.docs.clone()
+    pub fn docs(&self) -> Option<&str> {
+        self.docs.as_ref().map(String::as_str)
     }
 
-    pub fn description(&self) -> Option<String> {
-        self.description.clone()
+    pub fn description(&self) -> Option<&str> {
+        self.description.as_ref().map(String::as_str)
     }
 
     /// A "most interesting" range withing the `full_range`.