Enhancements for `manual_flatten` · rust-lang/rust-clippy#6676

2021-02-05T01:40:42.000Z

Enhancements for `manual_flatten` 1) `for_each` Before: ```rust iter.for_each(|o| { if let Some(x) = o { //.. } }) ``` After ```rust iter.flatten().for_each(|o| //..) ``` 2) nested loops Before: ```rust for x in vec![vec![1]] { for y in x { //.. x is not used again } } ``` After: ```rust for x in vec![vec![1]].into_iter().flatten() { // .. } ```

(4 comments) (1 reaction) (1 assignee)Rust (1.391 forks)batch import

C-enhancementL-stylegood first issue

Métricas do repositório

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

Description

Enhancements for manual_flatten

for_each

Before:

iter.for_each(|o| {
    if let Some(x) = o { //.. }
})

After

iter.flatten().for_each(|o| //..)

nested loops

Before:

for x in vec![vec![1]] {
    for y in x {
        //..  x is not used again
    }
}

After:

for x in vec![vec![1]].into_iter().flatten() {
    // ..
}

Guia do colaborador

Direção de pesquisa: A issue solicita melhorias para o lint `manual flatten` no Clippy. O contribuidor deve localizar a implementação existente no diretório `clippy lints/src`, provavelmente em um arquivo chamado `manual flatten.rs` ou similar. Dois padrões específicos precisam ser adicionados: um para `for each` com `if let Some` e outro para loops aninhados onde a variável externa não é usada. Como a issue tem um responsável atribuído (ayamdobhal), é recomendado verificar o status atual e coordenar, se possível. Os casos de teste existentes para `manual flatten` também devem ser estendidos.
Pilha de tecnologia: rust
Domain: developer experiencetooling
Tipo Issue: Funcionalidade
Difficulty: 3
Tempo estimado: 1-2 dias
Status da atividade: Ativo
Clarity: Claro
Prerequisites: Rust
Simpatia para novatos: 60

Métricas do repositório

Description

Guia do colaborador

Receba issues Easy novas por email.