[PATCH] D19165: [clang-tidy] Add modernize-increment-bool check.

Alexander Kornienko via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 3 16:06:58 PDT 2016


alexfh requested changes to this revision.
alexfh added a comment.
This revision now requires changes to proceed.

> -Wdeprecated-increment-bool does it. But what I see from SemaExpr.cpp in CheckIncrementDecrementOperand, it doesn't have any fixits.


If an automated fix for this issue makes sense at all, it should be implemented in the Clang's diagnostic. BTW, have you seen this pattern in the real code? I haven't. Maybe it's so rare that implementing a fix-it suggestion for it just isn't worth the effort?


http://reviews.llvm.org/D19165





More information about the cfe-commits mailing list