Move all_ids implementation into DenseMap

Author: hinshun

src/densemap.rs

@@ -50,6 +50,10 @@ impl<K: Index, V> DenseMap<K, V> {
         self.vec.push(val);
         id
     }
+
+    pub fn all_ids(&self) -> impl Iterator<Item = K> {
+        (0..self.vec.len()).map(|id| K::from(id))
+    }
 }
 
 impl<K: Index, V: Clone> DenseMap<K, V> {

src/graph.rs

@@ -368,10 +368,6 @@ impl GraphFiles {
             }
         }
     }
-
-    pub fn all_ids(&self) -> impl Iterator<Item = FileId> {
-        (0..self.by_id.next_id().0).map(|id| FileId(id))
-    }
 }
 
 /// MTime info gathered for a file.  This also models "file is absent".

src/work.rs

@@ -383,7 +383,7 @@ impl<'a> Work<'a> {
     }
 
     pub fn want_every_file(&mut self, exclude: Option<FileId>) -> anyhow::Result<()> {
-        for id in self.graph.files.all_ids() {
+        for id in self.graph.files.by_id.all_ids() {
             if let Some(exclude) = exclude {
                 if id == exclude {
                     continue;