rust-lang/rust-clippy

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

Open

#1.563 geöffnet am 21. Feb. 2017

Auf GitHub ansehen
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Rust (1.391 Forks)batch import
A-lintE-hardL-unnecessaryT-middlegood first issue

Repository-Metriken

Stars
 (10.406 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 19T 22h) (113 gemergte PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Dieses Issue schlägt einen neuen Lint für Clippy vor, der Funktionen überprüft, die eine Referenz (wie `&str`) entgegennehmen, diese aber nur verwenden, um einen eigenen Wert (wie `to owned()`) zu erzeugen. Zur Implementierung sollten bestehende Lints in der Crate `clippy lints` untersucht werden, insbesondere solche in `src/clippy lints/`, die Funktionsargumente und Typprüfung behandeln. Der Lint 'needless pass by value' kann als Referenz dienen. Dem aktuellen Issue fehlt eine spezifische Implementierungsstrategie; ein guter Ausgangspunkt ist es, den Umfang des Lints und mögliche falsch positive Ergebnisse mit den Maintainern zu besprechen, bevor mit der Codierung begonnen wird.
Tech Stack
rust
Domain
tooling
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
3-5 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
RustClippy internalsGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70