[PATCH] D15685: [clang-tidy] introducing cppcoreguidelines-enum-all-caps check

Alexander Kornienko via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 21 04:41:10 PST 2015


alexfh added a comment.

There's already the readability-identifier-naming <http://clang.llvm.org/extra/clang-tidy/checks/readability-identifier-naming.html> check that is highly configurable. It might miss the "use any naming style except for this one" functionality, but I'd prefer it to be added to that check rather than implementing another check for naming styles. After the required functionality is added to the readability-identifier-naming check, we can just register it in the cppcoreguidelines module with a custom name (corresponding to the rule) and a custom configuration that only enables verification of enumerator names (see `GoogleModule::getModuleOptions()` for an example of how to do this).


http://reviews.llvm.org/D15685





More information about the cfe-commits mailing list