Skip to content

Commit 88d87d4

Browse files
chris-laplantedjc
authored andcommitted
modify zombie handling code so it doesn't need to alloc
1 parent af5d925 commit 88d87d4

File tree

1 file changed

+2
-7
lines changed

1 file changed

+2
-7
lines changed

src/multi.rs

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -209,23 +209,18 @@ impl MultiState {
209209
now: Instant,
210210
) -> io::Result<()> {
211211
// Reap all consecutive 'zombie' progress bars from head of the list
212-
let mut zombies = vec![];
213212
let mut adjust = 0;
214-
for index in self.ordering.iter() {
215-
let member = &self.members[*index];
213+
while let Some(index) = self.ordering.first().copied() {
214+
let member = &self.members[index];
216215
if !member.is_zombie {
217216
break;
218217
}
219218

220-
zombies.push(*index);
221219
adjust += member
222220
.draw_state
223221
.as_ref()
224222
.map(|d| d.lines.len())
225223
.unwrap_or_default();
226-
}
227-
228-
for index in zombies {
229224
self.remove_idx(index);
230225
}
231226

0 commit comments

Comments
 (0)