Expand float_cmp to structs with PartialEq

Métricas do repositório

Stars

10.406 stars (10.406 stars)

Métricas de merge de PR

Mesclagem média 19d 22h (Mesclagem média 19d 22h)113 fundiu PRs em 30d (113 fundiu PRs em 30d)

Description

#[derive(PartialEq)]
struct Data {
    v: f64,
}

fn main() {
    let d1 = Data { v: 1.0 };
    let d2 = Data { v: 2.0 };

    // here should be a warning
    println!("{}", d1 == d2);
}

Guia do colaborador

Direção de pesquisa

Examine a implementação atual do lint float cmp em clippy lints/src/float cmp.rs. Identifique como ele detecta comparações diretas de floats e estenda o para detectar comparações de structs que derivam PartialEq e contêm campos float. Considere usar resolução de traits para verificar se o PartialEq da struct é derivado e se algum tipo de campo é f32 ou f64. Em seguida, adicione um caso de teste semelhante ao exemplo da issue.

Pilha de tecnologia

rust

Domain

developer experience

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.

Antigo

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

Claro

Prerequisites

RustClippy internals

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

40