[clang-tools-extra] Added options to readability-implicit-bool-conversion (PR #120087)

Julian Schmidt via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 19 16:25:07 PST 2024


================
@@ -319,10 +319,9 @@ Changes in existing checks
   diagnostic.
 
 - Improved :doc:`readability-implicit-bool-conversion
-  <clang-tidy/checks/readability/implicit-bool-conversion>` check
-  by adding the option `UseUpperCaseLiteralSuffix` to select the
-  case of the literal suffix in fixes and fixing false positive for implicit
-  conversion of comparison result in C23.
+  <clang-tidy/checks/readability/implicit-bool-conversion>` check by adding the
+   options: `UseUpperCaseLiteralSuffix`,`CheckConversionsToBool`,
+   `CheckConversionsFromBool`.
----------------
5chmidti wrote:

Please re-add what was already there to end up with something like: 

```
- Improved :doc:`readability-implicit-bool-conversion
  <clang-tidy/checks/readability/implicit-bool-conversion>` check
  by adding the option `UseUpperCaseLiteralSuffix` to select the
  case of the literal suffix in fixes and fixing false positive for implicit
  conversion of comparison result in C23, and by adding the
  options `CheckConversionsToBool` and
  `CheckConversionsFromBool` to configure which conversions 
  involving ``bool`` are allowed.
```

https://github.com/llvm/llvm-project/pull/120087


More information about the cfe-commits mailing list