[PATCH] D43778: [clang-tidy] Adding RestrictIncludes check to Fuchsia module

Julie Hockett via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 11 11:43:13 PDT 2018


juliehockett added inline comments.


================
Comment at: clang-tidy/fuchsia/RestrictSystemIncludesCheck.h:35
+  void storeOptions(ClangTidyOptions::OptionMap &Opts) override;
+  bool contains(StringRef FileName) {
+    return AllowedIncludesGlobList.contains(FileName);
----------------
aaron.ballman wrote:
> Function can be marked `const`.
GlobList::contains isn't `const`, so it can't...


https://reviews.llvm.org/D43778





More information about the cfe-commits mailing list