]> git.lizzy.rs Git - rust.git/commitdiff
Use the right `DefMap` when looking up modules
authorJonas Schievink <jonasschievink@gmail.com>
Tue, 2 Feb 2021 11:25:13 +0000 (12:25 +0100)
committerJonas Schievink <jonasschievink@gmail.com>
Tue, 2 Feb 2021 11:36:38 +0000 (12:36 +0100)
crates/hir_def/src/nameres.rs
crates/hir_def/src/nameres/collector.rs
crates/hir_def/src/nameres/path_resolution.rs

index 6169b3bbcbdd851a000bb8bc51f243dcfb400008..5efc2fe470a8ac8bb77f634913aaee77d926ece5 100644 (file)
@@ -258,6 +258,10 @@ pub(crate) fn krate(&self) -> CrateId {
         self.krate
     }
 
+    pub(crate) fn block_id(&self) -> Option<BlockId> {
+        self.block.as_ref().map(|block| block.block)
+    }
+
     pub(crate) fn prelude(&self) -> Option<ModuleId> {
         self.prelude
     }
index fcc8e2607ea517202a3113eb72e7b155f82432ed..6e86cc4a7abaff86db4ba5fc0df603c16e140f60 100644 (file)
@@ -592,7 +592,13 @@ fn record_resolved_import(&mut self, directive: &ImportDirective) {
                         // glob import from same crate => we do an initial
                         // import, and then need to propagate any further
                         // additions
-                        let scope = &self.def_map[m.local_id].scope;
+                        let def_map;
+                        let scope = if m.block == self.def_map.block_id() {
+                            &self.def_map[m.local_id].scope
+                        } else {
+                            def_map = m.def_map(self.db);
+                            &def_map[m.local_id].scope
+                        };
 
                         // Module scoped macros is included
                         let items = scope
index 2a0f8ec2bc856db477a52d9dce40f85e10d186bd..2d1477160e46d0ab9c7bf1584f578796991ec9bb 100644 (file)
@@ -271,8 +271,16 @@ pub(super) fn resolve_path_fp_with_macro_single(
                         );
                     }
 
+                    let def_map;
+                    let module_data = if module.block == self.block_id() {
+                        &self[module.local_id]
+                    } else {
+                        def_map = module.def_map(db);
+                        &def_map[module.local_id]
+                    };
+
                     // Since it is a qualified path here, it should not contains legacy macros
-                    self[module.local_id].scope.get(&segment)
+                    module_data.scope.get(&segment)
                 }
                 ModuleDefId::AdtId(AdtId::EnumId(e)) => {
                     // enum variant