Code review feedback.

Author: UnboundVariable

crates/ty_ide/src/lib.rs

@@ -131,23 +131,29 @@ pub enum ReferenceKind {
 /// includes information about whether the reference is a read or write operation.
 #[derive(Debug, Clone, PartialEq, Eq, Hash)]
 pub struct ReferenceTarget {
-    file: File,
-    range: TextRange,
+    file_range: FileRange,
     kind: ReferenceKind,
 }
 
 impl ReferenceTarget {
     /// Creates a new `ReferenceTarget`.
     pub fn new(file: File, range: TextRange, kind: ReferenceKind) -> Self {
-        Self { file, range, kind }
+        Self {
+            file_range: FileRange::new(file, range),
+            kind,
+        }
     }
 
     pub fn file(&self) -> File {
-        self.file
+        self.file_range.file()
     }
 
     pub fn range(&self) -> TextRange {
-        self.range
+        self.file_range.range()
+    }
+
+    pub fn file_range(&self) -> FileRange {
+        self.file_range
     }
 
     pub fn kind(&self) -> ReferenceKind {

crates/ty_server/src/document/location.rs

@@ -56,7 +56,7 @@ impl ToLink for NavigationTarget {
 
 impl ToLink for ReferenceTarget {
     fn to_location(&self, db: &dyn Db, encoding: PositionEncoding) -> Option<Location> {
-        FileRange::new(self.file(), self.range()).to_location(db, encoding)
+        self.file_range().to_location(db, encoding)
     }
 
     fn to_link(