The Acronyms and IncludeDefaultAcronyms options were deprecated in
https://reviews.llvm.org/D51832. These options can be removed.
Tested by running the clang-tidy tests.
This is an amended resubmission of https://reviews.llvm.org/D56945.
Paths
| Differential D57080
[clang-tidy] Delete obsolete objc-property-declaration options ✂️ ClosedPublic Authored by stephanemoore on Jan 22 2019, 7:56 PM.
Details Summary The Acronyms and IncludeDefaultAcronyms options were deprecated in Tested by running the clang-tidy tests. This is an amended resubmission of https://reviews.llvm.org/D56945.
Diff Detail
Event Timelinestephanemoore added inline comments.
This revision is now accepted and ready to land.Feb 13 2019, 6:34 AM Closed by commit rL354195: [clang-tidy] Delete obsolete objc-property-declaration options ✂️ (authored by stephanemoore). · Explain WhyFeb 15 2019, 8:28 PM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 183027 clang-tidy/objc/PropertyDeclarationCheck.h
clang-tidy/objc/PropertyDeclarationCheck.cpp
docs/ReleaseNotes.rstdocs/clang-tidy/checks/objc-property-declaration.rst
|
In https://reviews.llvm.org/D56945, I implemented the recommendation to use the :option: prefix for these options but the documentation generator produced a warning for an unknown option which I imagine is due to the deletion of the options. I imagine that means we can't use the :option: prefix for deleted options? Please let me know if you have other suggestions.