[clang-tidy objc-property-declaration] New option IncludeDefaultAcronyms
authorBen Hamilton <benhamilton@google.com>
Mon, 22 Jan 2018 15:45:25 +0000 (15:45 +0000)
committerBen Hamilton <benhamilton@google.com>
Mon, 22 Jan 2018 15:45:25 +0000 (15:45 +0000)
commitf94c10d91a95b0fe5d529b72280b7f52773ed740
tree3e2359a3562989cfabc7dadb0f4f38974312f7c0
parent145d63f1ada934f82af0c0796d3ba21461270e47
[clang-tidy objc-property-declaration] New option IncludeDefaultAcronyms

Summary:
The existing option objc-property-declaration.Acronyms
replaces the built-in set of acronyms.

While this behavior is OK for clients that don't want the default
behavior, many clients may just want to add their own custom acronyms
to the default list.

This revision introduces a new option,
objc-property-declaration.IncludeDefaultAcronyms, which controls
whether the acronyms in objc-property-declaration.Acronyms are
appended to the default list (the default behavior) or whether they
replace.

I also updated the documentation.

Test Plan: make -j12 check-clang-tools

Reviewers: Wizard, hokein, klimek

Reviewed By: hokein

Subscribers: Eugene.Zelenko, cfe-commits

Differential Revision: https://reviews.llvm.org/D42261

llvm-svn: 323130
clang-tools-extra/clang-tidy/objc/PropertyDeclarationCheck.cpp
clang-tools-extra/clang-tidy/objc/PropertyDeclarationCheck.h
clang-tools-extra/docs/clang-tidy/checks/objc-property-declaration.rst
clang-tools-extra/test/clang-tidy/objc-property-declaration-additional.m [new file with mode: 0644]
clang-tools-extra/test/clang-tidy/objc-property-declaration-custom.m