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