]> git.lizzy.rs Git - rust.git/commit
Auto merge of #7138 - mgacek8:issue6808_iter_cloned_collect_FN_with_large_array,...
authorbors <bors@rust-lang.org>
Tue, 27 Apr 2021 13:44:51 +0000 (13:44 +0000)
committerbors <bors@rust-lang.org>
Tue, 27 Apr 2021 13:44:51 +0000 (13:44 +0000)
commit9af07e65aa3f9f7bf753341fc04f42bcbc43c67e
tree39577f1b2dc0ea55d33f2cf085856f0bccdc7a94
parent0a330e682410a2b52f31e3f0ee5e6738b7a2227d
parentd7627dcfc8a60aaedccf002738dc44a2576fa8fd
Auto merge of #7138 - mgacek8:issue6808_iter_cloned_collect_FN_with_large_array, r=Manishearth

Fix FN in `iter_cloned_collect` with a large array

fixes #6808
changelog: Fix FN in `iter_cloned_collect` with a large array

I spotted that [is_iterable_array](https://github.com/rust-lang/rust-clippy/blob/a362a4d1d0edb66aef186c1d27b28c60573078f4/clippy_lints/src/loops/explicit_iter_loop.rs#L67-L75) function that `explicit_iter_loop` lint is using only works for array sizes <= 32.
There is this comment:
> IntoIterator is currently only implemented for array sizes <= 32 in rustc

I'm a bit confused, because I read that [IntoIterator for arrays](https://doc.rust-lang.org/src/core/array/mod.rs.html#194-201) with const generic `N` is stable since = "1.0.0". Although Const Generics MVP were stabilized in Rust 1.51.

Should I set MSRV for the current change? I will try to test with older compilers soon.