[PATCH] D56563: [clang-tidy] add options documentation to readability-identifier-naming checker

Eugene Zelenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 11 10:28:13 PST 2019


Eugene.Zelenko added inline comments.


================
Comment at: docs/clang-tidy/checks/readability-identifier-naming.rst:34
+
+    When defined, the checker will ensure abstract class names conform to the
+    selected casing.
----------------
MyDeveloperDay wrote:
> Eugene.Zelenko wrote:
> > check, please. Same in other places.
> I assumed you mean replace "ensure" with "check", now I'm not sure do you mean?
> 
> A) "the checker will check"
> B) "the check will ensure"
> C) "the check will check"
> 
I'm sorry for been ambiguous. I meant replacing checker with check.  Actually this wider problem: //check// is in Clang-tidy; //checker// - in Static Code Analyzer.


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

https://reviews.llvm.org/D56563





More information about the cfe-commits mailing list