rust-lang/rust-clippy

Lint unwrap in drop impls

Open

#1523 aperta il 8 feb 2017

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)Rust (1391 fork)batch import
A-lintE-hardL-correctnessT-middlegood first issue

Metriche repository

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

Descrizione

Would it make sense to deny panic-able fns in Drop::drop implementations? It recently came up in https://github.com/diesel-rs/diesel/pull/641#discussion_r100066589.

Guida contributor

Direzione di ricerca
Studia i lint di Clippy esistenti che controllano i panic (es. `panic` lint) e comprendi il sistema di registrazione dei lint. Quindi implementa un nuovo lint che rileva le chiamate unwrap all'interno delle implementazioni di `Drop` attraversando i corpi delle funzioni dei metodi `drop`.
Tech stack
rust
Dominio
developer experience
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.
60