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 commenti) (1 reazione) (1 assegnatario)Rust (1391 fork)batch import

C-enhancementL-stylegood first issue

Metriche repository

Star: (10.406 star)
Metriche merge PR: (Merge medio 19g 22h) (113 PR mergiate in 30 g)

Descrizione

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() {
    // ..
}

Guida contributor

Direzione di ricerca: La segnalazione richiede miglioramenti al lint `manual flatten` in Clippy. Il contributore dovrebbe individuare l'implementazione esistente nella directory `clippy lints/src`, probabilmente in un file chiamato `manual flatten.rs` o simile. È necessario aggiungere due pattern specifici: uno per `for each` con `if let Some` e uno per cicli annidati in cui la variabile esterna non viene utilizzata. Poiché la segnalazione ha un assegnatario (ayamdobhal), si consiglia di verificare lo stato attuale e coordinarsi se possibile. Anche i casi di test esistenti per `manual flatten` dovrebbero essere estesi.
Tech stack: rust
Dominio: developer experiencetooling
Tipo issue: Funzionalità
Difficoltà: 3
Tempo stimato: 1-2 giorni
Stato attività: Attiva
Chiarezza: Chiara
Prerequisiti: Rust
Adatta ai principianti: 60

Metriche repository

Descrizione

Guida contributor

Ricevi issue Easy fresche nella tua inbox.