[PATCH] D37972: [clangd] Introduced Logger interface.

Ilya Biryukov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 18 07:42:33 PDT 2017


ilya-biryukov updated this revision to Diff 115648.
ilya-biryukov marked 4 inline comments as done.
ilya-biryukov added a comment.

Addressed review comments.

- Replaced include with forward declaration in headers.
- Removed logging on each call to loadFromDirectory.


https://reviews.llvm.org/D37972

Files:
  clangd/CMakeLists.txt
  clangd/ClangdLSPServer.cpp
  clangd/ClangdServer.cpp
  clangd/ClangdServer.h
  clangd/ClangdUnit.cpp
  clangd/ClangdUnit.h
  clangd/ClangdUnitStore.cpp
  clangd/ClangdUnitStore.h
  clangd/GlobalCompilationDatabase.cpp
  clangd/GlobalCompilationDatabase.h
  clangd/JSONRPCDispatcher.h
  clangd/Logger.cpp
  clangd/Logger.h
  unittests/clangd/ClangdTests.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D37972.115648.patch
Type: text/x-patch
Size: 24164 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170918/21c834a5/attachment-0001.bin>


More information about the cfe-commits mailing list