You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@ant.apache.org by bu...@apache.org on 2012/12/09 16:35:24 UTC

[Bug 54147] Creation of the test report (xml, plain) ends with a java.lang.OutOfMemoryError exception

https://issues.apache.org/bugzilla/show_bug.cgi?id=54147

Daniel Trebbien <dt...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |dtrebbien@gmail.com

--- Comment #1 from Daniel Trebbien <dt...@gmail.com> ---
1.  Rather than moving and renaming the encodedata() method, why not make the
necessary changes in place? This way, history is better preserved.

2.  The new implementation is not correct.  It writes out "]]]]><![CDATA[>" as
soon as it sees "]]", but this means that "]]>" is replaced with
"]]]]><![CDATA[>>".

What you could do is write out "]]]]><![CDATA[" instead.  Though, this slightly
changes the behavior (i.e. instead of escaping "]]>", it actually escapes "]]")
and the JavaDoc comment would need to be updated.

3.  Because characters are written individually in the new code, there is
likely a negative impact on performance.

-- 
You are receiving this mail because:
You are the assignee for the bug.