rust-lang/rust-clippy

`or_fun_call` isn't always a performance lint

Open

#8 574 ouverte le 22 mars 2022

Voir sur GitHub
 (11 commentaires) (0 réactions) (0 assignés)Rust (1 391 forks)batch import
A-categoryL-nurserygood first issue

Métriques du dépôt

Stars
 (10 406 stars)
Métriques de merge PR
 (Merge moyen 19j 22h) (113 PRs mergées en 30 j)

Description

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?

Guide contributeur

Direction de recherche
Le problème discute du lint `or fun call` dans Clippy, qui suggère de remplacer `.unwrap or(Vec::new())` par `.unwrap or default()` pour la performance, mais le rapporteur soutient que le compilateur produit un code machine identique. Pour résoudre, on pourrait modifier le lint pour qu'il ne suggère le remplacement que lorsqu'il améliore réellement les performances (par exemple, lorsque le constructeur par défaut n'est pas trivial), ou changer la catégorie du lint de 'performance' à 'style' ou 'complexity'. Le problème comporte 11 commentaires, dont certains de mainteneurs, qui pourraient fournir des orientations supplémentaires sur l'approche souhaitée.
Stack technique
rust
Domaine
developer experience
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
RustClippy internals
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60