[PATCH] D75786: [clang-tidy] Move fuchsia-restrict-system-includes to portability module for general use.

Paula Toth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 6 16:34:23 PST 2020


PaulkaToast created this revision.
PaulkaToast added a reviewer: aaron.ballman.
PaulkaToast added a project: clang-tools-extra.
Herald added subscribers: cfe-commits, phosek, xazax.hun, mgorny.
Herald added a project: clang.

Created a general check for restrict-system-includes under portability as recommend in the comments under D75332 <https://reviews.llvm.org/D75332>. I also fleshed out the user facing documentation to show examples for common use-cases such as allow-list, block-list, and wild carding.

In the documentation I noted that this new check was moved out from fuchsia and an alias was created from the old name to the new name to keep everything working. I'm wondering if this is correctly formatted in the release notes.

Nothing should change on fuchsia's end, although feel free to add them if a check off is required. (:


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D75786

Files:
  clang-tools-extra/clang-tidy/fuchsia/CMakeLists.txt
  clang-tools-extra/clang-tidy/fuchsia/FuchsiaTidyModule.cpp
  clang-tools-extra/clang-tidy/fuchsia/RestrictSystemIncludesCheck.cpp
  clang-tools-extra/clang-tidy/fuchsia/RestrictSystemIncludesCheck.h
  clang-tools-extra/clang-tidy/portability/CMakeLists.txt
  clang-tools-extra/clang-tidy/portability/PortabilityTidyModule.cpp
  clang-tools-extra/clang-tidy/portability/RestrictSystemIncludesCheck.cpp
  clang-tools-extra/clang-tidy/portability/RestrictSystemIncludesCheck.h
  clang-tools-extra/docs/ReleaseNotes.rst
  clang-tools-extra/docs/clang-tidy/checks/list.rst
  clang-tools-extra/docs/clang-tidy/checks/portability-restrict-system-includes.rst
  clang-tools-extra/test/clang-tidy/checkers/portability-restrict-system-includes-allow.cpp
  clang-tools-extra/test/clang-tidy/checkers/portability-restrict-system-includes-disallow.cpp
  clang-tools-extra/test/clang-tidy/checkers/portability-restrict-system-includes-glob.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D75786.248850.patch
Type: text/x-patch
Size: 12442 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200307/14a06345/attachment-0001.bin>


More information about the cfe-commits mailing list