rust-lang/rust-clippy

Lint functions taking references as arguments but only use them to create an owned value

Open

#1 563 ouverte le 21 févr. 2017

Voir sur GitHub
 (3 commentaires) (0 réactions) (0 assignés)Rust (1 391 forks)batch import
A-lintE-hardL-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

A function that takes a &str and always converts it to a String, should be taking a String argument instead. This is hard to generalize, but we could start with anything implementing to_owned and only calling to_owned.

Guide contributeur

Direction de recherche
Cette issue propose un nouveau lint pour Clippy qui vérifie les fonctions prenant une référence (comme `&str`) mais ne l'utilisant que pour créer une valeur possédée (comme `to owned()`). Pour l'implémenter, examinez les lints existants dans la crate `clippy lints`, en particulier ceux dans `src/clippy lints/` qui traitent des arguments de fonction et de la vérification des types. Consultez le lint 'needless pass by value' comme référence. L'issue actuelle manque d'une stratégie d'implémentation spécifique ; un bon point de départ est de discuter de la portée du lint et des éventuels faux positifs avec les mainteneurs avant de coder.
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.
3-5 jours
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 internalsGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70