[PATCH] D77983: clang-tidy doc: add a note for every checker with an autofix

Sylvestre Ledru via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun May 10 02:39:39 PDT 2020


This revision was not accepted when it landed; it landed in state "Needs Review".
This revision was automatically updated to reflect the committed changes.
Closed by commit rGdc0f79ea5b99: clang-tidy doc: add a note for checkers with an autofix (authored by sylvestre.ledru).

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D77983/new/

https://reviews.llvm.org/D77983

Files:
  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


Index: clang-tools-extra/docs/clang-tidy/checks/modernize-use-bool-literals.rst
===================================================================
--- clang-tools-extra/docs/clang-tidy/checks/modernize-use-bool-literals.rst
+++ clang-tools-extra/docs/clang-tidy/checks/modernize-use-bool-literals.rst
@@ -26,3 +26,8 @@
 
    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.
Index: clang-tools-extra/docs/clang-tidy/checks/modernize-make-shared.rst
===================================================================
--- clang-tools-extra/docs/clang-tidy/checks/modernize-make-shared.rst
+++ clang-tools-extra/docs/clang-tidy/checks/modernize-make-shared.rst
@@ -48,3 +48,8 @@
 
    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.
Index: clang-tools-extra/docs/clang-tidy/checks/modernize-loop-convert.rst
===================================================================
--- clang-tools-extra/docs/clang-tidy/checks/modernize-loop-convert.rst
+++ clang-tools-extra/docs/clang-tidy/checks/modernize-loop-convert.rst
@@ -265,3 +265,8 @@
 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.
Index: clang-tools-extra/docs/clang-tidy/checks/modernize-deprecated-ios-base-aliases.rst
===================================================================
--- clang-tools-extra/docs/clang-tidy/checks/modernize-deprecated-ios-base-aliases.rst
+++ clang-tools-extra/docs/clang-tidy/checks/modernize-deprecated-ios-base-aliases.rst
@@ -15,3 +15,8 @@
 ``std::ios_base::streamoff``          
 ``std::ios_base::streampos``         
 ===================================  ===========================
+
+Automatic Fixes
+^^^^^^^^^^^^^^^
+
+The check provides suggested fixes. See the documentation above for the details.
Index: clang-tools-extra/docs/clang-tidy/checks/modernize-avoid-bind.rst
===================================================================
--- clang-tools-extra/docs/clang-tidy/checks/modernize-avoid-bind.rst
+++ clang-tools-extra/docs/clang-tidy/checks/modernize-avoid-bind.rst
@@ -86,3 +86,8 @@
 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.
Index: clang-tools-extra/docs/clang-tidy/checks/misc-unused-parameters.rst
===================================================================
--- clang-tools-extra/docs/clang-tidy/checks/misc-unused-parameters.rst
+++ clang-tools-extra/docs/clang-tidy/checks/misc-unused-parameters.rst
@@ -40,3 +40,8 @@
    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.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D77983.263062.patch
Type: text/x-patch
Size: 3430 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200510/37b09f2d/attachment.bin>


More information about the cfe-commits mailing list