[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