Menu

CondenseWhiteSpace bug?

Developer
2004-12-21
2013-05-20
  • Alexey Sokirko

    Alexey Sokirko - 2004-12-21

    Thank you for the library, but I have a problem.
    The following program causes an infinite cycle in tinyXml 2.3.2:

    {
      TiXmlDocument doc;
       doc.SetCondenseWhiteSpace(false);
       doc.Parse("<p><pb></pb>test</p>");
    }
    If one deletes the second line (with "SetCondenseWhiteSpace"), then it works perfectly. Is it a bug?

     
    • Alexey Sokirko

      Alexey Sokirko - 2004-12-22

      Am I right, that this bug should be fixed by inserting "pWithWhiteSpace = p;"
      into line 1160 in tinyxmlparser.cpp after
      "p = node->Parse( p, data, encoding );" ?
      No other places should be edited. Am I right?

       
      • Lee Thomason

        Lee Thomason - 2005-02-09

        I suspect you are - could you file a bug, and include the fix? It really helps me keep track of issues that need to be fixed - and this one really does.

        Thanks!
        lee

         
    • XFilez

      XFilez - 2005-02-01

      I'm having the same problem in tinyxml 2.3.2 and 2.3.3. In 2.3.2 I can modify tinyxmlparser.cpp as sokirko said, and it outputs  newlines as  &#x0A;&#x09; .  In 2.3.3 modifying the same line doesn't work, the parser still hangs in an infinite loop somewhere. 

       
    • Martin Robinson

      Martin Robinson - 2005-07-12

      Any chance of updating the download file to include this fix ?

      I don't really want to install (and learn how to use) the CVS software to process a .diff file.

      Thanks.

       
      • Ellers

        Ellers - 2005-07-13

        reply:

        nah, its easy :) And it will come in handy later anyway ;)

        you shouldn't necessarily need to download the CVS tree.

        Just do 'man patch' on a linux machine or in a cygwin window on Windows.

        HTH
        Ellers

         
    • Lee Thomason

      Lee Thomason - 2005-07-24

      This will be fixed in the next version (and should be fixed currently in CVS.) I am very busy these days - it could be a few weeks before new files are released.

       

Log in to post a comment.

MongoDB Logo MongoDB