[PATCH] D43778: [clang-tidy] Adding RestrictIncludes check to Fuchsia module
Aaron Ballman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri May 11 12:01:25 PDT 2018
aaron.ballman 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);
----------------
juliehockett wrote:
> aaron.ballman wrote:
> > Function can be marked `const`.
> GlobList::contains isn't `const`, so it can't...
Bah, I didn't see that. You can ignore, sorry for the noise.
https://reviews.llvm.org/D43778
More information about the cfe-commits
mailing list