sammccall accepted this revision. sammccall added inline comments. This revision is now accepted and ready to land.
================ Comment at: unittests/clangd/TestFS.cpp:93 return URI(Scheme, /*Authority=*/"", - llvm::sys::path::convert_to_slash(Body)); + StringRef(llvm::sys::path::convert_to_slash(Body)).ltrim('/')); } ---------------- ioeric wrote: > sammccall wrote: > > why this change? > The `consume_front` above would not consume the second slash in > `/clangd-test/x.h` which results in "/x.h" in the body, but that doesn't work > with `testPath()` which requires a relative path. That's why I added a > requirement that the body should be relative path. can we switch back to that path being represented by `unittest:/x.h` rather than `unittest:x.h`? That looks more like the URI schemes we use in practice. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D48290 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits