rust-lang/rust-clippy

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

Open

#1,563 创建于 2017年2月21日

在 GitHub 查看
 (3 评论) (0 反应) (0 负责人)Rust (1,391 fork)batch import
A-lintE-hardL-unnecessaryT-middlegood first issue

仓库指标

Star
 (10,406 star)
PR 合并指标
 (平均合并 19天 22小时) (30 天内合并 113 个 PR)

描述

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.

贡献者指南

研究方向
此问题提议为 Clippy 新增一个 lint，用于检查那些接收引用（如 `&str`）但仅用于创建拥有值（如 `to owned()`）的函数。为实现此功能，请查阅 `clippy lints` 包中的现有 lint，特别是 `src/clippy lints/` 目录下处理函数参数和类型检查的相关 lint。可参考 'needless pass by value' lint 作为示例。当前问题缺乏具体的实现策略；建议在编码前，与维护者讨论该 lint 的适用范围及可能的误报情况。
技术栈
rust
领域
tooling
议题类型
功能
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
3
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
3-5 天
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
较久未更新
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
RustClippy internalsGit
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
70