Expand float_cmp to structs with PartialEq

Repository-Metriken

Stars

10.406 Stars (10.406 Stars)

PR-Merge-Metriken

Durchschn. Merge 19T 22h (Durchschn. Merge 19T 22h)113 gemergte PRs in 30 T (113 gemergte PRs in 30 T)

Beschreibung

#[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);
}

Contributor Guide

Research-Richtung

Untersuche die aktuelle Implementierung des float cmp Lints in clippy lints/src/float cmp.rs. Identifiziere, wie es direkte Float Vergleiche erkennt, und erweitere es, um Vergleiche von Structs zu erkennen, die PartialEq ableiten und Float Felder enthalten. Erwäge die Verwendung von Trait Auflösung, um zu prüfen, ob das PartialEq des Structs abgeleitet ist und ob irgendein Feldtyp f32 oder f64 ist. Füge dann einen Testfall hinzu, der dem Beispiel im Issue ähnelt.

Tech Stack

rust

Domain

developer experience

Issue Type

Funktion

SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.

3

Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.

1-2 Tage

AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.

Veraltet

KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.

Klar

Voraussetzungen

RustClippy internals

EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.

40