#![warn(clippy::from_iter_instead_of_collect)] #![allow(unused_imports)] #![allow(clippy::useless_vec, clippy::manual_repeat_n)] use std::collections::{BTreeMap, BTreeSet, HashMap, VecDeque}; struct Foo(Vec); impl FromIterator for Foo { fn from_iter>(_: T) -> Self { todo!() } } impl<'a> FromIterator<&'a bool> for Foo { fn from_iter>(iter: T) -> Self { iter.into_iter().copied().collect::() //~^ from_iter_instead_of_collect } } fn main() { let iter_expr = std::iter::repeat(5).take(5); let _ = iter_expr.collect::>(); //~^ from_iter_instead_of_collect let _ = vec![5, 5, 5, 5].iter().enumerate().collect::>(); //~^ from_iter_instead_of_collect Vec::from_iter(vec![42u32]); let a = vec![0, 1, 2]; assert_eq!(a, (0..3).collect::>()); //~^ from_iter_instead_of_collect assert_eq!(a, (0..3).collect::>()); //~^ from_iter_instead_of_collect let mut b = (0..3).collect::>(); //~^ from_iter_instead_of_collect b.push_back(4); let mut b = (0..3).collect::>(); //~^ from_iter_instead_of_collect b.push_back(4); { use std::collections; let mut b = (0..3).collect::>(); //~^ from_iter_instead_of_collect b.push_back(4); } let values = [(0, 'a'), (1, 'b'), (2, 'c'), (3, 'd')]; let bm = values.iter().cloned().collect::>(); //~^ from_iter_instead_of_collect let mut bar = bm.range(0..2).collect::>(); //~^ from_iter_instead_of_collect bar.insert(&4, &'e'); let mut bts = (0..3).collect::>(); //~^ from_iter_instead_of_collect bts.insert(2); { use std::collections; let _ = (0..3).collect::>(); //~^ from_iter_instead_of_collect let _ = (0..3).collect::>(); //~^ from_iter_instead_of_collect } for _i in [1, 2, 3].iter().collect::>() {} //~^ from_iter_instead_of_collect for _i in [1, 2, 3].iter().collect::>() {} //~^ from_iter_instead_of_collect } fn issue14581() { let nums = [0, 1, 2]; let _ = &nums.iter().map(|&num| char::from_u32(num).unwrap()).collect::(); //~^ from_iter_instead_of_collect } fn test_implicit_generic_args(iter: impl Iterator + Copy) { struct S<'l, T = i32, const A: usize = 3, const B: usize = 3> { a: [&'l T; A], b: [&'l T; B], } impl<'l, T, const A: usize, const B: usize> FromIterator<&'l T> for S<'l, T, A, B> { fn from_iter>(_: I) -> Self { todo!() } } let _ = iter.collect::>(); //~^ from_iter_instead_of_collect let _ = iter.collect::>(); //~^ from_iter_instead_of_collect let _ = iter.collect::>(); //~^ from_iter_instead_of_collect let _ = iter.collect::>(); //~^ from_iter_instead_of_collect let _ = iter.collect::>(); //~^ from_iter_instead_of_collect let _ = iter.collect::>(); //~^ from_iter_instead_of_collect let _ = iter.collect::>(); //~^ from_iter_instead_of_collect let _ = iter.collect::(); //~^ from_iter_instead_of_collect }