benhamilton added inline comments.
================
Comment at: clang-tidy/objc/PropertyDeclarationCheck.cpp:89
+ assert(MatchedDecl->getName().size() > 0);
+ // Skip the check of lowerCamelCase if the name has prefix of special
acronyms
+ if (startsWithSpecialAcronyms(MatchedDecl->getName(), SpecialAcronyms)) {
----------------
For the acronyms, this means we will allow properties with names like:
URLfoo_bar_blech
when we do not want that to be legal.
Instead, let's generate the regular expression we pass to `matchesName()` from
the options. I will provide a suggestion shortly.
https://reviews.llvm.org/D39829
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
