Skip to content

Commit 79f88d3

Browse files
authored
Merge pull request #335 from fox0/clippy--manual_flatten
pr: Fix clippy::manual_flatten
2 parents ae0e83a + 23b4b16 commit 79f88d3

File tree

1 file changed

+6
-8
lines changed

1 file changed

+6
-8
lines changed

text/pr.rs

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -462,14 +462,12 @@ fn pr_merged(paths: &[PathBuf], params: &Parameters) -> io::Result<()> {
462462
);
463463
}
464464

465-
let mut required_rows = 0;
466-
for page in pages.iter() {
467-
if let Some(p) = page {
468-
if p.num_nonpadding_lines > required_rows {
469-
required_rows = p.num_nonpadding_lines;
470-
}
471-
}
472-
}
465+
let required_rows = pages
466+
.iter()
467+
.flatten()
468+
.map(|p| p.num_nonpadding_lines)
469+
.max()
470+
.unwrap_or_default();
473471

474472
let mut pages: Vec<_> = pages
475473
.into_iter()

0 commit comments

Comments
 (0)