refactor: break down is_shadowing_prev_scope to separate lines

amber-lang · Dec 11, 2024 · 4377b57 · 4377b57
1 parent 49043e1
commit 4377b57
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/src/utils/metadata/parser.rs b/src/utils/metadata/parser.rs
@@ -96,9 +96,15 @@ impl ParserMetadata {
     }
 
     pub fn is_shadowing_prev_scope(&self, name: &str) -> bool {
-        self.context.scopes.len() > 1
-            && self.context.scopes.iter().rev().skip(1)
-                .find_map(|scope| scope.get_var(name)).is_some()
+        if self.context.scopes.len() > 1 {
+            self.context.scopes.iter()
+                .rev()
+                .skip(1)
+                .find_map(|scope| scope.get_var(name))
+                .is_some()
+        } else {
+            false
+        }
     }
 
     /// Gets a variable from the current scope or any parent scope