rust-lang/rust-clippy

don't fold(..) an Option when you can map_or(..)

Open

#1658 aperta il 3 apr 2017

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

Metriche repository

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

Descrizione

catch my_option.iter().fold(my_val, |x, y| ..y), suggest my_option.map_or(my_val, |x| ..my_val)

Guida contributor

Direzione di ricerca
Studia i lint Clippy esistenti che suggeriscono chiamate a metodi alternativi, poi scrivi un nuovo lint che rilevi il pattern `my option.iter().fold(my val, |x, y| ..y)` e suggerisca `my option.map or(my val, |x| ..my val)`. Testa con varie annotazioni di tipo.
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.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
RustClippy lint development
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70