[PATCH] D42261: [clang-tidy objc-property-declaration] New option AdditionalAcronyms

Ben Hamilton via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 19 10:31:16 PST 2018


benhamilton updated this revision to Diff 130657.
benhamilton marked 2 inline comments as done.
benhamilton added a comment.

- Switch to IncludeDefaultAcronyms option (defaults to 1).
- Use array for default acronyms, since we no longer need to parse it.
- Don't regex-escape default acronyms, since we control them.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D42261

Files:
  clang-tidy/objc/PropertyDeclarationCheck.cpp
  clang-tidy/objc/PropertyDeclarationCheck.h
  docs/clang-tidy/checks/objc-property-declaration.rst
  test/clang-tidy/objc-property-declaration-additional.m
  test/clang-tidy/objc-property-declaration-custom.m

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D42261.130657.patch
Type: text/x-patch
Size: 9264 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180119/43eb0264/attachment-0001.bin>


More information about the cfe-commits mailing list