mirror of
https://github.com/Kitware/CMake.git
synced 2026-01-09 07:11:05 -06:00
cmOutputConverter: collapse ConvertToOutputForExisting functions
This commit is contained in:
@@ -27,10 +27,12 @@ cmOutputConverter::cmOutputConverter(cmState::Snapshot snapshot)
|
||||
assert(this->StateSnapshot.IsValid());
|
||||
}
|
||||
|
||||
std::string cmOutputConverter::ConvertToOutputForExistingCommon(
|
||||
const std::string& remote, std::string const& result,
|
||||
OutputFormat format) const
|
||||
std::string cmOutputConverter::ConvertToOutputForExisting(
|
||||
const std::string& remote, OutputFormat format) const
|
||||
{
|
||||
// Perform standard conversion.
|
||||
std::string result = this->ConvertToOutputFormat(remote, format);
|
||||
|
||||
// If this is a windows shell, the result has a space, and the path
|
||||
// already exists, we can use a short-path to reference it without a
|
||||
// space.
|
||||
@@ -46,16 +48,6 @@ std::string cmOutputConverter::ConvertToOutputForExistingCommon(
|
||||
return result;
|
||||
}
|
||||
|
||||
std::string cmOutputConverter::ConvertToOutputForExisting(
|
||||
const std::string& remote, OutputFormat format) const
|
||||
{
|
||||
// Perform standard conversion.
|
||||
std::string result = this->ConvertToOutputFormat(remote, format);
|
||||
|
||||
// Consider short-path.
|
||||
return this->ConvertToOutputForExistingCommon(remote, result, format);
|
||||
}
|
||||
|
||||
std::string cmOutputConverter::ConvertToOutputForExisting(
|
||||
RelativeRoot remote, OutputFormat format) const
|
||||
{
|
||||
@@ -66,11 +58,7 @@ std::string cmOutputConverter::ConvertToOutputForExisting(
|
||||
const char* remotePath = this->GetRelativeRootPath(remote);
|
||||
assert(remotePath != 0);
|
||||
|
||||
// Perform standard conversion.
|
||||
std::string result = this->ConvertToOutputFormat(remotePath, format);
|
||||
|
||||
// Consider short-path.
|
||||
return this->ConvertToOutputForExistingCommon(remotePath, result, format);
|
||||
return this->ConvertToOutputForExisting(remotePath, format);
|
||||
}
|
||||
|
||||
const char* cmOutputConverter::GetRelativeRootPath(RelativeRoot relroot) const
|
||||
|
||||
@@ -159,10 +159,6 @@ public:
|
||||
private:
|
||||
cmState* GetState() const;
|
||||
|
||||
std::string ConvertToOutputForExistingCommon(const std::string& remote,
|
||||
std::string const& result,
|
||||
OutputFormat format) const;
|
||||
|
||||
static int Shell__CharIsWhitespace(char c);
|
||||
static int Shell__CharNeedsQuotesOnUnix(char c);
|
||||
static int Shell__CharNeedsQuotesOnWindows(char c);
|
||||
|
||||
Reference in New Issue
Block a user