aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land.
Thank you for adding the glob feature, I think that will be very powerful in practice. Aside from a minor nit, this LGTM! ================ Comment at: clang-tidy/fuchsia/RestrictSystemIncludesCheck.h:35 + void storeOptions(ClangTidyOptions::OptionMap &Opts) override; + bool contains(StringRef FileName) { + return AllowedIncludesGlobList.contains(FileName); ---------------- Function can be marked `const`. https://reviews.llvm.org/D43778 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits