[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