Skip to content

Commit 0667d78

Browse files
committed
allow manual_repeat_n in other test
1 parent 2208d8a commit 0667d78

3 files changed

+6
-15
lines changed

tests/ui/from_iter_instead_of_collect.fixed

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
1-
#![warn(clippy::from_iter_instead_of_collect, clippy::manual_repeat_n)]
1+
#![warn(clippy::from_iter_instead_of_collect)]
22
#![allow(unused_imports)]
3-
#![allow(clippy::useless_vec)]
3+
#![allow(clippy::useless_vec, clippy::manual_repeat_n)]
44

55
use std::collections::{BTreeMap, BTreeSet, HashMap, VecDeque};
66

@@ -19,7 +19,7 @@ impl<'a> FromIterator<&'a bool> for Foo {
1919
}
2020

2121
fn main() {
22-
let iter_expr = std::iter::repeat_n(5, 5);
22+
let iter_expr = std::iter::repeat(5).take(5);
2323
let _ = iter_expr.collect::<Vec<_>>();
2424

2525
let _ = vec![5, 5, 5, 5].iter().enumerate().collect::<HashMap<usize, &i8>>();

tests/ui/from_iter_instead_of_collect.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
1-
#![warn(clippy::from_iter_instead_of_collect, clippy::manual_repeat_n)]
1+
#![warn(clippy::from_iter_instead_of_collect)]
22
#![allow(unused_imports)]
3-
#![allow(clippy::useless_vec)]
3+
#![allow(clippy::useless_vec, clippy::manual_repeat_n)]
44

55
use std::collections::{BTreeMap, BTreeSet, HashMap, VecDeque};
66

tests/ui/from_iter_instead_of_collect.stderr

Lines changed: 1 addition & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -7,15 +7,6 @@ LL | <Self as FromIterator<bool>>::from_iter(iter.into_iter().copied())
77
= note: `-D clippy::from-iter-instead-of-collect` implied by `-D warnings`
88
= help: to override `-D warnings` add `#[allow(clippy::from_iter_instead_of_collect)]`
99

10-
error: this `.repeat().take()` can be written more concisely
11-
--> tests/ui/from_iter_instead_of_collect.rs:22:21
12-
|
13-
LL | let iter_expr = std::iter::repeat(5).take(5);
14-
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using `repeat_n()` instead: `std::iter::repeat_n(5, 5)`
15-
|
16-
= note: `-D clippy::manual-repeat-n` implied by `-D warnings`
17-
= help: to override `-D warnings` add `#[allow(clippy::manual_repeat_n)]`
18-
1910
error: usage of `FromIterator::from_iter`
2011
--> tests/ui/from_iter_instead_of_collect.rs:23:13
2112
|
@@ -100,5 +91,5 @@ error: usage of `FromIterator::from_iter`
10091
LL | for _i in Vec::<&i32>::from_iter([1, 2, 3].iter()) {}
10192
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: use `.collect()` instead of `::from_iter()`: `[1, 2, 3].iter().collect::<Vec<&i32>>()`
10293

103-
error: aborting due to 16 previous errors
94+
error: aborting due to 15 previous errors
10495

0 commit comments

Comments
 (0)