Merge branch 'fix-ctest-xml-double-encoding' into release

This commit is contained in:
Brad King 2015-08-28 10:15:14 -04:00
commit f4bdda87d6
8 changed files with 8 additions and 20 deletions

View File

@ -14,7 +14,6 @@
#include "cmCTest.h" #include "cmCTest.h"
#include "cmSystemTools.h" #include "cmSystemTools.h"
#include "cmXMLParser.h" #include "cmXMLParser.h"
#include "cmXMLSafe.h"
#include <cmsys/RegularExpression.hxx> #include <cmsys/RegularExpression.hxx>

View File

@ -14,7 +14,6 @@
#include "cmCTest.h" #include "cmCTest.h"
#include "cmSystemTools.h" #include "cmSystemTools.h"
#include "cmAlgorithms.h" #include "cmAlgorithms.h"
#include "cmXMLSafe.h"
#include <cmsys/RegularExpression.hxx> #include <cmsys/RegularExpression.hxx>
#include <cmsys/ios/sstream> #include <cmsys/ios/sstream>

View File

@ -80,8 +80,8 @@ public:
int i = 0; int i = 0;
for(; atts[i] != 0; i+=2) for(; atts[i] != 0; i+=2)
{ {
ostr << " " << cmXMLSafe(atts[i]) ostr << " " << atts[i]
<< " - " << cmXMLSafe(atts[i+1]) << "\n"; << " - " << atts[i+1] << "\n";
} }
ostr << "\n"; ostr << "\n";
this->Log += ostr.str(); this->Log += ostr.str();
@ -856,7 +856,7 @@ bool cmCTestMemCheckHandler::ProcessMemCheckSanitizerOutput(
defects++; defects++;
ostr << "<b>" << this->ResultStrings[idx] << "</b> "; ostr << "<b>" << this->ResultStrings[idx] << "</b> ";
} }
ostr << cmXMLSafe(*i) << std::endl; ostr << *i << std::endl;
} }
log = ostr.str(); log = ostr.str();
if(defects) if(defects)
@ -908,7 +908,7 @@ bool cmCTestMemCheckHandler::ProcessMemCheckPurifyOutput(
results[failure] ++; results[failure] ++;
defects ++; defects ++;
} }
ostr << cmXMLSafe(*i) << std::endl; ostr << *i << std::endl;
} }
log = ostr.str(); log = ostr.str();
@ -1056,7 +1056,7 @@ bool cmCTestMemCheckHandler::ProcessMemCheckValgrindOutput(
defects ++; defects ++;
} }
totalOutputSize += lines[cc].size(); totalOutputSize += lines[cc].size();
ostr << cmXMLSafe(lines[cc]) << std::endl; ostr << lines[cc] << std::endl;
} }
else else
{ {
@ -1070,11 +1070,7 @@ bool cmCTestMemCheckHandler::ProcessMemCheckValgrindOutput(
nonValGrindOutput.begin(); i != nonValGrindOutput.end(); ++i) nonValGrindOutput.begin(); i != nonValGrindOutput.end(); ++i)
{ {
totalOutputSize += lines[*i].size(); totalOutputSize += lines[*i].size();
cmCTestOptionalLog(this->CTest, DEBUG, "before xml safe " ostr << lines[*i] << std::endl;
<< lines[*i] << std::endl, this->Quiet);
cmCTestOptionalLog(this->CTest, DEBUG, "after xml safe "
<< cmXMLSafe(lines[*i]) << std::endl, this->Quiet);
ostr << cmXMLSafe(lines[*i]) << std::endl;
if(!unlimitedOutput && totalOutputSize > if(!unlimitedOutput && totalOutputSize >
static_cast<size_t>(this->CustomMaximumFailedTestOutputSize)) static_cast<size_t>(this->CustomMaximumFailedTestOutputSize))
{ {

View File

@ -13,7 +13,6 @@
#include "cmCTest.h" #include "cmCTest.h"
#include "cmSystemTools.h" #include "cmSystemTools.h"
#include "cmXMLSafe.h"
#include <cmsys/RegularExpression.hxx> #include <cmsys/RegularExpression.hxx>
#include <cmsys/ios/sstream> #include <cmsys/ios/sstream>

View File

@ -2070,7 +2070,7 @@ bool cmCTestTestHandler::CleanTestOutput(std::string& output, size_t length)
} }
current = next; current = next;
} }
else // Bad byte will be handled by cmXMLSafe. else // Bad byte will be handled by cmXMLWriter.
{ {
++current; ++current;
} }

View File

@ -22,7 +22,6 @@
#include <cmsys/FStream.hxx> #include <cmsys/FStream.hxx>
#include "cmDynamicLoader.h" #include "cmDynamicLoader.h"
#include "cmGeneratedFileStream.h" #include "cmGeneratedFileStream.h"
#include "cmXMLSafe.h"
#include "cmVersionMacros.h" #include "cmVersionMacros.h"
#include "cmCTestCommand.h" #include "cmCTestCommand.h"
#include "cmCTestStartCommand.h" #include "cmCTestStartCommand.h"
@ -156,7 +155,7 @@ std::string cmCTest::CurrentTime()
strftime(current_time, 1000, "%a %b %d %H:%M:%S %Z %Y", t); strftime(current_time, 1000, "%a %b %d %H:%M:%S %Z %Y", t);
} }
cmCTestLog(this, DEBUG, " Current_Time: " << current_time << std::endl); cmCTestLog(this, DEBUG, " Current_Time: " << current_time << std::endl);
return cmXMLSafe(cmCTest::CleanString(current_time)).str(); return cmCTest::CleanString(current_time);
} }
//---------------------------------------------------------------------- //----------------------------------------------------------------------
@ -1477,8 +1476,6 @@ std::string cmCTest::SafeBuildIdField(const std::string& value)
cmSystemTools::ReplaceString(safevalue, replace, ""); cmSystemTools::ReplaceString(safevalue, replace, "");
} }
} }
safevalue = cmXMLSafe(safevalue).str();
} }
if (safevalue == "") if (safevalue == "")

View File

@ -19,7 +19,6 @@
#include "cmGeneratedFileStream.h" #include "cmGeneratedFileStream.h"
#include "cmTarget.h" #include "cmTarget.h"
#include "cmSystemTools.h" #include "cmSystemTools.h"
#include "cmXMLSafe.h"
#include <cmsys/SystemTools.hxx> #include <cmsys/SystemTools.hxx>

View File

@ -21,7 +21,6 @@
#include "cmSourceFile.h" #include "cmSourceFile.h"
#include "cmSystemTools.h" #include "cmSystemTools.h"
#include "cmTarget.h" #include "cmTarget.h"
#include "cmXMLSafe.h"
#include <cmsys/SystemTools.hxx> #include <cmsys/SystemTools.hxx>