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/



_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to