[libcxx-commits] [libcxx] [libc++] Fix different slashes confuses lexically_proximate and lexically_relative (PR #99780)

via libcxx-commits libcxx-commits at lists.llvm.org
Sat Jul 20 23:09:38 PDT 2024


github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {clang-format}-->


:warning: C/C++ code formatter, clang-format found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
git-clang-format --diff 4d8e42ea6a89c73f90941fd1b6e899912e31dd34 572ed17cb895614a7f9855fe5592a38b73f88c95 --extensions cpp,h -- libcxx/src/filesystem/path.cpp libcxx/src/filesystem/path_parser.h libcxx/test/std/input.output/filesystems/class.path/path.member/path.decompose/path.decompose.pass.cpp
``````````

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/libcxx/test/std/input.output/filesystems/class.path/path.member/path.decompose/path.decompose.pass.cpp b/libcxx/test/std/input.output/filesystems/class.path/path.member/path.decompose/path.decompose.pass.cpp
index 9a1b581fa1..df84de3dd9 100644
--- a/libcxx/test/std/input.output/filesystems/class.path/path.member/path.decompose/path.decompose.pass.cpp
+++ b/libcxx/test/std/input.output/filesystems/class.path/path.member/path.decompose/path.decompose.pass.cpp
@@ -76,47 +76,49 @@ struct PathDecomposeTestcase
     std::string filename;
 };
 
-const PathDecomposeTestcase PathTestCases[] =
-  {
-      {"", {}, "", "", "", "", "", ""}
-    , {".", {"."}, "", "", "", ".", "", "."}
-    , {"..", {".."}, "", "", "", "..", "", ".."}
-    , {"foo", {"foo"}, "", "", "", "foo", "", "foo"}
-    , {"/", {"/"}, "/", "", "/", "", "/", ""}
-    , {"/foo", {"/", "foo"}, "/", "", "/", "foo", "/", "foo"}
-    , {"foo/", {"foo", ""}, "", "", "", "foo/", "foo", ""}
-    , {"/foo/", {"/", "foo", ""}, "/", "", "/", "foo/", "/foo", ""}
-    , {"foo/bar", {"foo","bar"}, "",  "", "",  "foo/bar", "foo", "bar"}
-    , {"/foo//bar", {"/","foo","bar"}, "/", "", "/", "foo/bar", "/foo", "bar"}
+const PathDecomposeTestcase PathTestCases[] = {
+    {"", {}, "", "", "", "", "", ""},
+    {".", {"."}, "", "", "", ".", "", "."},
+    {"..", {".."}, "", "", "", "..", "", ".."},
+    {"foo", {"foo"}, "", "", "", "foo", "", "foo"},
+    {"/", {"/"}, "/", "", "/", "", "/", ""},
+    {"/foo", {"/", "foo"}, "/", "", "/", "foo", "/", "foo"},
+    {"foo/", {"foo", ""}, "", "", "", "foo/", "foo", ""},
+    {"/foo/", {"/", "foo", ""}, "/", "", "/", "foo/", "/foo", ""},
+    {"foo/bar", {"foo", "bar"}, "", "", "", "foo/bar", "foo", "bar"},
+    {"/foo//bar", {"/", "foo", "bar"}, "/", "", "/", "foo/bar", "/foo", "bar"}
 #ifdef _WIN32
-    , {"//net", {"//net"}, "//net", "//net", "", "", "//net", ""}
-    , {"//net/", {"//net", "/"}, "//net/", "//net", "/", "", "//net/", ""}
-    , {"//net/foo", {"//net", "/", "foo"}, "//net/", "//net", "/", "foo", "//net/", "foo"}
+    ,
+    {"//net", {"//net"}, "//net", "//net", "", "", "//net", ""},
+    {"//net/", {"//net", "/"}, "//net/", "//net", "/", "", "//net/", ""},
+    {"//net/foo", {"//net", "/", "foo"}, "//net/", "//net", "/", "foo", "//net/", "foo"}
 #else
     , {"//net", {"/", "net"}, "/", "", "/", "net", "/", "net"}
     , {"//net/", {"/", "net", ""}, "/", "", "/", "net/", "//net", ""}
     , {"//net/foo", {"/", "net", "foo"}, "/", "", "/", "net/foo", "/net", "foo"}
 #endif
-    , {"///foo///", {"/", "foo", ""}, "/", "", "/", "foo///", "///foo", ""}
-    , {"///foo///bar", {"/", "foo", "bar"}, "/", "", "/", "foo///bar", "///foo", "bar"}
-    , {"/.", {"/", "."}, "/", "", "/", ".", "/", "."}
-    , {"./", {".", ""}, "", "", "", "./", ".", ""}
-    , {"/..", {"/", ".."}, "/", "", "/", "..", "/", ".."}
-    , {"../", {"..", ""}, "", "", "", "../", "..", ""}
-    , {"foo/.", {"foo", "."}, "", "", "", "foo/.", "foo", "."}
-    , {"foo/..", {"foo", ".."}, "", "", "", "foo/..", "foo", ".."}
-    , {"foo/./", {"foo", ".", ""}, "", "", "", "foo/./", "foo/.", ""}
-    , {"foo/./bar", {"foo", ".", "bar"}, "", "", "", "foo/./bar", "foo/.", "bar"}
-    , {"foo/../", {"foo", "..", ""}, "", "", "", "foo/../", "foo/..", ""}
-    , {"foo/../bar", {"foo", "..", "bar"}, "", "", "", "foo/../bar", "foo/..", "bar"}
+    ,
+    {"///foo///", {"/", "foo", ""}, "/", "", "/", "foo///", "///foo", ""},
+    {"///foo///bar", {"/", "foo", "bar"}, "/", "", "/", "foo///bar", "///foo", "bar"},
+    {"/.", {"/", "."}, "/", "", "/", ".", "/", "."},
+    {"./", {".", ""}, "", "", "", "./", ".", ""},
+    {"/..", {"/", ".."}, "/", "", "/", "..", "/", ".."},
+    {"../", {"..", ""}, "", "", "", "../", "..", ""},
+    {"foo/.", {"foo", "."}, "", "", "", "foo/.", "foo", "."},
+    {"foo/..", {"foo", ".."}, "", "", "", "foo/..", "foo", ".."},
+    {"foo/./", {"foo", ".", ""}, "", "", "", "foo/./", "foo/.", ""},
+    {"foo/./bar", {"foo", ".", "bar"}, "", "", "", "foo/./bar", "foo/.", "bar"},
+    {"foo/../", {"foo", "..", ""}, "", "", "", "foo/../", "foo/..", ""},
+    {"foo/../bar", {"foo", "..", "bar"}, "", "", "", "foo/../bar", "foo/..", "bar"}
 #ifdef _WIN32
-    , {"c:", {"c:"}, "c:", "c:", "", "", "c:", ""}
-    , {"c:/", {"c:", "/"}, "c:/", "c:", "/", "", "c:/", ""}
-    , {"c:foo", {"c:", "foo"}, "c:", "c:", "", "foo", "c:", "foo"}
-    , {"c:/foo", {"c:", "/", "foo"}, "c:/", "c:", "/", "foo", "c:/", "foo"}
-    , {"c:foo/", {"c:", "foo", ""}, "c:", "c:", "", "foo/", "c:foo", ""}
-    , {"c:/foo/", {"c:", "/", "foo", ""}, "c:/", "c:", "/", "foo/",  "c:/foo", ""}
-    , {"c:/foo/bar", {"c:", "/", "foo", "bar"}, "c:/", "c:", "/", "foo/bar", "c:/foo", "bar"}
+    ,
+    {"c:", {"c:"}, "c:", "c:", "", "", "c:", ""},
+    {"c:/", {"c:", "/"}, "c:/", "c:", "/", "", "c:/", ""},
+    {"c:foo", {"c:", "foo"}, "c:", "c:", "", "foo", "c:", "foo"},
+    {"c:/foo", {"c:", "/", "foo"}, "c:/", "c:", "/", "foo", "c:/", "foo"},
+    {"c:foo/", {"c:", "foo", ""}, "c:", "c:", "", "foo/", "c:foo", ""},
+    {"c:/foo/", {"c:", "/", "foo", ""}, "c:/", "c:", "/", "foo/", "c:/foo", ""},
+    {"c:/foo/bar", {"c:", "/", "foo", "bar"}, "c:/", "c:", "/", "foo/bar", "c:/foo", "bar"}
 #else
     , {"c:", {"c:"}, "", "", "", "c:", "", "c:"}
     , {"c:/", {"c:", ""}, "", "", "", "c:/", "c:", ""}
@@ -126,14 +128,16 @@ const PathDecomposeTestcase PathTestCases[] =
     , {"c:/foo/", {"c:", "foo", ""}, "", "", "", "c:/foo/",  "c:/foo", ""}
     , {"c:/foo/bar", {"c:", "foo", "bar"}, "", "", "", "c:/foo/bar", "c:/foo", "bar"}
 #endif
-    , {"prn:", {"prn:"}, "", "", "", "prn:", "", "prn:"}
+    ,
+    {"prn:", {"prn:"}, "", "", "", "prn:", "", "prn:"}
 #ifdef _WIN32
-    , {"c:\\", {"c:", "/"}, "c:\\", "c:", "\\", "", "c:\\", ""}
-    , {"c:\\foo", {"c:", "/", "foo"}, "c:\\", "c:", "\\", "foo", "c:\\", "foo"}
-    , {"c:foo\\", {"c:", "foo", ""}, "c:", "c:", "", "foo\\", "c:foo", ""}
-    , {"c:\\foo\\", {"c:", "/", "foo", ""}, "c:\\", "c:", "\\", "foo\\", "c:\\foo", ""}
-    , {"c:\\foo/",  {"c:", "/", "foo", ""}, "c:\\", "c:", "\\", "foo/", "c:\\foo", ""}
-    , {"c:/foo\\bar", {"c:", "/", "foo", "bar"}, "c:\\", "c:", "\\", "foo\\bar", "c:/foo", "bar"}
+    ,
+    {"c:\\", {"c:", "/"}, "c:\\", "c:", "\\", "", "c:\\", ""},
+    {"c:\\foo", {"c:", "/", "foo"}, "c:\\", "c:", "\\", "foo", "c:\\", "foo"},
+    {"c:foo\\", {"c:", "foo", ""}, "c:", "c:", "", "foo\\", "c:foo", ""},
+    {"c:\\foo\\", {"c:", "/", "foo", ""}, "c:\\", "c:", "\\", "foo\\", "c:\\foo", ""},
+    {"c:\\foo/", {"c:", "/", "foo", ""}, "c:\\", "c:", "\\", "foo/", "c:\\foo", ""},
+    {"c:/foo\\bar", {"c:", "/", "foo", "bar"}, "c:\\", "c:", "\\", "foo\\bar", "c:/foo", "bar"}
 #else
     , {"c:\\", {"c:\\"}, "", "", "", "c:\\", "", "c:\\"}
     , {"c:\\foo", {"c:\\foo"}, "", "", "", "c:\\foo", "", "c:\\foo"}
@@ -142,8 +146,8 @@ const PathDecomposeTestcase PathTestCases[] =
     , {"c:\\foo/",  {"c:\\foo", ""}, "", "", "", "c:\\foo/", "c:\\foo", ""}
     , {"c:/foo\\bar", {"c:", "foo\\bar"}, "", "", "", "c:/foo\\bar", "c:", "foo\\bar"}
 #endif
-    , {"//", {"/"}, "/", "", "/", "", "/", ""}
-  };
+    ,
+    {"//", {"/"}, "/", "", "/", "", "/", ""}};
 
 void decompPathTest()
 {

``````````

</details>


https://github.com/llvm/llvm-project/pull/99780


More information about the libcxx-commits mailing list