Enhancements for `manual_flatten` · rust-lang/rust-clippy#6676

2021-02-05T01:40:42.000Z

Enhancements for `manual_flatten` 1) `for_each` Before: ```rust iter.for_each(|o| { if let Some(x) = o { //.. } }) ``` After ```rust iter.flatten().for_each(|o| //..) ``` 2) nested loops Before: ```rust for x in vec![vec![1]] { for y in x { //.. x is not used again } } ``` After: ```rust for x in vec![vec![1]].into_iter().flatten() { // .. } ```

(4 评论) (1 反应) (1 负责人)Rust (1,391 fork)batch import

C-enhancementL-stylegood first issue

仓库指标

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

描述

Enhancements for manual_flatten

for_each

Before:

iter.for_each(|o| {
    if let Some(x) = o { //.. }
})

After

iter.flatten().for_each(|o| //..)

nested loops

Before:

for x in vec![vec![1]] {
    for y in x {
        //..  x is not used again
    }
}

After:

for x in vec![vec![1]].into_iter().flatten() {
    // ..
}

贡献者指南

研究方向: 该问题要求增强 Clippy 中的 `manual flatten` lint。贡献者应在 `clippy lints/src` 目录中找到现有实现，很可能在名为 `manual flatten.rs` 或类似的文件中。需要添加两种特定模式：一种是带有 `if let Some` 的 `for each`，另一种是外层变量未被使用的嵌套循环。由于问题已有分配者 (ayamdobhal)，建议检查当前状态并在可能的情况下进行协调。还应扩展现有的 `manual flatten` 测试用例。
技术栈: rust
领域: developer experiencetooling
议题类型: 功能
难度: 3
预计时间: 1-2 天
活动状态: 活跃
清晰度: 清晰
前置要求: Rust
新手友好度: 60

仓库指标

描述

贡献者指南

每天在邮箱收到新鲜 Easy issues。