[PATCH] Improved llvm-namespace-comment check.

Alexander Kornienko alexfh at google.com
Mon May 19 07:26:24 PDT 2014


Hi klimek,

Handle various forms of existing namespace closing comments, fix
existing comments with wrong namespace name, ignore short namespaces.

The state of this check now seems to be enough to enable it by default to gather
user feedback ;)

http://reviews.llvm.org/D3825

Files:
  clang-tidy/llvm/CMakeLists.txt
  clang-tidy/llvm/IncludeOrderCheck.cpp
  clang-tidy/llvm/IncludeOrderCheck.h
  clang-tidy/llvm/LLVMTidyModule.cpp
  clang-tidy/llvm/LLVMTidyModule.h
  clang-tidy/llvm/NamespaceCommentCheck.cpp
  clang-tidy/llvm/NamespaceCommentCheck.h
  clang-tidy/tool/ClangTidyMain.cpp
  unittests/clang-tidy/LLVMModuleTest.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D3825.9570.patch
Type: text/x-patch
Size: 17292 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140519/ae979d43/attachment.bin>


More information about the cfe-commits mailing list