Merge #787

787: rename combine -> or r=matklad a=matklad



Co-authored-by: Aleksey Kladov <[email protected]>

2 files changed, 3 insertions, 3 deletions

diff --git a/crates/ra_hir/src/nameres.rs b/crates/ra_hir/src/nameres.rs
index 4b8097235..261ebdf97 100644
--- a/crates/ra_hir/src/nameres.rs
+++ b/crates/ra_hir/src/nameres.rs
@@ -135,7 +135,7 @@ impl<T> PerNs<T> {
         PerNs { types: self.types.as_ref(), values: self.values.as_ref() }
     }
 
-    pub fn combine(self, other: PerNs<T>) -> PerNs<T> {
+    pub fn or(self, other: PerNs<T>) -> PerNs<T> {
         PerNs { types: self.types.or(other.types), values: self.values.or(other.values) }
     }
 
@@ -439,7 +439,7 @@ impl ItemMap {
         let from_extern_prelude =
             self.extern_prelude.get(name).map_or(PerNs::none(), |&it| PerNs::types(it));
 
-        from_scope.combine(from_extern_prelude)
+        from_scope.or(from_extern_prelude)
     }
 
     // Returns Yes if we are sure that additions to `ItemMap` wouldn't change
diff --git a/crates/ra_hir/src/resolve.rs b/crates/ra_hir/src/resolve.rs
index e8abac5bc..40c860cf4 100644
--- a/crates/ra_hir/src/resolve.rs
+++ b/crates/ra_hir/src/resolve.rs
@@ -59,7 +59,7 @@ impl Resolver {
     pub fn resolve_name(&self, name: &Name) -> PerNs<Resolution> {
         let mut resolution = PerNs::none();
         for scope in self.scopes.iter().rev() {
-            resolution = resolution.combine(scope.resolve_name(name));
+            resolution = resolution.or(scope.resolve_name(name));
             if resolution.is_both() {
                 return resolution;
             }