[PATCH] D30931: [clang-tidy] modified identifier naming case to use CT_AnyCase for ignoring case style

Juliusz Toczydłowski via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 17 02:46:32 PDT 2017


jutocz updated this revision to Diff 92108.
jutocz retitled this revision from "[clang-tidy] added new identifier naming case type for ignoring case style" to "[clang-tidy] modified identifier naming case to use CT_AnyCase for ignoring case style".
jutocz edited the summary of this revision.
jutocz added a comment.

I've modified patch to use llvm::Optional for not defined naming style and naming case.
I had to use llvm::Optional in two places:

- for entries in collection of naming styles (IdentifierNamingCheck::NamingStyles) where style for specific identifier was not set (more general style will be used in this case)
- for case type in NamingStyle, where case style was not set but prefix or postfix was (this remains a bit unintuitive, bacause case style will be ignored and only prefix or postfix will get validated)


https://reviews.llvm.org/D30931

Files:
  clang-tidy/readability/IdentifierNamingCheck.cpp
  clang-tidy/readability/IdentifierNamingCheck.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D30931.92108.patch
Type: text/x-patch
Size: 16362 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170317/c1a36936/attachment-0001.bin>


More information about the cfe-commits mailing list