[PATCH] D16248: [Clang-tidy] rename misc-inefficient-algorithm to performance-inefficient-algorithm

Alexander Kornienko via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 15 18:44:04 PST 2016


alexfh added a comment.

Makes sense. One comment re: documentation.


================
Comment at: docs/clang-tidy/checks/performance-inefficient-algorithm.rst:1
@@ -1,2 +1,2 @@
-.. title:: clang-tidy - misc-inefficient-algorithm
+.. title:: clang-tidy - performance-inefficient-algorithm
 
----------------
I'd leave the old documentation file with a redirect to the new one. See examples in the cert module. But instead of calling the old check name 'alias', mention that it was renamed. We also don't need the old file to appear in the index, so it should be marked `:orphan:` and the add_new_check.py script should be updated to not add orphaned files to the list.


Repository:
  rL LLVM

http://reviews.llvm.org/D16248





More information about the cfe-commits mailing list