ioeric added inline comments.
================ Comment at: unittests/clangd/SyncAPI.h:8 +// +//===---------------------------------------------------------------------===// +#ifndef LLVM_CLANG_TOOLS_EXTRA_UNITTESTS_CLANGD_SYNCAPI_H ---------------- ilya-biryukov wrote: > ioeric wrote: > > Any reason not to put sync APIs in ClangdServer by their async versions, so > > that they are easier to find? I might be missing context. But if there is a > > good reason, please explain in the file doc. > That's a good question. I don't think we want anyone using the sync versions > of the API. The tests are somewhat special, though, as they'll get really > complicated if we move everything to callbacks and we don't want that. > > This probably deserves a comment. If these are expected to be used by tests only, I'd suggest moving the file to the test directory. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D43068 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
