[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