[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.
More information about the cfe-commits