This revision was automatically updated to reflect the committed changes.
Closed by commit rL330286: add extra acronyms for objc property names (authored
by Wizard, committed by ).
Herald added a subscriber: llvm-commits.
Repository:
rL LLVM
https://reviews.llvm.org/D45750
Files:
clang-tool
Wizard added inline comments.
Comment at: clang-tidy/objc/PropertyDeclarationCheck.cpp:42
constexpr llvm::StringLiteral DefaultSpecialAcronyms[] = {
+"[2-9]G",
"ACL",
benhamilton wrote:
> Probably should just make this:
>
> "\\d+G"
>
It seems llvm::
Wizard updated this revision to Diff 142985.
Wizard marked 2 inline comments as done.
Wizard added a comment.
resolve comments
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D45750
Files:
clang-tidy/objc/PropertyDeclarationCheck.cpp
test/clang-tidy/objc-property-declaration.
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",
-
Wizard created this revision.
Herald added subscribers: cfe-commits, klimek.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D45750
Files:
clang-tidy/objc/PropertyDeclarationCheck.cpp
test/clang-tidy/objc-property-declaration.m
Index: test/clang-tidy/objc-property-declaration
