rust-lang/rust-clippy

Lint suggesting from_fn instead of while let Some(x)

Open

#3,940 建立於 2019年4月11日

在 GitHub 查看
 (1 留言) (0 反應) (0 負責人)Rust (10,406 star) (1,391 fork)batch import
A-lintL-styleL-suggestiongood first issue

描述

With the release of 1.34 with the from_fn function, I'd like to propose a new lint that matches: while let Some(x) = expr and suggests replacing it with for x in from_fn(|| expr). (With a special-case for while let Some(x) = fn() that suggests for x in from_fn(fn) if possible.)

貢獻者指南

技術棧
rust
領域
tooling
議題類型
feature
難度面向新貢獻者的預計實作難度，1 表示很小改動，5 表示專家級工作。
3
預計時間有經驗貢獻者完成調查、實作、測試並準備 pull request 的粗略時間範圍。
1-3 hours
活動狀態議題目前的可參與程度：新鮮、活躍、陳舊、阻塞或等待維護者輸入。
stale
清晰度議題是否清楚說明預期改動、驗收標準和下一步。
clear
前置要求
Rust programmingclippy contributing guide
新手友善度1-100 的估計分數，表示該議題對首次貢獻者的友善程度。
70
研究方向
Examine existing clippy lints in the clippy lints crate to understand the structure for adding a new lint. The pattern to match is `while let Some(x) = expr` where `expr` is a function call. Refer to the std::iter::from fn documentation and clippy's developer guide for adding lints.