From f0ab852000b5c1ea17509c6e57e61942ba878984 Mon Sep 17 00:00:00 2001 From: Brad King Date: Thu, 11 Jun 2009 09:03:56 -0400 Subject: [PATCH] COMP: Fix build with system-installed expat 2.0.1 In cmXMLParser::ReportXmlParseError we were accidentally passing a value of type 'XML_Parser*' to expat methods instead of 'XML_Parser'. It was not caught because XML_Parser was just 'void*' in the cmexpat version. Newer system-installed expat versions catch the error because XML_Parser is now a pointer to a real type. This correct the type. --- Source/cmXMLParser.cxx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Source/cmXMLParser.cxx b/Source/cmXMLParser.cxx index 0f68cb617..db9a27c5e 100644 --- a/Source/cmXMLParser.cxx +++ b/Source/cmXMLParser.cxx @@ -228,7 +228,7 @@ void cmXMLParserCharacterDataHandler(void* parser, const char* data, //---------------------------------------------------------------------------- void cmXMLParser::ReportXmlParseError() { - XML_Parser* parser = static_cast(this->Parser); + XML_Parser parser = static_cast(this->Parser); this->ReportError(XML_GetCurrentLineNumber(parser), XML_GetCurrentColumnNumber(parser), XML_ErrorString(XML_GetErrorCode(parser)));