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 commento) (0 reazioni) (1 assegnatario)Rust (1391 fork)batch import

A-lintC-enhancementgood first issue

Metriche repository

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

could be

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

Direzione di ricerca: Esamina l'implementazione corrente del lint collapsible if in rust clippy (file clippy lints/src/collapsible if.rs). Comprendi come attualmente rileva le istruzioni if nidificate semplici. Il miglioramento richiederebbe l'aggiunta di logica per rilevare i casi in cui il ramo else contiene la stessa espressione del corpo dell'if, e trasformarli in un singolo condizionale. Cerca eventuali discussioni o tentativi esistenti (l'issue è assegnata ma non c'è PR). Il maintainer potrebbe avere input sull'approccio desiderato. Concentrati sui pattern AST per l'esempio fornito.
Tech stack: rust
Dominio: tooling
Tipo issue: Funzionalità
Difficoltà: 3
Tempo stimato: 1-2 giorni
Stato attività: Attiva
Chiarezza: Chiara
Prerequisiti: Rust
Adatta ai principianti: 65