Enhancements for `manual_flatten` · rust-lang/rust-clippy#6676

2021-02-05T01:40:42.000Z

Enhancements for `manual_flatten` 1) `for_each` Before: ```rust iter.for_each(|o| { if let Some(x) = o { //.. } }) ``` After ```rust iter.flatten().for_each(|o| //..) ``` 2) nested loops Before: ```rust for x in vec![vec![1]] { for y in x { //.. x is not used again } } ``` After: ```rust for x in vec![vec![1]].into_iter().flatten() { // .. } ```

(4 commentaires) (1 réaction) (1 assigné)Rust (1 391 forks)batch import

C-enhancementL-stylegood first issue

Métriques du dépôt

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

Description

Enhancements for manual_flatten

for_each

Before:

iter.for_each(|o| {
    if let Some(x) = o { //.. }
})

After

iter.flatten().for_each(|o| //..)

nested loops

Before:

for x in vec![vec![1]] {
    for y in x {
        //..  x is not used again
    }
}

After:

for x in vec![vec![1]].into_iter().flatten() {
    // ..
}

Guide contributeur

Direction de recherche: Cette demande sollicite des améliorations pour le lint `manual flatten` dans Clippy. Le contributeur doit localiser l'implémentation existante dans le répertoire `clippy lints/src`, probablement dans un fichier nommé `manual flatten.rs` ou similaire. Deux motifs spécifiques doivent être ajoutés : un pour `for each` avec `if let Some` et un pour les boucles imbriquées où la variable externe n'est pas utilisée. Comme la demande a un assigné (ayamdobhal), il est recommandé de vérifier l'état actuel et de coordonner si possible. Les cas de test existants pour `manual flatten` doivent également être étendus.
Stack technique: rust
Domaine: developer experiencetooling
Type d'issue: Fonctionnalité
Difficulté: 3
Temps estimé: 1-2 jours
Statut d'activité: Active
Clarté: Claire
Prérequis: Rust
Accessibilité débutant: 60

Métriques du dépôt

Description

Guide contributeur

Recevez de nouvelles issues Easy par e-mail.