simark added inline comments.
================ Comment at: clangd/ClangdLSPServer.cpp:302 +// FIXME: This function needs to be properly tested. +void ClangdLSPServer::onChangeConfiguration( ---------------- ilya-biryukov wrote: > Are you planning to to address this FIXME before checking the code in? Following what you said here: https://reviews.llvm.org/D39571?id=124024#inline-359345 I have not really looked into what was wrong with the test, and what is missing in the infrastructure to make it work. But I assumed that the situation did not change since then. Can you enlighten me on what the problem was, and what is missing? ================ Comment at: clangd/GlobalCompilationDatabase.h:64 + void setCompileCommandsDir(Path P) { + std::lock_guard<std::mutex> Lock(Mutex); ---------------- ilya-biryukov wrote: > Maybe move definition to `.cpp` file? > > Will do. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D39571 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits