diff --git a/Source/cmLocalVisualStudio10Generator.cxx b/Source/cmLocalVisualStudio10Generator.cxx index 44318f4ed..80f98f2f7 100644 --- a/Source/cmLocalVisualStudio10Generator.cxx +++ b/Source/cmLocalVisualStudio10Generator.cxx @@ -68,7 +68,6 @@ class cmVS10XMLParser : public cmXMLParser //---------------------------------------------------------------------------- cmLocalVisualStudio10Generator::cmLocalVisualStudio10Generator() { - this->NeedXMLEscape = true; } cmLocalVisualStudio10Generator::~cmLocalVisualStudio10Generator() diff --git a/Source/cmLocalVisualStudioGenerator.cxx b/Source/cmLocalVisualStudioGenerator.cxx index da214cf65..4706a966b 100644 --- a/Source/cmLocalVisualStudioGenerator.cxx +++ b/Source/cmLocalVisualStudioGenerator.cxx @@ -26,7 +26,6 @@ cmLocalVisualStudioGenerator::cmLocalVisualStudioGenerator() { this->WindowsShell = true; this->WindowsVSIDE = true; - this->NeedXMLEscape = false; } //---------------------------------------------------------------------------- @@ -232,26 +231,8 @@ cmLocalVisualStudioGenerator } else { - if(this->NeedXMLEscape) - { - std::string arg = commandLine[j]; - cmSystemTools::ReplaceString(arg, "&", "&"); - cmSystemTools::ReplaceString(arg, "<", "<"); - cmSystemTools::ReplaceString(arg, ">", ">"); - if(arg.find(" ") != arg.npos) - { - std::string q("\""); - arg = q + arg +q; - } - script += arg; - //script += this->EscapeForShell(arg.c_str(), - //escapeAllowMakeVars); - } - else - { - script += this->EscapeForShell(commandLine[j].c_str(), + script += this->EscapeForShell(commandLine[j].c_str(), escapeAllowMakeVars); - } } } } diff --git a/Source/cmLocalVisualStudioGenerator.h b/Source/cmLocalVisualStudioGenerator.h index 4b31599bf..a6bdfbf80 100644 --- a/Source/cmLocalVisualStudioGenerator.h +++ b/Source/cmLocalVisualStudioGenerator.h @@ -56,7 +56,6 @@ protected: std::map& count); void InsertNeedObjectNames(const std::vector& groups, std::map& count); - bool NeedXMLEscape; std::set NeedObjectName; friend class cmVisualStudio10TargetGenerator; }; diff --git a/Source/cmVisualStudio10TargetGenerator.cxx b/Source/cmVisualStudio10TargetGenerator.cxx index 2a969327f..e8aaf6162 100644 --- a/Source/cmVisualStudio10TargetGenerator.cxx +++ b/Source/cmVisualStudio10TargetGenerator.cxx @@ -27,6 +27,13 @@ #include "cmVS10LinkFlagTable.h" #include "cmVS10LibFlagTable.h" +static std::string cmVS10EscapeXML(std::string arg) +{ + cmSystemTools::ReplaceString(arg, "&", "&"); + cmSystemTools::ReplaceString(arg, "<", "<"); + cmSystemTools::ReplaceString(arg, ">", ">"); + return arg; +} cmVisualStudio10TargetGenerator:: cmVisualStudio10TargetGenerator(cmTarget* target, @@ -289,12 +296,14 @@ cmVisualStudio10TargetGenerator::WriteCustomRule(cmSourceFile* source, for(std::vector::iterator i = configs->begin(); i != configs->end(); ++i) { - std::string script = - lg->ConstructScript(command.GetCommandLines(), - command.GetWorkingDirectory(), - i->c_str(), - command.GetEscapeOldStyle(), - command.GetEscapeAllowMakeVars()); + std::string script = + cmVS10EscapeXML( + lg->ConstructScript(command.GetCommandLines(), + command.GetWorkingDirectory(), + i->c_str(), + command.GetEscapeOldStyle(), + command.GetEscapeAllowMakeVars()) + ); this->WritePlatformConfigTag("Message",i->c_str(), 3); (*this->BuildFileStream ) << comment << "\n"; this->WritePlatformConfigTag("Command", i->c_str(), 3); @@ -1202,12 +1211,14 @@ void cmVisualStudio10TargetGenerator::WriteEvent( comment += lg->ConstructComment(command); script += pre; pre = "\n"; - script += - lg->ConstructScript(command.GetCommandLines(), - command.GetWorkingDirectory(), - configName.c_str(), - command.GetEscapeOldStyle(), - command.GetEscapeAllowMakeVars()); + script += + cmVS10EscapeXML( + lg->ConstructScript(command.GetCommandLines(), + command.GetWorkingDirectory(), + configName.c_str(), + command.GetEscapeOldStyle(), + command.GetEscapeAllowMakeVars()) + ); } this->WriteString("",3); (*this->BuildFileStream ) << comment << "\n";