rust-lang/rust-clippy

from_iter_instead_of_collect / needless_collect interaction questionable

Chiusa

#6533 aperta il 1 gen 2021

Vedi su GitHubGuida contributor
 (2 commenti) (0 reazioni) (0 assegnatari)Rust (1391 fork)batch import
C-enhancementI-false-negativeL-nurserygood first issue

Metriche repository

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

Descrizione

for _i in Vec::from_iter([1, 2, 3].iter()) {}

clippy will suggest to use .collect() instead of from_iter() here but blindly replacing will not work because .collect() will need some kind of type hint.

Them again

for _i in [1, 2, 3].iter().collect::<Vec<&i32>>() {}

does not trigger clippy::needless_collect (although it should!; neither does the first example) because we can iterate over

for _i in [1, 2, 3].iter() {}

directly.

Guida contributor

Direzione di ricerca
Indaga i lint `from iter instead of collect` e `needless collect` nel codice sorgente di Clippy, probabilmente nei file come `clippy lints/src/from iter instead of collect.rs` e `clippy lints/src/needless collect.rs`. Determina come interagiscono e perché il suggerimento di usare `collect()` fallisce senza hint di tipo. Considera di modificare il lint per suggerire un hint di tipo appropriato o sopprimere il suggerimento quando causerebbe un errore. Verifica anche se `needless collect` dovrebbe attivarsi per il caso `from iter`.
Tech stack
rust
Dominio
tooling
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
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.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
Basic understanding of Rust lintsFamiliarity with Clippy's lint structure
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40