rust-lang/rust-clippy

from_iter_instead_of_collect / needless_collect interaction questionable

已关闭

#6,533 创建于 2021年1月1日

在 GitHub 查看贡献者指南
 (2 条评论) (0 个反应) (0 位负责人)Rust (1,391 个派生)batch import
C-enhancementI-false-negativeL-nurserygood first issue

仓库指标

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

描述

for _i in Vec::from_iter([1, 2, 3].iter()) {}

clippy will suggest to use .collect() instead of from_iter() here but blindly replacing will not work because .collect() will need some kind of type hint.

Them again

for _i in [1, 2, 3].iter().collect::<Vec<&i32>>() {}

does not trigger clippy::needless_collect (although it should!; neither does the first example) because we can iterate over

for _i in [1, 2, 3].iter() {}

directly.

贡献者指南

研究方向
研究 Clippy 源代码中的 `from iter instead of collect` 和 `needless collect` lint，相关文件可能位于 `clippy lints/src/from iter instead of collect.rs` 和 `clippy lints/src/needless collect.rs`。确定它们如何交互，以及为何在没有类型提示的情况下建议使用 `collect()` 会失败。考虑调整 lint，使其建议正确的类型提示，或在可能导致错误时抑制该建议。同时检查 `needless collect` 是否应针对 `from iter` 的情况触发。
技术栈
rust
领域
tooling
议题类型
缺陷
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
3
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-3 小时
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
新近可参与
清晰度议题是否清楚说明期望改动、验收标准和下一步。
基本清晰
前置要求
Basic understanding of Rust lintsFamiliarity with Clippy's lint structure
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
40