[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