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 Kommentare) (1 Reaktion) (1 zugewiesene Person)Rust (1.391 Forks)batch import

C-enhancementL-stylegood first issue

Repository-Metriken

Stars: (10.406 Stars)
PR-Merge-Metriken: (Durchschn. Merge 19T 22h) (113 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung: Das Issue fordert Verbesserungen am `manual flatten` Lint in Clippy. Der Beitragende sollte die bestehende Implementierung im Verzeichnis `clippy lints/src` finden, wahrscheinlich in einer Datei namens `manual flatten.rs` oder ähnlich. Zwei spezifische Muster müssen hinzugefügt werden: eines für `for each` mit `if let Some` und eines für verschachtelte Schleifen, bei denen die äußere Variable unbenutzt ist. Da das Issue einen Zugewiesenen (ayamdobhal) hat, wird empfohlen, den aktuellen Status zu prüfen und gegebenenfalls zu koordinieren. Die bestehenden Testfälle für `manual flatten` sollten ebenfalls erweitert werden.
Tech Stack: rust
Domain: developer experiencetooling
Issue Type: Funktion
Schwierigkeit: 3
Geschätzte Zeit: 1-2 Tage
Aktivitätsstatus: Aktiv
Klarheit: Klar
Voraussetzungen: Rust
Einsteigerfreundlichkeit: 60

Repository-Metriken

Beschreibung

Contributor Guide

Erhalte frische Easy Issues per E-Mail.