aaron.ballman added inline comments.

================
Comment at: clang-tidy/misc/CMakeLists.txt:10
@@ -9,3 +9,2 @@
   InaccurateEraseCheck.cpp
-  InefficientAlgorithmCheck.cpp
   MacroParenthesesCheck.cpp
----------------
What about MoveConstantArgumentCheck.cpp?

================
Comment at: clang-tidy/misc/MiscTidyModule.cpp:58
@@ -57,3 +56,1 @@
-    CheckFactories.registerCheck<InefficientAlgorithmCheck>(
-        "misc-inefficient-algorithm");
     CheckFactories.registerCheck<MacroParenthesesCheck>(
----------------
This will break projects that enable the misc-inefficient-algorithm check 
(which clang 3.7 exposed). Is there a reason to not keep this check registered 
under this name?

(Perhaps a follow-up patch to allow deprecation of check names so that users 
are given guidance would make sense.)


http://reviews.llvm.org/D16248



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to