diff options
| author | Dylan DPC <dylan.dpc@gmail.com> | 2021-05-27 03:02:10 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-05-27 03:02:10 +0200 |
| commit | de1d7dbd0fad3da8c783fcbef742df3e60409bb7 (patch) | |
| tree | e5be6fa39fddf7778c6bef75e9556e7ba2bc8097 /compiler/rustc_codegen_llvm/src/allocator.rs | |
| parent | 0c53acc6f8ce7ecea99ca4844bd77334afaab3cb (diff) | |
| parent | caf6faf95107ab52d1f58b90d6bec764b428fdaa (diff) | |
| download | rust-de1d7dbd0fad3da8c783fcbef742df3e60409bb7.tar.gz rust-de1d7dbd0fad3da8c783fcbef742df3e60409bb7.zip | |
Rollup merge of #85689 - m-ou-se:array-intoiter-3, r=estebank
Remove Iterator #[rustc_on_unimplemented]s that no longer apply.
Now that `IntoIterator` is implemented for arrays, all the `rustc_on_unimplemented` for arrays of ranges (e.g. `for _ in [1..3] {}`) no longer apply, since they are now valid Rust.
Separated these from #85670, because we should discuss a potential new (clippy?) lint for these.
Until Rust 1.52, `for _ in [1..3] {}` produced:
```
error[E0277]: `[std::ops::Range<{integer}>; 1]` is not an iterator
--> src/main.rs:2:14
|
2 | for _ in [1..3] {}
| ^^^^^^ if you meant to iterate between two values, remove the square brackets
|
= help: the trait `std::iter::Iterator` is not implemented for `[std::ops::Range<{integer}>; 1]`
= note: `[start..end]` is an array of one `Range`; you might have meant to have a `Range` without the brackets: `start..end`
= note: required by `std::iter::IntoIterator::into_iter`
```
But in Rust 1.53 and later, it compiles fine. It iterates over the array by value, for one iteration with the element `1..3`.
This is probably a mistake, which is no longer caught. Should we have a lint for it? Should Clippy have a lint for it?
cc ```@estebank``` ```@flip1995```
cc https://github.com/rust-lang/rust/issues/84513
Diffstat (limited to 'compiler/rustc_codegen_llvm/src/allocator.rs')
0 files changed, 0 insertions, 0 deletions
