rust-lang/rust-clippy

don't fold(..) an Option when you can map_or(..)

Open

#1 658 ouverte le 3 avr. 2017

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)Rust (1 391 forks)batch import
A-lintL-styleT-middlegood 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

catch my_option.iter().fold(my_val, |x, y| ..y), suggest my_option.map_or(my_val, |x| ..my_val)

Guide contributeur

Direction de recherche
Étudiez les lints Clippy existants qui suggèrent des appels de méthode alternatifs, puis écrivez un nouveau lint qui détecte le motif `my option.iter().fold(my val, |x, y| ..y)` et suggère `my option.map or(my val, |x| ..my val)`. Testez avec diverses annotations de type.
Stack technique
rust
Domaine
tooling
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
RustClippy lint development
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70