[PATCH] D43510: [clangd] don't insert new includes if either original header or canonical header is already included.

Eric Liu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 20 09:02:19 PST 2018


ioeric updated this revision to Diff 135074.
ioeric marked 16 inline comments as done.
ioeric added a comment.

- Addressed review comments.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D43510

Files:
  clangd/ClangdLSPServer.cpp
  clangd/ClangdServer.cpp
  clangd/ClangdServer.h
  clangd/CodeComplete.cpp
  clangd/Headers.cpp
  clangd/Headers.h
  clangd/Protocol.cpp
  clangd/Protocol.h
  test/clangd/insert-include.test
  unittests/clangd/ClangdTests.cpp
  unittests/clangd/HeadersTests.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D43510.135074.patch
Type: text/x-patch
Size: 22882 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180220/1eebe99a/attachment-0001.bin>


More information about the cfe-commits mailing list