[PATCH] D82024: [clangd] Rename FSProvider to TFS in case of ThreadsafeFS

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 17 11:18:51 PDT 2020


sammccall accepted this revision.
sammccall added inline comments.
This revision is now accepted and ready to land.


================
Comment at: clang-tools-extra/clangd/unittests/ParsedASTTests.cpp:252
 TEST(ParsedASTTest, CanBuildInvocationWithUnknownArgs) {
   MockFSProvider FSProvider;
   FSProvider.Files = {{testPath("foo.cpp"), "void test() {}"}};
----------------
These are still FSProvider... would suggest MockFS FS


================
Comment at: clang-tools-extra/clangd/unittests/TestTU.cpp:23
 
 ParseInputs TestTU::inputs(MockFSProvider &FSProvider) const {
   std::string FullFilename = testPath(Filename),
----------------
Still seem to be a few FSProviders here


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D82024/new/

https://reviews.llvm.org/D82024





More information about the cfe-commits mailing list