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 +@property(assign, nonatomic) int centerX; +@property(assign, nonatomic) int enbale2GBackgroundFetch; @end ---------------- enbale -> enable Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D45750 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits