[PATCH] D20857: [clang-tidy] Add modernize-explicit-operator-bool check.

Murray Cumming via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 2 01:10:58 PDT 2016


murrayc added a comment.

In http://reviews.llvm.org/D20857#446101, @etienneb wrote:

> I wonder if these two checks should not be merge in one checker.


Personally I find it cleaner to keep them separate, but I would be happy to combine them if that's wanted. I guessed that it would be easier to accept the explicit bool check than the operator void pointer check, and didn't want to make acceptance harder. I can also imagine someone wanting to disable one but not the other.


http://reviews.llvm.org/D20857





More information about the cfe-commits mailing list