[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