rust-lang/rust-clippy

New lint: clone on arguments taken by reference

Open

#2 074 ouverte le 19 sept. 2017

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

If a function takes an argument by reference, but later calls .clone() on it (or otherwise requires an impl Clone or impl Copy from it), lint that taking it by ownership allows the caller to pass in its memory rather than forcing a clone on the data.

I don't know if this is possible, but can it default to allow on public functions and deny on crate-private functions (due to the API change)?

Guide contributeur

Direction de recherche
Étudiez le guide de création de lint Clippy. Implémentez un nouveau lint qui détecte quand un argument de fonction est pris par référence mais cloné par la suite, indiquant qu'il devrait être pris par valeur. Considérez les valeurs par défaut allow/deny en fonction de la visibilité.
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.
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.
Plutôt claire
Prérequis
Rust
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70