bendns' repos / rust-analyzer
Unnamed repository; edit this file 'description' to name the repository.
Diffstat (limited to 'crates/hir-def/src/test_db.rs')
-rw-r--r--crates/hir-def/src/test_db.rs33
1 files changed, 9 insertions, 24 deletions
diff --git a/crates/hir-def/src/test_db.rs b/crates/hir-def/src/test_db.rs
index 3bb9c361b3..12a1c1554c 100644
--- a/crates/hir-def/src/test_db.rs
+++ b/crates/hir-def/src/test_db.rs
@@ -190,15 +190,7 @@ impl TestDB {
let mut res = DefMap::ROOT;
for (module, data) in def_map.modules() {
let src = data.definition_source(self);
- // We're not comparing the `base_db::EditionedFileId`, but rather the VFS `FileId`, because
- // `position.file_id` is created before the def map, causing it to have to wrong crate
- // attached often, which means it won't compare equal. This should not be a problem in real
- // r-a session, only in tests, because in real r-a we only guess the crate on syntactic-only
- // (e.g. on-enter) handlers. The rest pick the `EditionedFileId` from the def map.
- let Some(file_id) = src.file_id.file_id() else {
- continue;
- };
- if file_id.file_id(self) != position.file_id.file_id(self) {
+ if src.file_id != position.file_id {
continue;
}
@@ -238,15 +230,7 @@ impl TestDB {
let mut fn_def = None;
for (_, module) in def_map.modules() {
let file_id = module.definition_source(self).file_id;
- // We're not comparing the `base_db::EditionedFileId`, but rather the VFS `FileId`, because
- // `position.file_id` is created before the def map, causing it to have to wrong crate
- // attached often, which means it won't compare equal. This should not be a problem in real
- // r-a session, only in tests, because in real r-a we only guess the crate on syntactic-only
- // (e.g. on-enter) handlers. The rest pick the `EditionedFileId` from the def map.
- let Some(file_id) = file_id.file_id() else {
- continue;
- };
- if file_id.file_id(self) != position.file_id.file_id(self) {
+ if file_id != position.file_id {
continue;
}
for decl in module.scope.declarations() {
@@ -269,25 +253,26 @@ impl TestDB {
};
if size != Some(new_size) {
size = Some(new_size);
- fn_def = Some((it, file_id));
+ fn_def = Some(it);
}
}
}
}
// Find the innermost block expression that has a `DefMap`.
- let (def_with_body, file_id) = fn_def?;
- let def_with_body = def_with_body.into();
+ let def_with_body = fn_def?.into();
let source_map = self.body_with_source_map(def_with_body).1;
let scopes = self.expr_scopes(def_with_body);
- let root_syntax_node = self.parse(file_id).syntax_node();
+ let root_syntax_node = self.parse(position.file_id).syntax_node();
let scope_iter =
algo::ancestors_at_offset(&root_syntax_node, position.offset).filter_map(|node| {
let block = ast::BlockExpr::cast(node)?;
let expr = ast::Expr::from(block);
- let expr_id =
- source_map.node_expr(InFile::new(file_id.into(), &expr))?.as_expr().unwrap();
+ let expr_id = source_map
+ .node_expr(InFile::new(position.file_id.into(), &expr))?
+ .as_expr()
+ .unwrap();
let scope = scopes.scope_for(expr_id).unwrap();
Some(scope)
});