[PATCH] D55415: Revert removal of tidy plugin support from libclang
Ivan Donchevskii via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Dec 7 00:16:06 PST 2018
yvvan created this revision.
yvvan added reviewers: bkramer, arphaman, nik.
Herald added subscribers: kadircet, jkorous, ioeric, javed.absar, ilya-biryukov.
libclang has nothing to do with clangd so I don't see why having the last one is the reason to remove features from libclang. Especially without reviews.
https://reviews.llvm.org/D55415
Files:
unittests/clangd/TUSchedulerTests.cpp
Index: unittests/clangd/TUSchedulerTests.cpp
===================================================================
--- unittests/clangd/TUSchedulerTests.cpp
+++ unittests/clangd/TUSchedulerTests.cpp
@@ -709,29 +709,6 @@
TUState(TUAction::Idle, /*No action*/ "")));
}
-TEST_F(TUSchedulerTests, NoTUStatusEmittedForRemovedFile) {
- class CaptureTUStatus : public DiagnosticsConsumer {
- public:
- void onDiagnosticsReady(PathRef File,
- std::vector<Diag> Diagnostics) override {}
-
- void onFileUpdated(PathRef File, const TUStatus &Status) override {
- // Block the worker thread until the document is removed.
- Removed.wait();
- }
- Notification Removed;
- } CaptureTUStatus;
- MockFSProvider FS;
- MockCompilationDatabase CDB;
- ClangdServer Server(CDB, FS, CaptureTUStatus, ClangdServer::optsForTest());
-
- Server.addDocument(testPath("foo.cpp"), "int main() {}",
- WantDiagnostics::Yes);
- Server.removeDocument(testPath("foo.cpp"));
- CaptureTUStatus.Removed.notify();
- ASSERT_TRUE(Server.blockUntilIdleForTest()) << "Waiting for finishing";
-}
-
} // namespace
} // namespace clangd
} // namespace clang
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D55415.177133.patch
Type: text/x-patch
Size: 1214 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20181207/a290329b/attachment.bin>
More information about the cfe-commits
mailing list