[PATCH] D89407: [clang-tidy] Add scoped enum constants to identifier naming check

Nathan James via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 16 03:40:36 PDT 2020


njames93 added a comment.

In D89407#2333560 <https://reviews.llvm.org/D89407#2333560>, @Eugene.Zelenko wrote:

> I think will be good idea to add test for suffix/prefix too.

Not strictly necessary, there are tests that make sure prefixes and suffixes are applied correctly. Then there are tests that make sure a type of decl is correctly matched against its style, this is one of these tests. You'll find that's the convention with most of the style types tests in here.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D89407/new/

https://reviews.llvm.org/D89407



More information about the cfe-commits mailing list