[PATCH] D157428: [clang-tidy] `readability-implicit-bool-conversion.AllowIntegerConditions` ignores `DoStmt`s

Fabian Wolff via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 8 13:51:53 PDT 2023


fwolff marked an inline comment as done.
fwolff added a comment.

Thanks for reviewing this @PiotrZSL!

In D157428#4570497 <https://reviews.llvm.org/D157428#4570497>, @PiotrZSL wrote:

> What about switchStmt ?

The `AllowIntegerConditions` and `AllowPointerConditions` options only apply to `int -> bool` conversions, whereas switch statements have integral conditions.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D157428/new/

https://reviews.llvm.org/D157428



More information about the cfe-commits mailing list