lint None.into_iter()?

[matthiaskrgr]

What it does

If we want to have something like an iterator that starts with certain item and then continues with another range of items, we can do something like chaining an iterator to an option:

Some(0).into_iter().chain(vec![1, 2, 3].into_iter())

Although it does not make a lot of sense, this works with None as well but the first item will not be saved in the iterator.

 None::<i32>.into_iter().chain(vec![1, 2, 3].into_iter())

I wonder if it makes sense to lint something like None.(into_)iter() and suggest std::iter::empty or removing the code entirely.

Lint Name

none_to_iter? none_into_iter?

Category

style, complexity

Advantage

No response

Drawbacks

No response

Example

 None::<i32>.into_iter().chain(vec![1, 2, 3].into_iter())

Could be written as:

vec![1,2,3].into_iter()