[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)
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 16362 bytes
Desc: not available
More information about the cfe-commits