[PATCH] D56945: [clang-tidy] Delete obsolete objc-property-declaration options ✂️
Stephane Moore via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jan 22 19:45:44 PST 2019
stephanemoore marked an inline comment as done.
stephanemoore added inline comments.
================
Comment at: docs/ReleaseNotes.rst:248
+- The `Acronyms` and `IncludeDefaultAcronyms` options for the
+ :doc:`objc-property-declaration<clang-tidy/checks/objc-property-declaration>`
----------------
Eugene.Zelenko wrote:
> Please rebase from trunk and use :option: prefix for options..
Could using the :option: prefix cause issues if I am removing the option?
The documentation generation seems to emit a warning for an unknown option:
"llvm/src/tools/clang/tools/extra/docs/ReleaseNotes.rst:76: WARNING: unknown option: Acronyms"
http://lab.llvm.org:8011/builders/clang-tools-sphinx-docs/builds/36425/steps/docs-clang-tools-html/logs/stdio
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D56945/new/
https://reviews.llvm.org/D56945
More information about the cfe-commits
mailing list