[PATCH] D45750: add extra acronyms for objc property names

Ben Hamilton via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 18 08:23:54 PDT 2018


benhamilton accepted this revision.
benhamilton added inline comments.
This revision is now accepted and ready to land.


================
Comment at: clang-tidy/objc/PropertyDeclarationCheck.cpp:42
 constexpr llvm::StringLiteral DefaultSpecialAcronyms[] = {
+    "[2-9]G",
     "ACL",
----------------
Probably should just make this:

  "\\d+G"



================
Comment at: test/clang-tidy/objc-property-declaration.m:21
+ at property(assign, nonatomic) int centerX;
+ at property(assign, nonatomic) int enbale2GBackgroundFetch;
 @end
----------------
enbale -> enable



Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D45750





More information about the cfe-commits mailing list