llvm/llvm-project

Add option to readability-implicit-bool-conversion check to allow implicit conversion *to bool*

Open

#36,323 建立於 2018年4月2日

在 GitHub 查看
 (4 留言) (0 反應) (1 負責人)C++ (10,782 fork)batch import
bugzillaclang-tidyenhancementgood first issue

倉庫指標

Star
 (26,378 star)
PR 合併指標
 (平均合併 1天 2小時) (30 天內合併 1,000 個 PR)

描述
Bugzilla Link 36975
Version unspecified
OS All
Reporter LLVM Bugzilla Contributor

Extended Description

From the readability-implicit-bool-conversion check, I'd like to take advantage of these parts:

  • boolean expression/literal to integer,
  • boolean expression/literal to floating.

But not these:

  • integer expression/literal to boolean,
  • floating expression/literal to boolean,
  • pointer/pointer to member/nullptr/NULL to boolean,

Seems like an AllowConversionsToBool option would take care of this. Maybe for completeness/symmetry also a AllowConversionsFromBool option. Both defaulting to 0.

貢獻者指南

研究方向
該問題要求在 clang tidy 的 readability implicit bool conversion 檢查中新增選項。首先檢查現有檢查實作 clang tools extra/clang tidy/readability/ImplicitBoolConversionCheck.cpp。尋找目前的選項處理邏輯，並新增兩個布林選項：AllowConversionsToBool 和 AllowConversionsFromBool。修改檢查邏輯，根據這些選項抑制警告。更新檢查的文件，並在 clang tools extra/test/clang tidy/checkers/readability implicit bool conversion.cpp 中新增測試案例。查看問題評論以取得更多上下文。
技術棧
cpp
領域
toolingdeveloper experience
議題類型
功能
難度面向新貢獻者的預計實作難度，1 表示很小改動，5 表示專家級工作。
3
預計時間有經驗貢獻者完成調查、實作、測試並準備 pull request 的粗略時間範圍。
3-5 天
活動狀態議題目前的可參與程度：新鮮、活躍、陳舊、阻塞或等待維護者輸入。
活躍
清晰度議題是否清楚說明預期改動、驗收標準和下一步。
清晰
前置要求
C++GitLLVM build system
新手友善度1-100 的估計分數，表示該議題對首次貢獻者的友善程度。
65