[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