sammccall accepted this revision.
sammccall added a comment.
This revision is now accepted and ready to land.

Thanks, and sorry for the delay!



================
Comment at: clangd/Headers.cpp:45
 private:
-  std::set<std::string> &Headers;
+  std::set<std::string> &WrittenHeaders;
+  std::set<std::string> &ResolvedHeaders;
----------------
nit: StringSet? std::set is pretty bad unless we need ordering


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D43510



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to