This should also fix the current windows buildbot breakage
(http://lab.llvm.org:8011/builders/clang-x86-windows-msvc2015/builds/9838/steps/ninja%20check%201/logs/stdio)
Details
Details
- Reviewers
sammccall - Commits
- rG5740ff5bed8a: [clangd] Add a test URI scheme for lit tests to unbreak platform-specific URI…
rL323885: [clangd] Add a test URI scheme for lit tests to unbreak platform-specific URI…
rCTE323885: [clangd] Add a test URI scheme for lit tests to unbreak platform-specific URI…
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
Thanks for fixing this once and for all!
clangd/ClangdLSPServer.cpp | ||
---|---|---|
23 ↗ | (On Diff #132167) | nit: URi -> URI |
28 ↗ | (On Diff #132167) | nit: just inline this at the registry site? we shouldn't be using it elsewhere |
47 ↗ | (On Diff #132167) | on windows you're passing C:\clangd-test and /foo.cpp here. |
clangd/Protocol.cpp | ||
36 ↗ | (On Diff #132167) | I'd suggest not mentioning 'test' in the log message, and just making this a comment. |