[clang-tools-extra] a832fc4 - Revert two patches, not ready to be shared
Sylvestre Ledru via cfe-commits
cfe-commits at lists.llvm.org
Sun May 10 02:27:39 PDT 2020
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.
More information about the cfe-commits
mailing list