rust-lang/rust-clippy

from_iter_instead_of_collect / needless_collect interaction questionable

Fermée

#6 533 ouverte le 1 janv. 2021

Voir sur GitHubGuide contributeur
 (2 commentaires) (0 réaction) (0 personne assignée)Rust (1 391 forks)batch import
C-enhancementI-false-negativeL-nurserygood first issue

Métriques du dépôt

Stars
 (10 406 étoiles)
Métriques de merge PR
 (Merge moyen 19j 22h) (113 PRs mergées en 30 j)

Description

for _i in Vec::from_iter([1, 2, 3].iter()) {}

clippy will suggest to use .collect() instead of from_iter() here but blindly replacing will not work because .collect() will need some kind of type hint.

Them again

for _i in [1, 2, 3].iter().collect::<Vec<&i32>>() {}

does not trigger clippy::needless_collect (although it should!; neither does the first example) because we can iterate over

for _i in [1, 2, 3].iter() {}

directly.

Guide contributeur

Direction de recherche
Étudiez les lints `from iter instead of collect` et `needless collect` dans le code source de Clippy, probablement dans des fichiers comme `clippy lints/src/from iter instead of collect.rs` et `clippy lints/src/needless collect.rs`. Déterminez comment ils interagissent et pourquoi la suggestion d'utiliser `collect()` échoue sans indications de type. Envisagez d'ajuster le lint pour suggérer une indication de type appropriée ou supprimer la suggestion lorsqu'elle provoquerait une erreur. Vérifiez également si `needless collect` devrait se déclencher pour le cas `from iter`.
Stack technique
rust
Domaine
tooling
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
Basic understanding of Rust lintsFamiliarity with Clippy's lint structure
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40