rust-lang/rust-clippy

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

Open

#1.563 aberto em 21 de fev. de 2017

Ver no GitHub
 (3 comments) (0 reactions) (0 assignees)Rust (1.391 forks)batch import
A-lintE-hardL-unnecessaryT-middlegood first issue

Métricas do repositório

Stars
 (10.406 stars)
Métricas de merge de PR
 (Mesclagem média 19d 22h) (113 fundiu PRs em 30d)

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.

Guia do colaborador

Direção de pesquisa
Esta issue propõe um novo lint para o Clippy que verifica funções que recebem uma referência (como `&str`) mas a usam apenas para criar um valor com posse (como `to owned()`). Para implementar, examine os lints existentes na crate `clippy lints`, especialmente aqueles em `src/clippy lints/` que lidam com argumentos de função e verificação de tipos. Consulte o lint 'needless pass by value' como referência. A issue atual carece de uma estratégia de implementação específica; um bom ponto de partida é discutir o escopo do lint e possíveis falsos positivos com os mantenedores antes de codificar.
Pilha de tecnologia
rust
Domain
tooling
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
3-5 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
RustClippy internalsGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70