improve collapsible if in complex cases with same expressions

Métricas do repositório

Stars

10.406 estrelas (10.406 estrelas)

Métricas de merge de PR

Métricas PR pendentes (Métricas PR pendentes)

Description

if a {
    if b {
        f();
    }
} else {
    f();
}

if !a || b {
    f();
}

Guia do colaborador

Direção de pesquisa

Examine a implementação atual do lint collapsible if no rust clippy (arquivo clippy lints/src/collapsible if.rs). Entenda como ele atualmente detecta instruções if aninhadas simples. A melhoria exigiria adicionar lógica para detectar casos em que o ramo else contém a mesma expressão que o corpo do if, e transformar em um único condicional. Procure por discussões ou tentativas existentes (a issue está atribuída, mas não há PR). O mantenedor pode ter opiniões sobre a abordagem desejada. Concentre se nos padrões AST para o exemplo fornecido.

Pilha de tecnologia

rust

Domain

tooling

Tipo Issue

Funcionalidade

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-2 dias

Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.

Ativo

ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.

Claro

Prerequisites

Rust

Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.

65