[PATCH] D10933: Add new IdentifierNaming check
Beren Minor via cfe-commits
cfe-commits at lists.llvm.org
Thu Aug 6 07:38:27 PDT 2015
berenm updated this revision to Diff 31447.
berenm added a comment.
Here is an updated version with some style fixes, and function splits.
The styles are still selected the same way as before (no split between finding the best type style and falling back to available style), but there is only the StyleKind returned and the warning message is deduced from the StyleNames (lower case conversion and underscores replaced with spaces).
I can still implement the mechanism suggested in comment #218645 if you find it is cleaner.
http://reviews.llvm.org/D10933
Files:
clang-tidy/readability/CMakeLists.txt
clang-tidy/readability/IdentifierNamingCheck.cpp
clang-tidy/readability/IdentifierNamingCheck.h
clang-tidy/readability/ReadabilityTidyModule.cpp
test/clang-tidy/readability-identifier-naming.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D10933.31447.patch
Type: text/x-patch
Size: 35349 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150806/ab66e552/attachment-0001.bin>
More information about the cfe-commits
mailing list