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