[PATCH] D40108: [clang-tidy] Adding Fuchsia checkers to clang-tidy

Eugene Zelenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 17 17:08:49 PST 2017


Eugene.Zelenko added inline comments.


================
Comment at: docs/ReleaseNotes.rst:63
+
+  Warns if a function is declared or called with default arguments.
+
----------------
I think will be good idea to change to //function// to //function or method//. Same in documentation.


================
Comment at: docs/clang-tidy/checks/fuchsia-default-arguments.rst:8
+
+Example: The declaration:
+
----------------
I briefly look on other checks documentation, so will be good idea to use just //Example:// or //For example, the declaration:// . But will be good idea to hear opinion of native English speaker.


https://reviews.llvm.org/D40108





More information about the cfe-commits mailing list