}
/// Get a name from current module scope, legacy macros are not included
- pub(crate) fn get(&self, name: &Name, shadow: BuiltinShadowMode) -> Option<PerNs> {
+ pub(crate) fn get(&self, name: &Name, shadow: BuiltinShadowMode) -> PerNs {
match shadow {
- BuiltinShadowMode::Module => {
- self.visible.get(name).or_else(|| BUILTIN_SCOPE.get(name)).copied()
- }
+ BuiltinShadowMode::Module => self
+ .visible
+ .get(name)
+ .or_else(|| BUILTIN_SCOPE.get(name))
+ .copied()
+ .unwrap_or_else(PerNs::none),
BuiltinShadowMode::Other => {
let item = self.visible.get(name).copied();
if let Some(def) = item {
if let Some(ModuleDefId::ModuleId(_)) = def.take_types() {
- return BUILTIN_SCOPE.get(name).copied().or(item);
+ return BUILTIN_SCOPE
+ .get(name)
+ .copied()
+ .or(item)
+ .unwrap_or_else(PerNs::none);
}
}
- item.or_else(|| BUILTIN_SCOPE.get(name).copied())
+ item.or_else(|| BUILTIN_SCOPE.get(name).copied()).unwrap_or_else(PerNs::none)
}
}
}
}
// Since it is a qualified path here, it should not contains legacy macros
- match self[module.local_id].scope.get(&segment, prefer_module(i)) {
- Some(def) => def,
- _ => {
- log::debug!("path segment {:?} not found", segment);
- return ResolvePathResult::empty(ReachedFixedPoint::No);
- }
- }
+ self[module.local_id].scope.get(&segment, prefer_module(i))
}
ModuleDefId::AdtId(AdtId::EnumId(e)) => {
// enum variant
// - std prelude
let from_legacy_macro =
self[module].scope.get_legacy_macro(name).map_or_else(PerNs::none, PerNs::macros);
- let from_scope = self[module].scope.get(name, shadow).unwrap_or_else(PerNs::none);
+ let from_scope = self[module].scope.get(name, shadow);
let from_extern_prelude =
self.extern_prelude.get(name).map_or(PerNs::none(), |&it| PerNs::types(it));
let from_prelude = self.resolve_in_prelude(db, name, shadow);
name: &Name,
shadow: BuiltinShadowMode,
) -> PerNs {
- let from_crate_root = self[self.root].scope.get(name, shadow).unwrap_or_else(PerNs::none);
+ let from_crate_root = self[self.root].scope.get(name, shadow);
let from_extern_prelude = self.resolve_name_in_extern_prelude(name);
from_crate_root.or(from_extern_prelude)
keep = db.crate_def_map(prelude.krate);
&keep
};
- def_map[prelude.local_id].scope.get(name, shadow).unwrap_or_else(PerNs::none)
+ def_map[prelude.local_id].scope.get(name, shadow)
} else {
PerNs::none()
}