[PATCH] D18821: Add modernize-bool-to-integer-conversion

Piotr Padlewski via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 6 14:11:42 PDT 2016


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

In http://reviews.llvm.org/D18821#393556, @mnbvmar wrote:

> This check throws a warning also on the conversion to floats (probably very rare ones):
>
>   double number = true;
>
>
> Even though this behavior is correct, the code warns about the implicit conversion to **integers**.


That case is extremaly rare. I will see what I can do


http://reviews.llvm.org/D18821





More information about the cfe-commits mailing list