Developer forums (C::B DEVELOPMENT STRICTLY!) > Development

TinyXml 2.5.3

(1/2) > >>

killerbot:
yesterday TinyXml in CB has been updated from 2.5.2 to 2.5.3.

Today I discovered a bug in TinyXml, it happens with CDATA and escaping of all '<' and '>'

I think we should revert to 2.5.2 for now.

Other people any ideas/suggestions

mandrav:

--- Quote from: killerbot on May 11, 2007, 03:31:38 pm ---yesterday TinyXml in CB has been updated from 2.5.2 to 2.5.3.

Today I discovered a bug in TinyXml, it happens with CDATA and escaping of all '<' and '>'

I think we should revert to 2.5.2 for now.

Other people any ideas/suggestions

--- End quote ---

Revert it then.

killerbot:
I have found the bug (well I think it is a bug) in TinyXml . I will keep the update but apply the fix :


--- Code: ---bool TiXmlPrinter::Visit( const TiXmlText& text )
{
if ( text.CDATA() )
{
DoIndent();
buffer += "<![CDATA[";
buffer += text.Value();
buffer += "]]>";
DoLineBreak();
}
else if ( simpleTextPrint )
{
buffer += text.Value();
// TIXML_STRING str;
// TiXmlBase::EncodeString( text.ValueTStr(), &str );
// buffer += str;
}
else
{
DoIndent();
buffer += text.Value();
// TIXML_STRING str;
// TiXmlBase::EncodeString( text.ValueTStr(), &str );
// buffer += str;
DoLineBreak();
}
return true;
}

--- End code ---


buffer += text.Value();    was replaced in the new version by the lines I have commented out now. I will investigate further.

killerbot:
good news, it is probably a false alarm, and all is well.
Will do last test and adjust tomorrow   :P

MortenMacFly:

--- Quote from: killerbot on May 11, 2007, 10:57:58 pm ---good news, it is probably a false alarm, and all is well.
Will do last test and adjust tomorrow   :P

--- End quote ---
I would have wondered otherwise. Because I have carefully compared both revisions and truly there was not much difference except bug fixing. (You can checkout both versions yourself to see). That's why I've applied the update on SVN.
With regards, Morten.

Navigation

[0] Message Index

[#] Next page