From 32accc160762ca80f1138d4e9c710d9edb72479c Mon Sep 17 00:00:00 2001 From: Andy Cedilnik Date: Fri, 26 Dec 2003 15:00:27 -0500 Subject: BUG: For consistency, use cmStdString. Also, there was a bug in SplitString which make it lose the first character. --- Source/cmSystemTools.cxx | 14 +++++++------- Source/cmSystemTools.h | 2 +- 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/Source/cmSystemTools.cxx b/Source/cmSystemTools.cxx index 5c4f307..9716dc6 100644 --- a/Source/cmSystemTools.cxx +++ b/Source/cmSystemTools.cxx @@ -1066,19 +1066,19 @@ bool cmSystemTools::CreateSymlink(const char* origName, const char* newName) #endif -std::vector cmSystemTools::SplitString(const char* p, char sep) +std::vector cmSystemTools::SplitString(const char* p, char sep) { std::string path = p; - std::vector paths; + std::vector paths; std::string::size_type pos1 = 0; std::string::size_type pos2 = path.find(sep, pos1+1); while(pos2 != std::string::npos) { - paths.push_back(path.substr(pos1+1, pos2-pos1-1)); - pos1 = pos2; + paths.push_back(path.substr(pos1, pos2-pos1)); + pos1 = pos2+1; pos2 = path.find(sep, pos1+1); } - paths.push_back(path.substr(pos1+1, pos2-pos1-1)); + paths.push_back(path.substr(pos1, pos2-pos1)); return paths; } @@ -1104,8 +1104,8 @@ std::string cmSystemTools::RelativePath(const char* local, const char* remote) } std::string relativePath; // result string // split up both paths into arrays of strings using / as a separator - std::vector fileSplit = cmSystemTools::SplitString(local); - std::vector relativeSplit = cmSystemTools::SplitString(remote); + std::vector fileSplit = cmSystemTools::SplitString(local); + std::vector relativeSplit = cmSystemTools::SplitString(remote); // count up how many mathing directory names there are from the start unsigned int sameCount = 0; while(sameCount < fileSplit.size()-1 && sameCount < relativeSplit.size()-1 && diff --git a/Source/cmSystemTools.h b/Source/cmSystemTools.h index 4aeaf18..939d178 100644 --- a/Source/cmSystemTools.h +++ b/Source/cmSystemTools.h @@ -249,7 +249,7 @@ public: ///! compute the relative path from local to remote static std::string RelativePath(const char* local, const char* remote); ///! split a path by separator into an array of strings, default is / - static std::vector SplitString(const char* s, char separator = '/'); + static std::vector SplitString(const char* s, char separator = '/'); private: static bool s_ForceUnixPaths; static bool s_RunCommandHideConsole; -- cgit v0.12