Skip to content

Commit 1c6a7ae

Browse files
committed
Use new std::iter::repeat_n method
as per clippy
1 parent 9b55e26 commit 1c6a7ae

File tree

2 files changed

+12
-14
lines changed

2 files changed

+12
-14
lines changed

src/inflate.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -183,7 +183,7 @@ fn deflate_chunks(
183183
let newdata: Vec<u8> = keyword
184184
.iter()
185185
.cloned()
186-
.chain(std::iter::repeat(0).take(2))
186+
.chain(std::iter::repeat_n(0, 2))
187187
.chain(iter)
188188
.collect();
189189
Ok(png::Chunk {

src/zlib/deflate.rs

Lines changed: 11 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -517,7 +517,7 @@ mod tests {
517517

518518
#[test]
519519
fn fixed_coded_l010d1() {
520-
let exp: Vec<u8> = (0..=255).chain(std::iter::repeat(255).take(10)).collect();
520+
let exp: Vec<u8> = (0..=255).chain(std::iter::repeat_n(255, 10)).collect();
521521
let dut: Vec<u8> = [0, 0, 1, 0xFF, 0xFE]
522522
.iter()
523523
.cloned()
@@ -532,7 +532,7 @@ mod tests {
532532

533533
#[test]
534534
fn fixed_coded_l257d1() {
535-
let exp: Vec<u8> = (0..=255).chain(std::iter::repeat(255).take(257)).collect();
535+
let exp: Vec<u8> = (0..=255).chain(std::iter::repeat_n(255, 257)).collect();
536536
let dut: Vec<u8> = [0, 0, 1, 0xFF, 0xFE]
537537
.iter()
538538
.cloned()
@@ -547,7 +547,7 @@ mod tests {
547547

548548
#[test]
549549
fn fixed_coded_l258d1() {
550-
let exp: Vec<u8> = (0..=255).chain(std::iter::repeat(255).take(258)).collect();
550+
let exp: Vec<u8> = (0..=255).chain(std::iter::repeat_n(255, 258)).collect();
551551
let dut: Vec<u8> = [0, 0, 1, 0xFF, 0xFE]
552552
.iter()
553553
.cloned()
@@ -607,15 +607,14 @@ mod tests {
607607

608608
#[test]
609609
fn fixed_codes_l003d24577() {
610-
let base: Vec<u8> = std::iter::repeat(5)
611-
.take(16)
612-
.chain(std::iter::repeat(80).take(3))
613-
.chain(std::iter::repeat(120).take(24574))
610+
let base: Vec<u8> = std::iter::repeat_n(5, 16)
611+
.chain(std::iter::repeat_n(80, 3))
612+
.chain(std::iter::repeat_n(120, 24574))
614613
.collect();
615614
let exp: Vec<u8> = base
616615
.iter()
617616
.cloned()
618-
.chain(std::iter::repeat(80).take(3))
617+
.chain(std::iter::repeat_n(80, 3))
619618
.collect();
620619
let dut: Vec<u8> = [0, 0x11, 0x60, 0xEE, 0x9F]
621620
.iter()
@@ -631,15 +630,14 @@ mod tests {
631630

632631
#[test]
633632
fn fixed_codes_l003d32768() {
634-
let base: Vec<u8> = std::iter::repeat(5)
635-
.take(16)
636-
.chain(std::iter::repeat(80).take(3))
637-
.chain(std::iter::repeat(120).take(32768 - 3))
633+
let base: Vec<u8> = std::iter::repeat_n(5, 16)
634+
.chain(std::iter::repeat_n(80, 3))
635+
.chain(std::iter::repeat_n(120, 32768 - 3))
638636
.collect();
639637
let exp: Vec<u8> = base
640638
.iter()
641639
.cloned()
642-
.chain(std::iter::repeat(80).take(3))
640+
.chain(std::iter::repeat_n(80, 3))
643641
.collect();
644642
let dut: Vec<u8> = [0, 0x10, 0x80, 0xEF, 0x7F]
645643
.iter()

0 commit comments

Comments
 (0)