hokein added inline comments.
================
Comment at: clang-tidy/objc/PropertyDeclarationCheck.h:38
const std::vector<std::string> SpecialAcronyms;
+ const std::vector<std::string> AdditionalAcronyms;
};
----------------
nit: code indent
================
Comment at: docs/clang-tidy/checks/objc-property-declaration.rst:47
+
+.. option:: AdditionalAcronyms
+
----------------
It seems to me the `Acronyms` and `AdditionalAcronyms` play most same role
here.
If we want to keep the long default list, how about using a bool flag
`IncludeDefaultList` + the existing `Acronyms` option?
* if `IncludeDefaultList` is on, the acronyms will be "default" + "Acronyms".
* if `IncludeDefaultList` is off, the acronyms will be only "Acronyms".
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D42261
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
