[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