improve collapsible if in complex cases with same expressions · rust-lang/rust-clippy#812

2016-03-30T14:34:06.000Z

``` rust if a { if b { f(); } } else { f(); } ``` could be ``` rust if !a || b { f(); } ```

(1 Kommentar) (0 Reaktionen) (1 zugewiesene Person)Rust (1.391 Forks)batch import

A-lintC-enhancementgood first issue

Repository-Metriken

if a {
    if b {
        f();
    }
} else {
    f();
}

could be

if !a || b {
    f();
}

Research-Richtung: Untersuche die aktuelle Implementierung des collapsible if Lints in rust clippy (Datei clippy lints/src/collapsible if.rs). Verstehe, wie es derzeit einfache verschachtelte if Anweisungen erkennt. Die Erweiterung erfordert das Hinzufügen von Logik, um Fälle zu erkennen, in denen der else Zweig denselben Ausdruck wie der if Körper enthält, und diese in eine einzelne Bedingung umzuwandeln. Suche nach bestehenden Diskussionen oder Versuchen (das Issue ist zugewiesen, aber es gibt keinen PR). Der Maintainer könnte Input zum gewünschten Ansatz haben. Konzentriere dich auf die AST Muster für das gegebene Beispiel.
Tech Stack: rust
Domain: tooling
Issue Type: Funktion
Schwierigkeit: 3
Geschätzte Zeit: 1-2 Tage
Aktivitätsstatus: Aktiv
Klarheit: Klar
Voraussetzungen: Rust
Einsteigerfreundlichkeit: 65