[PATCH] D62437: [clang-tidy] Splits fuchsia-default-arguments

Eugene Zelenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 6 14:37:29 PDT 2019


Eugene.Zelenko added inline comments.


================
Comment at: clang-tools-extra/docs/ReleaseNotes.rst:122
+  Warns if a function or method is called with default arguments.
+  This was previously done by fuchsia-default-arguments check, which has been
+  removed.
----------------
Please highlight fuchsia-default-arguments with single back-ticks.


================
Comment at: clang-tools-extra/docs/ReleaseNotes.rst:129
+  Warns if a function or method is declared with default parameters.
+  This was previously done by fuchsia-default-arguments check, which has been
+  removed.
----------------
Please highlight fuchsia-default-arguments with single back-ticks.


================
Comment at: clang-tools-extra/docs/ReleaseNotes.rst:176
+  Warnings of function or method calls and declarations with default arguments
+  were moved to fuchsia-default-arguments-calls and
+  fuchsia-default-arguments-declarations checks respectively.
----------------
Please add links to fuchsia-default-arguments-calls and fuchsia-default-arguments-declarations.


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

https://reviews.llvm.org/D62437





More information about the cfe-commits mailing list