Skip to content

Commit

Permalink
use implicit iter loops
Browse files Browse the repository at this point in the history
  • Loading branch information
danieleades authored and djc committed Dec 11, 2022
1 parent f363b7b commit c582ebc
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
6 changes: 3 additions & 3 deletions src/multi.rs
Original file line number Diff line number Diff line change
Expand Up @@ -251,7 +251,7 @@ impl MultiState {

// Reap all consecutive 'zombie' progress bars from head of the list.
let mut adjust = 0;
for &index in self.ordering.iter() {
for &index in &self.ordering {
let member = &self.members[index];
if !member.is_zombie {
break;
Expand Down Expand Up @@ -299,7 +299,7 @@ impl MultiState {
// Add lines from `ProgressBar::println` call.
draw_state.lines.append(&mut self.orphan_lines);

for index in self.ordering.iter() {
for index in &self.ordering {
let member = &self.members[*index];
if let Some(state) = &member.draw_state {
draw_state.lines.extend_from_slice(&state.lines[..]);
Expand All @@ -309,7 +309,7 @@ impl MultiState {
drop(draw_state);
let drawable = drawable.draw();

for index in reap_indices.into_iter() {
for index in reap_indices {
self.remove_idx(index);
}

Expand Down
4 changes: 2 additions & 2 deletions src/state.rs
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@ impl BarState {
self.state.pos.reset(now);
self.state.status = Status::InProgress;

for (_, tracker) in self.style.format_map.iter_mut() {
for tracker in self.style.format_map.values_mut() {
tracker.reset(&self.state, now);
}

Expand Down Expand Up @@ -126,7 +126,7 @@ impl BarState {
self.state.est.record(pos, now);
let _ = self.draw(false, now);

for (_, tracker) in self.style.format_map.iter_mut() {
for tracker in self.style.format_map.values_mut() {
tracker.tick(&self.state, now);
}
}
Expand Down
2 changes: 1 addition & 1 deletion src/style.rs
Original file line number Diff line number Diff line change
Expand Up @@ -590,7 +590,7 @@ impl Template {
}

fn set_tab_width(&mut self, new_tab_width: usize) {
for part in self.parts.iter_mut() {
for part in &mut self.parts {
if let TemplatePart::Literal(s) = part {
s.set_tab_width(new_tab_width);
}
Expand Down

0 comments on commit c582ebc

Please sign in to comment.