use iterative rather than recursive method for following aliases (#573)

* use iterative rather than recursive method for following aliases

* this avoids consuming stack via plain recursive calls

lib/codegen/src/write.rs

@@ -283,10 +283,14 @@ fn write_value_aliases(
     target: Value,
     indent: usize,
 ) -> fmt::Result {
-    for &a in &aliases[target] {
+    let mut todo_stack = vec![target];
-        writeln!(w, "{1:0$}{2} -> {3}", indent, "", a, target)?;
+    while let Some(target) = todo_stack.pop() {
-        write_value_aliases(w, aliases, a, indent)?;
+        for &a in &aliases[target] {
+            writeln!(w, "{1:0$}{2} -> {3}", indent, "", a, target)?;
+            todo_stack.push(a);
+        }
     }
+
     Ok(())
 }
 

lib/entity/src/map.rs

@@ -84,12 +84,12 @@ where
         Keys::with_len(self.elems.len())
     }
 
-    /// Iterate over all the keys in this map.
+    /// Iterate over all the values in this map.
     pub fn values(&self) -> slice::Iter<V> {
         self.elems.iter()
     }
 
-    /// Iterate over all the keys in this map, mutable edition.
+    /// Iterate over all the values in this map, mutable edition.
     pub fn values_mut(&mut self) -> slice::IterMut<V> {
         self.elems.iter_mut()
     }