[PATCH] D25659: [clang-tidy] Avoid running aliased checks twice

Malcolm Parsons via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 8 09:34:54 PST 2016


malcolm.parsons added a comment.

In https://reviews.llvm.org/D25659#588658, @alexfh wrote:

> I think, silently choosing one of the checks may be confusing and counter-intuitive. Should we just warn in case we see the same check enabled by multiple aliases?


A warning is useful if the checks have different settings, e.g. google-readability-braces-around-statements and readability-braces-around-statements.

But a warning is just noise for other aliases, e.g. cert-dcl54-cpp and misc-new-delete-overloads.


https://reviews.llvm.org/D25659





More information about the cfe-commits mailing list