[PATCH] D54845: [clangd] Canonicalize file path in URIForFile.

Eric Liu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 23 02:24:07 PST 2018


ioeric created this revision.
ioeric added a reviewer: sammccall.
Herald added subscribers: cfe-commits, kadircet, arphaman, jkorous, MaskRay, ilya-biryukov.

File paths in URIForFile can come from index or local AST. Path from
index goes through URI transformation and the final path is resolved by URI
scheme and could be potentially different from the original path. Hence, we
should do the same transformation for all paths. We do this in URIForFile, which
now converts a path to URI and back to a canonicalized path.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D54845

Files:
  clangd/FindSymbols.cpp
  clangd/Protocol.cpp
  clangd/Protocol.h
  clangd/URI.cpp
  clangd/URI.h
  clangd/XRefs.cpp
  unittests/clangd/ClangdTests.cpp
  unittests/clangd/ClangdUnitTests.cpp
  unittests/clangd/TestFS.cpp
  unittests/clangd/URITests.cpp
  unittests/clangd/XRefsTests.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D54845.175094.patch
Type: text/x-patch
Size: 13698 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20181123/4c5a6950/attachment-0001.bin>


More information about the cfe-commits mailing list