rust-lang/rust-clippy

`or_fun_call` isn't always a performance lint

Open

#8.574 geöffnet am 22. März 2022

Auf GitHub ansehen
 (11 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Rust (1.391 Forks)batch import
A-categoryL-nurserygood first issue

Repository-Metriken

Stars
 (10.406 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 19T 22h) (113 gemergte PRs in 30 T)

Beschreibung

Replacing .unwrap_or(Vec::new()) with .unwrap_or_default() doesn't do anything for performance, they produce identical machine code with optimizations: https://rust.godbolt.org/z/hzTdazGqW.

Perhaps the lint category could be changed or it could be changed to not detect cases where it's not actually a performance warning?

Contributor Guide

Research-Richtung
Das Issue diskutiert den `or fun call` Lint in Clippy, der vorschlägt, `.unwrap or(Vec::new())` durch `.unwrap or default()` für bessere Leistung zu ersetzen, aber der Melder argumentiert, dass der Compiler identischen Maschinencode erzeugt. Zur Lösung könnte man den Lint so ändern, dass er den Ersatz nur vorschlägt, wenn er tatsächlich die Leistung verbessert (z. B. wenn der Standardkonstruktor nicht trivial ist), oder die Lint Kategorie von 'performance' in 'style' oder 'complexity' ändern. Das Issue hat 11 Kommentare, darunter einige von Maintainern, die weitere Anleitung zum gewünschten Ansatz geben könnten.
Tech Stack
rust
Domain
developer experience
Issue Type
Fehler
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.
Aktiv
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.
60