================
@@ -465,14 +465,9 @@ TEST(RemoteMarshallingTest, CrossPlatformPathsRoundTrip) {
Location.End.setLine(3);
Location.End.setColumn(4);
// Construct the URI as a Windows machine would have serialized it into the
- // index: file:///C:/remote/project/lib/File.cpp.
+ // index: file:///C:\remote\project\lib\File.cpp.
Location.FileURI =
- Strings
- .save("file:///" +
- convert_to_slash(RemoteIndexRoot,
- llvm::sys::path::Style::windows) +
- "lib/File.cpp")
- .begin();
+ Strings.save("file:///" + RemoteIndexRoot + "lib\\File.cpp").begin();
----------------
ArcsinX wrote:
`file:///C:\remote\project\lib\File.cpp` is not a valid file URI. So, I don't
think that we need to test such cases.
https://github.com/llvm/llvm-project/pull/217745
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
