rust-lang/rust-clippy

Always true expressions in if expressions

Open

#1593 aperta il 3 mar 2017

Vedi su GitHub
 (3 commenti) (0 reazioni) (1 assegnatario)Rust (1391 fork)batch import
A-lintL-correctnessT-middlegood first issue

Metriche repository

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

Descrizione

I'd like Clippy to find this problem in the code, this expression is always true, and Clippy should also suggest the usage of && here instead:

fn foo1a(name: &str) {
    if name != "Min" || name != "Max" {}
}

Another example:

fn foo1b(x: i32) -> bool {
    x > 10 || x < 25
}

Guida contributor

Direzione di ricerca
Studia i lint di Clippy esistenti che rilevano errori logici nelle espressioni booleane. Implementa un nuovo lint che controlli le condizioni sempre vere/false nelle espressioni if utilizzando il pattern matching sull'AST. Scrivi test e assicurati che il lint suggerisca di sostituire `||` con `&&` quando appropriato.
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
RustGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60