diff --git a/src/main/cpp/util/file.cc b/src/main/cpp/util/file.cc index 392af372afd2f0..cae87198a7a10a 100644 --- a/src/main/cpp/util/file.cc +++ b/src/main/cpp/util/file.cc @@ -77,31 +77,6 @@ string Basename(const string &path) { return SplitPath(path).second; } -string JoinPath(const string &path1, const string &path2) { - if (path1.empty()) { - // "" + "/bar" - return path2; - } - - if (path1[path1.size() - 1] == '/') { - if (path2.find('/') == 0) { - // foo/ + /bar - return path1 + path2.substr(1); - } else { - // foo/ + bar - return path1 + path2; - } - } else { - if (path2.find('/') == 0) { - // foo + /bar - return path1 + path2; - } else { - // foo + bar - return path1 + "/" + path2; - } - } -} - class DirectoryTreeWalker : public DirectoryEntryConsumer { public: DirectoryTreeWalker(vector *files, diff --git a/src/main/cpp/util/file.h b/src/main/cpp/util/file.h index efe963c6d7ce8b..e3a744f652a5df 100644 --- a/src/main/cpp/util/file.h +++ b/src/main/cpp/util/file.h @@ -17,6 +17,7 @@ #include #include #include +#include "src/main/cpp/util/file_platform.h" namespace blaze_util { @@ -59,8 +60,6 @@ std::string Dirname(const std::string &path); // "/" in the path, the result is the same as the input. std::string Basename(const std::string &path); -std::string JoinPath(const std::string &path1, const std::string &path2); - // Lists all files in `path` and all of its subdirectories. // // Does not follow symlinks / junctions. diff --git a/src/main/cpp/util/file_platform.h b/src/main/cpp/util/file_platform.h index 600820ba0c5200..99cf99044b2ad6 100644 --- a/src/main/cpp/util/file_platform.h +++ b/src/main/cpp/util/file_platform.h @@ -50,6 +50,8 @@ class IFileMtime { // Creates a platform-specific implementation of `IFileMtime`. IFileMtime *CreateFileMtime(); +std::string JoinPath(const std::string &path1, const std::string &path2); + // Split a path to dirname and basename parts. std::pair SplitPath(const std::string &path); diff --git a/src/main/cpp/util/file_posix.cc b/src/main/cpp/util/file_posix.cc index a1d806a9d9fc6e..57c994a3fffa3e 100644 --- a/src/main/cpp/util/file_posix.cc +++ b/src/main/cpp/util/file_posix.cc @@ -168,6 +168,35 @@ IPipe* CreatePipe() { return new PosixPipe(fd[0], fd[1]); } +string JoinPath(const string &path1, const string &path2) { + if (path1.empty()) { + // "" + "/bar" + return path2; + } + if (path2.empty()) { + // "foo/" + "" + return path1; + } + + if (path1.back() == '/') { + if (path2.front() == '/') { + // foo/ + /bar + return path1 + path2.substr(1); + } else { + // foo/ + bar + return path1 + path2; + } + } else { + if (path2.front() == '/') { + // foo + /bar + return path1 + path2; + } else { + // foo + bar + return path1 + "/" + path2; + } + } +} + pair SplitPath(const string &path) { size_t pos = path.rfind('/'); diff --git a/src/main/cpp/util/file_windows.cc b/src/main/cpp/util/file_windows.cc index b1fadddf4af26e..2deee0b44a8d3c 100644 --- a/src/main/cpp/util/file_windows.cc +++ b/src/main/cpp/util/file_windows.cc @@ -323,6 +323,44 @@ static bool IsRootOrAbsolute(const basic_string& path, HasDriveSpecifierPrefix(path.c_str() + 4) && IsPathSeparator(path[6])); } +string JoinPath(const string& path1, const string& path2) { + if (path1.empty()) { + // "" + "/bar" + return path2; + } + if (path2.empty()) { + // "foo/" + "" + return path1; + } + + string p1 = path1; + if (path1.back() == '/') { + p1.pop_back(); + } + string p2 = path2; + if (path2.front() == '/') { + p2[0] = '\\'; + } + + if (IsPathSeparator(p1.back())) { + if (!p2.empty() && IsPathSeparator(p2.front())) { + // foo/ + /bar + return p1 + p2.substr(1); + } else { + // foo/ + bar + return p1 + p2; + } + } else { + if (!p2.empty() && IsPathSeparator(p2.front())) { + // foo + /bar + return p1 + p2; + } else { + // foo + bar + return p1 + "\\" + p2; + } + } +} + pair SplitPath(const string& path) { if (path.empty()) { return std::make_pair("", "");