[PATCH] D70368: [clang-tidy] Rewrite modernize-avoid-bind check

Eugene Zelenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 22 09:04:46 PST 2019


Eugene.Zelenko added inline comments.


================
Comment at: clang-tools-extra/docs/ReleaseNotes.rst:194
+  The check now supports supports diagnosing and fixing arbitrary callables instead of
+  only simple free functions. The ``PermissiveParameterList`` option has also been
+  added to address situations where the existing fix-it logic would sometimes generate
----------------
Please use single back-tics for options.


================
Comment at: clang-tools-extra/docs/clang-tidy/checks/modernize-avoid-bind.rst:50
+  If the option is set to non-zero, the check will append ``auto&&...`` to the end
+  of every placeholder parameter list.  Without this, it is possible for a fixit
+  to perform an incorrect transformation in the case where the result of the ``bind``
----------------
Double space here and in other places are still not fixed. Same for fixit.


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

https://reviews.llvm.org/D70368





More information about the cfe-commits mailing list