[PATCH] D34636: [clangd] LLVM-ify codebase

Kirill Bobyrev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 26 11:43:58 PDT 2017


omtcyfz created this revision.

This patch introduces cosmetic changes while making ClangD code slightly more LLVM Coding Standards-compliant by

- Convert names of struct fields in Protocol.h from `camelCase` to `CamelCase`
- Enclose code in .cpp implementation files in appropriate namespaces instead of doing `using namespace clang; using namespace clangd;`
- Putting few `const`s and references where appropriate

Testing:
$ ninja check-clang-tools
All ClangD-related tests are green.


https://reviews.llvm.org/D34636

Files:
  clangd/ClangdLSPServer.cpp
  clangd/ClangdServer.cpp
  clangd/ClangdServer.h
  clangd/ClangdUnit.cpp
  clangd/ClangdUnit.h
  clangd/ClangdUnitStore.cpp
  clangd/DraftStore.cpp
  clangd/GlobalCompilationDatabase.cpp
  clangd/JSONRPCDispatcher.cpp
  clangd/Protocol.cpp
  clangd/Protocol.h
  clangd/ProtocolHandlers.cpp
  clangd/tool/ClangdMain.cpp
  unittests/clangd/ClangdTests.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D34636.103900.patch
Type: text/x-patch
Size: 43892 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170626/a5e422a9/attachment-0001.bin>


More information about the cfe-commits mailing list