[PATCH] D46159: [clang-tidy] Add a flag to enable alpha checkers

Devin Coughlin via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 27 11:24:28 PDT 2018


dcoughlin added a comment.

In https://reviews.llvm.org/D46159#1081371, @lebedev.ri wrote:

> Note that it is completely off by default, and is not listed in documentation, `clang-tidy --help`,
>  so one would have to know of the existence of that hidden flag first, and only then when that flag is passed, those alpha checks could be enabled.


All it takes is one stack overflow comment or blog post recommending the flag and then we're back where we started. I'm really worried about well-intentioned ("running more checks means higher-quality code, so let's run ALL the checks") people enabling this option and having a bad experience.

Can you explain what the benefit of this flag is? How do you envision it being used?


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D46159





More information about the cfe-commits mailing list