improve collapsible if in complex cases with same expressions

Metriche repository

Star

10.406 stelle (10.406 stelle)

Metriche merge PR

Merge medio 19g 22h (Merge medio 19g 22h)113 PR mergiate in 30 g (113 PR mergiate in 30 g)

Descrizione

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

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

Guida contributor

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àQuanto dovrebbe essere impegnativa per un nuovo contributor.

3

Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.

1-2 giorni

Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.

Attiva

ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.

Chiara

Prerequisiti

Rust

Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.

65