Use more functional programming in ArenaMap::insert

I find this more readable and it flattens out the body a little.

1 files changed, 3 insertions, 5 deletions

diff --git a/crates/ra_arena/src/map.rs b/crates/ra_arena/src/map.rs
index 5e764113d..b9521c342 100644
--- a/crates/ra_arena/src/map.rs
+++ b/crates/ra_arena/src/map.rs
@@ -17,11 +17,9 @@ impl<T, V> ArenaMap<Idx<T>, V> {
         if self.v.capacity() <= idx {
             self.v.reserve(idx + 1 - self.v.capacity());
         }
-        if self.v.len() <= idx {
-            while self.v.len() <= idx {
-                self.v.push(None);
-            }
-        }
+
+        let fill = (idx + 1).saturating_sub(self.v.len());
+        self.v.extend(std::iter::repeat_with(|| None).take(fill));
         self.v[idx] = Some(t);
     }