[PATCH] D38639: [clangd] #include statements support for Open definition

[William Enright via Phabricator via cfe-commits]

Nebiroth updated this revision to Diff 127386.
Nebiroth marked 14 inline comments as done.
Nebiroth added a comment.
Herald added a subscriber: mgrang.

  CppFilePreambleCallbacks no longer extends PPCallbacks
  CppFilePreambleCallbacks no longer needs SourceManager parameter
  Removed temporary vector TempPreambleIncludeLocations
  IncludeReferenceMap in ParsedAST no longer passed by reference
  Code handling includes outside of preambles is now separated in IncludeRefsCollector class
  Removed addLocation
  Changed isSameLine function name and now checks if searched location is in range instead of verifying the line
  Removed changes to findDefinitions interface


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D38639

Files:
  clangd/ClangdServer.cpp
  clangd/ClangdUnit.cpp
  clangd/ClangdUnit.h
  clangd/Protocol.h
  unittests/clangd/ClangdTests.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D38639.127386.patch
Type: text/x-patch
Size: 18529 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20171218/2331e772/attachment-0001.bin>