[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