rust-lang/rust-clippy

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

Open

#1563 aperta il 21 feb 2017

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)Rust (1391 fork)batch import
A-lintE-hardL-unnecessaryT-middlegood first issue

Metriche repository

Star
 (10.406 star)
Metriche merge PR
 (Merge medio 19g 22h) (113 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Questa issue propone un nuovo lint per Clippy che controlla le funzioni che accettano un riferimento (come `&str`) ma lo usano solo per creare un valore posseduto (come `to owned()`). Per implementarlo, esamina i lint esistenti nella crate `clippy lints`, in particolare quelli in `src/clippy lints/` che gestiscono argomenti di funzione e controllo dei tipi. Guarda il lint 'needless pass by value' come riferimento. L'issue attuale manca di una strategia di implementazione specifica; un buon punto di partenza è discutere l'ambito del lint e i possibili falsi positivi con i maintainer prima di scrivere codice.
Tech stack
rust
Dominio
tooling
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
3-5 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
RustClippy internalsGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70