rust-lang/rust-clippy

from_iter_instead_of_collect / needless_collect interaction questionable

Fechada

#6.533 aberto em 1 de jan. de 2021

Ver no GitHubGuia do colaborador
 (2 comentários) (0 reação) (0 responsável)Rust (1.391 forks)batch import
C-enhancementI-false-negativeL-nurserygood first issue

Métricas do repositório

Stars
 (10.406 estrelas)
Métricas de merge de PR
 (Mesclagem média 19d 22h) (113 fundiu PRs em 30d)

Description

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.

Guia do colaborador

Direção de pesquisa
Investigue os lints `from iter instead of collect` e `needless collect` no código fonte do Clippy, provavelmente em arquivos como `clippy lints/src/from iter instead of collect.rs` e `clippy lints/src/needless collect.rs`. Determine como eles interagem e por que a sugestão de usar `collect()` falha sem dicas de tipo. Considere ajustar o lint para sugerir uma dica de tipo adequada ou suprimir a sugestão quando ela causaria um erro. Verifique também se `needless collect` deve ser acionado para o caso `from iter`.
Pilha de tecnologia
rust
Domain
tooling
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
Basic understanding of Rust lintsFamiliarity with Clippy's lint structure
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40