rust-lang/rust-clippy

`or_fun_call` isn't always a performance lint

Open

#8574 aperta il 22 mar 2022

Vedi su GitHub
 (11 commenti) (0 reazioni) (0 assegnatari)Rust (1391 fork)batch import
A-categoryL-nurserygood first issue

Metriche repository

Star
 (10.406 star)
Metriche merge PR
 (Merge medio 19g 22h) (113 PR mergiate in 30 g)

Descrizione

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?

Guida contributor

Direzione di ricerca
Il problema discute il lint `or fun call` in Clippy, che suggerisce di sostituire `.unwrap or(Vec::new())` con `.unwrap or default()` per migliorare le prestazioni, ma il segnalante sostiene che il compilatore produce codice macchina identico. Per risolvere, si potrebbe modificare il lint in modo che suggerisca la sostituzione solo quando effettivamente migliora le prestazioni (ad esempio, quando il costruttore predefinito non è banale), oppure cambiare la categoria del lint da 'performance' a 'style' o 'complexity'. Il problema ha 11 commenti, inclusi alcuni dei maintainer, che potrebbero fornire ulteriori indicazioni sull'approccio desiderato.
Tech stack
rust
Dominio
developer experience
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
RustClippy internals
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60