FromIterator is not in the prelude, so crates that want to provide an easily accessible from_iter method might want to duplicate it as an inherent method (providing both FromIterator and an inherent method). For this reason, I think the lint fires incorrectly - it suggests implementing FromIterator (the trait is already implemented, too) and implicitly suggests removing the inherent method (style/convenience I would disagree with).