Lint suggestion: use .collect() instead of ::from_iter()

[meithecatte]

When the user writes, say, HashMap::from_iter($iter_expr), suggest $iter_expr.collect().

This would be primarily beneficial to beginners. In fact, I've just seen this in the wild on Exercism ;)

[flip1995]

See also the documentation of FromIterator: https://doc.rust-lang.org/std/iter/trait.FromIterator.html

[meithecatte]

True, the documentation already mentions this, but it is not visible when you find the method in the list of traits a type implements.

Actually, I just realized that this is more general. I have also seen people explicitly use std::ops::Add::add, without realizing that this is equivalent to using the + operator.

[craigpastro]

Would it be okay if I gave this one a try?

[meithecatte]

Go ahead!

[pitiK3U]

@siyopao Still working on it or can I give it a try?

[craigpastro]

Hi @pitiK3U, sorry, yes, please go ahead.