[all-commits] [llvm/llvm-project] ebdb98: [clang-tidy] Move fuchsia-restrict-system-includes...

[Paula Toth via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: ebdb98f254f632b506109b9d20c6e8e19697765f
      https://github.com/llvm/llvm-project/commit/ebdb98f254f632b506109b9d20c6e8e19697765f
  Author: Paula Toth <paulatoth at google.com>
  Date:   2020-03-10 (Tue, 10 Mar 2020)

  Changed paths:
    M clang-tools-extra/clang-tidy/fuchsia/CMakeLists.txt
    M clang-tools-extra/clang-tidy/fuchsia/FuchsiaTidyModule.cpp
    R clang-tools-extra/clang-tidy/fuchsia/RestrictSystemIncludesCheck.cpp
    R clang-tools-extra/clang-tidy/fuchsia/RestrictSystemIncludesCheck.h
    M clang-tools-extra/clang-tidy/portability/CMakeLists.txt
    M clang-tools-extra/clang-tidy/portability/PortabilityTidyModule.cpp
    A clang-tools-extra/clang-tidy/portability/RestrictSystemIncludesCheck.cpp
    A clang-tools-extra/clang-tidy/portability/RestrictSystemIncludesCheck.h
    M clang-tools-extra/docs/ReleaseNotes.rst
    R clang-tools-extra/docs/clang-tidy/checks/fuchsia-restrict-system-includes.rst
    M clang-tools-extra/docs/clang-tidy/checks/list.rst
    A clang-tools-extra/docs/clang-tidy/checks/portability-restrict-system-includes.rst
    R clang-tools-extra/test/clang-tidy/checkers/Inputs/fuchsia-restrict-system-includes/a.h
    R clang-tools-extra/test/clang-tidy/checkers/Inputs/fuchsia-restrict-system-includes/system/cstdarg.h
    R clang-tools-extra/test/clang-tidy/checkers/Inputs/fuchsia-restrict-system-includes/system/cstdlib.h
    R clang-tools-extra/test/clang-tidy/checkers/Inputs/fuchsia-restrict-system-includes/system/j.h
    R clang-tools-extra/test/clang-tidy/checkers/Inputs/fuchsia-restrict-system-includes/system/r.h
    R clang-tools-extra/test/clang-tidy/checkers/Inputs/fuchsia-restrict-system-includes/system/s.h
    R clang-tools-extra/test/clang-tidy/checkers/Inputs/fuchsia-restrict-system-includes/system/t.h
    R clang-tools-extra/test/clang-tidy/checkers/Inputs/fuchsia-restrict-system-includes/system/transitive.h
    R clang-tools-extra/test/clang-tidy/checkers/Inputs/fuchsia-restrict-system-includes/transitive2.h
    A clang-tools-extra/test/clang-tidy/checkers/Inputs/portability-restrict-system-includes/a.h
    A clang-tools-extra/test/clang-tidy/checkers/Inputs/portability-restrict-system-includes/system/cstdarg.h
    A clang-tools-extra/test/clang-tidy/checkers/Inputs/portability-restrict-system-includes/system/cstdlib.h
    A clang-tools-extra/test/clang-tidy/checkers/Inputs/portability-restrict-system-includes/system/j.h
    A clang-tools-extra/test/clang-tidy/checkers/Inputs/portability-restrict-system-includes/system/r.h
    A clang-tools-extra/test/clang-tidy/checkers/Inputs/portability-restrict-system-includes/system/s.h
    A clang-tools-extra/test/clang-tidy/checkers/Inputs/portability-restrict-system-includes/system/t.h
    A clang-tools-extra/test/clang-tidy/checkers/Inputs/portability-restrict-system-includes/system/transitive.h
    A clang-tools-extra/test/clang-tidy/checkers/Inputs/portability-restrict-system-includes/transitive2.h
    R clang-tools-extra/test/clang-tidy/checkers/fuchsia-restrict-system-includes-all.cpp
    R clang-tools-extra/test/clang-tidy/checkers/fuchsia-restrict-system-includes-glob.cpp
    R clang-tools-extra/test/clang-tidy/checkers/fuchsia-restrict-system-includes-headers.cpp
    R clang-tools-extra/test/clang-tidy/checkers/fuchsia-restrict-system-includes.cpp
    A clang-tools-extra/test/clang-tidy/checkers/portability-restrict-system-includes-allow.cpp
    A clang-tools-extra/test/clang-tidy/checkers/portability-restrict-system-includes-disallow.cpp
    A clang-tools-extra/test/clang-tidy/checkers/portability-restrict-system-includes-glob.cpp
    A clang-tools-extra/test/clang-tidy/checkers/portability-restrict-system-includes-transitive.cpp

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

Summary:
Created a general check for restrict-system-includes under portability as recommend in the comments under 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.

Removed fuchsia's check as per phosek sugguestion.

Reviewers: aaron.ballman, phosek, alexfh, hokein, njames93

Reviewed By: phosek

Subscribers: Eugene.Zelenko, mgorny, xazax.hun, phosek, cfe-commits, MaskRay

Tags: #clang-tools-extra, #clang

Differential Revision: https://reviews.llvm.org/D75786