Author: Sylvestre Ledru Date: 2020-05-10T11:27:13+02:00 New Revision: a832fc4b0c1e056edbb458973fac1f0c6db7a9e5
URL: https://github.com/llvm/llvm-project/commit/a832fc4b0c1e056edbb458973fac1f0c6db7a9e5 DIFF: https://github.com/llvm/llvm-project/commit/a832fc4b0c1e056edbb458973fac1f0c6db7a9e5.diff LOG: Revert two patches, not ready to be shared Revert "clang-tidy doc: add a note for checkers with an autofix" This reverts commit dc0f79ea5b99b9609a21c29089c294ab82d88094. Revert "add_new_check.py: Update of the template to add an autofix section" This reverts commit f97f92e5b0fd14240c2ec27c44e1b3cfddfdf668. Added: Modified: clang-tools-extra/clang-tidy/add_new_check.py clang-tools-extra/docs/clang-tidy/checks/misc-unused-parameters.rst clang-tools-extra/docs/clang-tidy/checks/modernize-avoid-bind.rst clang-tools-extra/docs/clang-tidy/checks/modernize-deprecated-ios-base-aliases.rst clang-tools-extra/docs/clang-tidy/checks/modernize-loop-convert.rst clang-tools-extra/docs/clang-tidy/checks/modernize-make-shared.rst clang-tools-extra/docs/clang-tidy/checks/modernize-use-bool-literals.rst Removed: ################################################################################ diff --git a/clang-tools-extra/clang-tidy/add_new_check.py b/clang-tools-extra/clang-tidy/add_new_check.py index fa8b4da4d624..4477444cc127 100755 --- a/clang-tools-extra/clang-tidy/add_new_check.py +++ b/clang-tools-extra/clang-tidy/add_new_check.py @@ -391,30 +391,6 @@ def write_docs(module_path, module, check_name): %(underline)s FIXME: Describe what patterns does the check detect and why. Give examples. - -FIXME: Remove of the following section: - -Automatic Fixes -^^^^^^^^^^^^^^^ - -The check doesn't provide an automatic fix. - -or - -Automatic Fixes -^^^^^^^^^^^^^^^ - -The check provides suggested fixes. See the documentation above for the details. - -or - -Automatic Fixes -^^^^^^^^^^^^^^^ - -The check provides suggested fixes in the following cases: -1. -2. -3. """ % {'check_name_dashes': check_name_dashes, 'underline': '=' * len(check_name_dashes)}) diff --git a/clang-tools-extra/docs/clang-tidy/checks/misc-unused-parameters.rst b/clang-tools-extra/docs/clang-tidy/checks/misc-unused-parameters.rst index de520e9511c7..3dfeb299de06 100644 --- a/clang-tools-extra/docs/clang-tidy/checks/misc-unused-parameters.rst +++ b/clang-tools-extra/docs/clang-tidy/checks/misc-unused-parameters.rst @@ -40,8 +40,3 @@ Options constructors - no constructor initializers). When the function body is empty, an unused parameter is unlikely to be unnoticed by a human reader, and there's basically no place for a bug to hide. - -Automatic Fixes -^^^^^^^^^^^^^^^ - -The check provides suggested fixes. See the documentation above for the details. diff --git a/clang-tools-extra/docs/clang-tidy/checks/modernize-avoid-bind.rst b/clang-tools-extra/docs/clang-tidy/checks/modernize-avoid-bind.rst index 35be77d11c08..82c290e4a21b 100644 --- a/clang-tools-extra/docs/clang-tidy/checks/modernize-avoid-bind.rst +++ b/clang-tools-extra/docs/clang-tidy/checks/modernize-avoid-bind.rst @@ -86,8 +86,3 @@ that accepts 2 arguments. With permissive parameter list, it instead generates which is correct. This check requires using C++14 or higher to run. - -Automatic Fixes -^^^^^^^^^^^^^^^ - -The check provides suggested fixes. See the documentation above for the details. diff --git a/clang-tools-extra/docs/clang-tidy/checks/modernize-deprecated-ios-base-aliases.rst b/clang-tools-extra/docs/clang-tidy/checks/modernize-deprecated-ios-base-aliases.rst index a36d58e4e31d..f217ff090445 100644 --- a/clang-tools-extra/docs/clang-tidy/checks/modernize-deprecated-ios-base-aliases.rst +++ b/clang-tools-extra/docs/clang-tidy/checks/modernize-deprecated-ios-base-aliases.rst @@ -15,8 +15,3 @@ Deprecated member type Replacement ``std::ios_base::streamoff`` ``std::ios_base::streampos`` =================================== =========================== - -Automatic Fixes -^^^^^^^^^^^^^^^ - -The check provides suggested fixes. See the documentation above for the details. diff --git a/clang-tools-extra/docs/clang-tidy/checks/modernize-loop-convert.rst b/clang-tools-extra/docs/clang-tidy/checks/modernize-loop-convert.rst index 89d4a0c95a01..82b27bbe9020 100644 --- a/clang-tools-extra/docs/clang-tidy/checks/modernize-loop-convert.rst +++ b/clang-tools-extra/docs/clang-tidy/checks/modernize-loop-convert.rst @@ -265,8 +265,3 @@ exclude incorrect diagnostics on OpenMP for loops prior to OpenMP 5. To prevent this check to be applied (and to break) OpenMP for loops but still be applied to non-OpenMP for loops the usage of ``NOLINT`` (see :ref:`clang-tidy-nolint`) on the specific for loops is recommended. - -Automatic Fixes -^^^^^^^^^^^^^^^ - -The check provides suggested fixes. See the documentation above for the details. diff --git a/clang-tools-extra/docs/clang-tidy/checks/modernize-make-shared.rst b/clang-tools-extra/docs/clang-tidy/checks/modernize-make-shared.rst index f81ac2c9cbb9..e73b32034f3d 100644 --- a/clang-tools-extra/docs/clang-tidy/checks/modernize-make-shared.rst +++ b/clang-tools-extra/docs/clang-tidy/checks/modernize-make-shared.rst @@ -48,8 +48,3 @@ Options If set to non-zero, the check will not give warnings inside macros. Default is `1`. - -Automatic Fixes -^^^^^^^^^^^^^^^ - -The check provides suggested fixes. See the documentation above for the details. diff --git a/clang-tools-extra/docs/clang-tidy/checks/modernize-use-bool-literals.rst b/clang-tools-extra/docs/clang-tidy/checks/modernize-use-bool-literals.rst index 09556800ed86..792d3a4c3dc4 100644 --- a/clang-tools-extra/docs/clang-tidy/checks/modernize-use-bool-literals.rst +++ b/clang-tools-extra/docs/clang-tidy/checks/modernize-use-bool-literals.rst @@ -26,8 +26,3 @@ Options If set to non-zero, the check will not give warnings inside macros. Default is `1`. - -Automatic Fixes -^^^^^^^^^^^^^^^ - -The check provides suggested fixes. See the documentation above for the details. _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
