rust-lang/rust-clippy

`bool_comparison` should not fire when testing for false

Open

#1,043 创建于 2016年6月26日

在 GitHub 查看
 (4 评论) (2 反应) (0 负责人)Rust (1,391 fork)batch import
A-lintL-styleS-needs-discussiongood first issue

仓库指标

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

描述

I think that the bool_comparison lint should not fire when the comparison is var == false because, at least I, find that version much easier to see when quickly scrolling through the code than the !var as it is too simple to miss that single !. However, I'm just putting this here for discussion I'll go turn the lint off in that specific place.

Enjoy your rusty endeavors.

贡献者指南

研究方向
检查 clippy lints/src/bool comparison.rs 中 bool comparison lint 的实现。查看 issue 评论中是否有维护者的反馈或之前的尝试。此更改可能涉及为 `== false` 比较添加例外，或添加配置选项。实现更改并包含测试。
技术栈
rust
领域
developer experience
议题类型
缺陷
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
3
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-3 小时
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
活跃
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
RustGit
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
75