[PATCH] Refactor: Move llvm clang-tidy checks to namespace clang::tidy::llvm

Alexander Kornienko alexfh at google.com
Mon Mar 2 04:42:53 PST 2015


Committed revision 230952.

Modified before commit: fixed issues pointed out in the comments.


================
Comment at: clang-tidy/llvm/IncludeOrderCheck.cpp:50
@@ -48,3 +49,3 @@
   Compiler.getPreprocessor().addPPCallbacks(
-      llvm::make_unique<IncludeOrderPPCallbacks>(*this,
+      ::llvm::make_unique<IncludeOrderPPCallbacks>(*this,
                                                  Compiler.getSourceManager()));
----------------
alexfh wrote:
> Wow, just one case of compiler being confused with another namespace llvm ;)
BTW, this needs to be clang-formatted.

http://reviews.llvm.org/D7995

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list