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
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits