rust-lang/rust-clippy

Manual assert_eq! (enhance `if_then_panic`)

Open

#7716 aperta il 24 set 2021

Vedi su GitHub
 (7 commenti) (0 reazioni) (1 assegnatario)Rust (1391 fork)batch import
C-enhancementgood first issue

Metriche repository

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

Descrizione

We just added if_then_panic, but then I realized there's a possible enhancement for assert_eq!/assert_ne!:

// before
if a != b {
    panic!("hi");
}
// after
assert_eq!(a, b, "hi");

I think a rename to manual_assert would be best. But we could also keep the current name (it still kinda makes sense) or create a new lint for this case.

Guida contributor

Direzione di ricerca
Esplora l'implementazione esistente del lint `if then panic` in rust clippy. L'issue suggerisce di migliorarlo per attivarsi anche sui pattern `assert eq!`/`assert ne!`. La decisione principale è se rinominare il lint in `manual assert`, mantenere il nome attuale o creare un nuovo lint. Guarda la discussione nei commenti e la PR #7669 per il contesto. L'utente assegnato è Labelray, quindi controlla i suoi progressi o commenti. L'implementazione comporterebbe l'aggiunta di una nuova corrispondenza di pattern nella logica del lint. Vedi file come `clippy lints/src/if then panic.rs` per riferimento.
Tech stack
rust
Dominio
tooling
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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